Skip to content

Commit 808407a

Browse files
SNOW-1660409: Fix directory permission for GET (#2080)
1 parent 31d6a9c commit 808407a

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

src/main/java/net/snowflake/client/jdbc/SnowflakeUtil.java

+4-3
Original file line numberDiff line numberDiff line change
@@ -73,8 +73,8 @@ public class SnowflakeUtil {
7373
private static final SFLogger logger = SFLoggerFactory.getLogger(SnowflakeUtil.class);
7474
private static final ObjectMapper OBJECT_MAPPER = ObjectMapperFactory.getObjectMapper();
7575

76-
private static final Set<PosixFilePermission> ownerOnlyPermission =
77-
PosixFilePermissions.fromString("rw-------");
76+
private static final Set<PosixFilePermission> directoryOwnerOnlyPermission =
77+
PosixFilePermissions.fromString("rwx------");
7878

7979
/** Additional data types not covered by standard JDBC */
8080
public static final int EXTRA_TYPES_TIMESTAMP_LTZ = 50000;
@@ -922,7 +922,8 @@ public static boolean createOwnerOnlyPermissionDir(String location) {
922922
boolean isDirCreated = true;
923923
Path dir = Paths.get(location);
924924
try {
925-
Files.createDirectory(dir, PosixFilePermissions.asFileAttribute(ownerOnlyPermission));
925+
Files.createDirectory(
926+
dir, PosixFilePermissions.asFileAttribute(directoryOwnerOnlyPermission));
926927
} catch (IOException e) {
927928
logger.error(
928929
"Failed to set OwnerOnly permission for {}. This may cause the file download to fail ",

src/test/java/net/snowflake/client/jdbc/SnowflakeUtilTest.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -139,7 +139,7 @@ public void testCreateOwnerOnlyPermissionDir(@TempDir Path tempDir)
139139
assertTrue(tmp.toFile().isDirectory());
140140
PosixFileAttributes attributes = Files.readAttributes(tmp, PosixFileAttributes.class);
141141
Set<PosixFilePermission> permissions = attributes.permissions();
142-
assertEquals(PosixFilePermissions.toString(permissions), "rw-------");
142+
assertEquals(PosixFilePermissions.toString(permissions), "rwx------");
143143
}
144144

145145
@Test

0 commit comments

Comments
 (0)