Skip to content

Unnecessary permission updates for files #45

@rajsite

Description

@rajsite

When using upload-pages-artifacts@v1 I get warnings about updating permissions for the following files from permissions 0744 to 0755:

Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/dotnet.wasm' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/dotnet.6.0.11.agdggo08ph.js' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/icudt_EFIGS.dat' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/Faker.NETCore.dll' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/icudt_CJK.dat' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/icudt_no_CJK.dat' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/blazor.webassembly.js' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/icudt.dat' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/de-DE/Faker.NETCore.resources.dll' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)
Warning: mode of 'packages/site/dist/storybook/blazor-client-app/wwwroot/_framework/dotnet.timezones.blat' changed from 0744 (rwxr--r--) to 0755 (rwxr-xr-x)

I don't fully understand why the permissions of files are being changed as it looks like the command is targeting directories:

chmod -c -R +rX "$INPUT_PATH" | while read line; do

Regardless, for the use case I am hitting maybe the action should just update permissions for the current user (which these files already had the correct permissions for)?:

chmod -c -R u+rX "$INPUT_PATH" | while read line; do

Another alternative would be to add a parameter to the action to suppress the warnings in my summary logs (I don't mind them in the logs but the ::warning annotation is making the build summary noisy).

Metadata

Metadata

Assignees

Labels

permissionsRelated to file permissions

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions