Skip to content

Avoid change chmod by using Nette\Utils\FileSystem with pass null on 3rd argument on write #209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 28, 2024

Conversation

samsonasik
Copy link
Collaborator

@samsonasik samsonasik commented May 24, 2024

@samsonasik
Copy link
Collaborator Author

@vasilvestre could you verify this patch? Thank you.

@@ -4,6 +4,7 @@

namespace Symplify\EasyCodingStandard\Caching\ValueObject\Storage;

use Nette\Utils\FileSystem as UtilsFileSystem;
Copy link
Collaborator Author

@samsonasik samsonasik May 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vasilvestre for manual test, with directly change on vendor, change it with prefixed use statement:

use ECSPrefix202405\Nette\Utils\FileSystem as UtilsFileSystem;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will try soon

@vasilvestre
Copy link

It's not working for me, the ownership still changes to root:root despite an original user with UID 1000

@samsonasik
Copy link
Collaborator Author

I am not sure why it can change ownership, chaning chmod is what we seen in the past, but ownership is strange as it should not know what new one, it probably related with somewhere on your php setting and executable as user you run.

I suggest to not run on docker as it may be the root of the issue.

I will may still merge it to sync with rector.

@samsonasik
Copy link
Collaborator Author

@TomasVotruba I am merging it so we have sync usage with rector ;)

@samsonasik samsonasik merged commit d82d8ee into main May 28, 2024
7 checks passed
@samsonasik samsonasik deleted the avoid-change-chmod branch May 28, 2024 08:54
@vasilvestre
Copy link

I am not sure why it can change ownership, chaning chmod is what we seen in the past, but ownership is strange as it should not know what new one, it probably related with somewhere on your php setting and executable as user you run.

I suggest to not run on docker as it may be the root of the issue.

I will may still merge it to sync with rector.

Not an option in our workflow. Thank you for the test tho !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants