Fix file permissions problem. #329

Open-EO · Oct 17, 2024 · 704ddb3 · 704ddb3
1 parent 945d360
commit 704ddb3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/openeo-geotrellis/src/main/scala/org/openeo/geotrellis/geotiff/package.scala b/openeo-geotrellis/src/main/scala/org/openeo/geotrellis/geotiff/package.scala
@@ -836,7 +836,7 @@ package object geotiff {
       uploadToS3(tempFile, correctS3Path)
 
     } else {
-      val tempFile = Files.createTempFile(null, ".tif")
+      val tempFile = getTempFile(null, ".tif")
       // TODO: Try to run fsync on the file opened by GeoTrellis (without the temporary copy)
       geoTiff.write(tempFile.toString, optimizedOrder = true)
 

diff --git a/openeo-geotrellis/src/main/scala/org/openeo/geotrellis/package.scala b/openeo-geotrellis/src/main/scala/org/openeo/geotrellis/package.scala
@@ -128,7 +128,9 @@ package object geotrellis {
   }
 
   /**
-   * Inspired on 'Files.createTempFile'
+   * Inspired on 'Files.createTempFile', but does not create an empty file.
+   * The default permissions of 'createTempFile' are a bit too strict too: 600, which is not accessible by other users.
+   * This function could have default 664 for example.
    */
   def getTempFile(prefix: String, suffix: String): Path = {
     val prefixNonNull = if (prefix == null) "" else suffix