Skip to content
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.

Sign up for GitHub

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

Jump to bottom

NFS-friendly is_writable() checks #4780

Merged
merged 1 commit into from
Nov 14, 2022
Merged

NFS-friendly is_writable() checks #4780

merged 1 commit into from
Nov 14, 2022

Conversation

Alkarex
Copy link
Member

@Alkarex Alkarex commented Oct 26, 2022

#fix #4779

@Alkarex Alkarex added this to the 1.20.1 milestone Oct 26, 2022
@Alkarex Alkarex mentioned this pull request Oct 26, 2022
Closed
@Alkarex
Copy link
Member Author

Alkarex commented Oct 26, 2022

P.S. touching the index.html files is actually a minor positive feature, as it ensures they exist, which is good for avoiding listing directories on badly configured servers.

@Alkarex Alkarex modified the milestones: 1.20.1, 1.21.0 Oct 30, 2022
@Alkarex Alkarex merged commit 8864d51 into FreshRSS:edge Nov 14, 2022
@Alkarex Alkarex deleted the robust-is_writable branch November 14, 2022 13:58
anders-larsson added a commit to anders-larsson/gentoo-overlay that referenced this pull request Mar 4, 2023
@anders-larsson
freshrss: remove index.html files
08673ba 
Freshrss 1.21.0 and newer checks if it can write to index.html in several
directories. Every time webapp-config runs it reinstalls these files with
the webserver user as their owner. RSS feed refresh script fails to run
if PHP user cannot write to the required files causing the refresh to fail.
Introduced by FreshRSS/FreshRSS#4780.
@anders-larsson anders-larsson mentioned this pull request Mar 4, 2023
Merged
anders-larsson added a commit to anders-larsson/gentoo-overlay that referenced this pull request Mar 4, 2023
@anders-larsson
freshrss: remove index.html files
dae3d03 
Freshrss 1.21.0 and newer checks if it can write to index.html in several
directories. Every time webapp-config runs it reinstalls these files with
the webserver user as their owner. RSS feed refresh script fails to run
if PHP user cannot write to the required files causing the refresh to fail.
Introduced by FreshRSS/FreshRSS#4780.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.21.0
Development

Successfully merging this pull request may close these issues.

[BUG] check if (data) directory is writable fails on NFS
1 participant
@Alkarex