Uploaded image for project: 'GeoServer'
  1. GEOS-8851

importer uploads cleanup task deletes raster files for existing installs


    • Type: Bug
    • Status: Resolved
    • Priority: High
    • Resolution: Fixed
    • Affects Version/s: 2.14-RC
    • Fix Version/s: 2.15-M0
    • Component/s: Importer
    • Labels:


      The fix for GEOS-8649 Closed made in the commit
      here causes all existing raster uploads to be deleted when the scheduled importer cleaner task is run in the JobQueue class because existing upload directories do not contain the .locked file thats used for checking.

      This commit might also cause issues for cases where you're using the importers import from directory feature since you might upload to that folder from an external app and not have a .locked file created there (though you could just create a .locked file from your app).

      A clean way to fix this might be to have a separate ".clean-me" file created for all new uploads and the cleaner task ignores any upload folders without it. This would avoid any messiness with a migration script for existing installs and solves the second minor issue with uploads from external services neatly.

      (We actually have our own upload cleaner script we wrote separate from geoserver to solve this and upload files separately and use import from directory in the importer rest call. Just removing this new code from a forked geosever repo fixes it for us completely so I'm not going to fix this myself and don't need it fixed for us I just felt you guys might want to know about a potential user data deleting issue)




            • Assignee:
              afabiani Alessio Fabiani
              maddie.abboud Maddie Abboud
            • Votes:
              0 Vote for this issue
              2 Start watching this issue


              • Created: