Skip to content

Commit 29fa29f

Browse files
committedJul 10, 2024·
change variable name hasPermissions
1 parent c970a1b commit 29fa29f

File tree

1 file changed

+6
-8
lines changed

1 file changed

+6
-8
lines changed
 

‎Snowflake.Data/Configuration/EasyLoggingConfigParser.cs

+6-8
Original file line numberDiff line numberDiff line change
@@ -108,19 +108,17 @@ private void CheckIfValidPermissions(FileStream fileStream, string filePath)
108108

109109
#if NET8_0_OR_GREATER
110110
var unixFileMode = File.GetUnixFileMode(fileStream.SafeFileHandle);
111-
var hasPermissions = !(((UnixFileMode.GroupWrite | UnixFileMode.OtherWrite) & unixFileMode) != 0);
111+
var hasGroupOrOtherWritePermissions = (((UnixFileMode.GroupWrite | UnixFileMode.OtherWrite) & unixFileMode) != 0);
112112
#else
113113
var entitlements = FileAccessPermissions.GroupWrite | FileAccessPermissions.OtherWrite;
114-
var hasPermissions = !_unixOperations.CheckFileHasAnyOfPermissions(filePath, entitlements);
114+
var hasGroupOrOtherWritePermissions = _unixOperations.CheckFileHasAnyOfPermissions(filePath, entitlements);
115115
#endif
116-
if (hasPermissions)
116+
if (hasGroupOrOtherWritePermissions)
117117
{
118-
return;
118+
var errorMessage = $"Error due to other users having permission to modify the config file: {filePath}";
119+
s_logger.Error(errorMessage);
120+
throw new Exception(errorMessage);
119121
}
120-
121-
var errorMessage = $"Error due to other users having permission to modify the config file: {filePath}";
122-
s_logger.Error(errorMessage);
123-
throw new Exception(errorMessage);
124122
}
125123
}
126124
}

0 commit comments

Comments
 (0)
Please sign in to comment.