-
-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Fix path of file_get_contents
#28473
Conversation
Does this also work with anything other than a local filesystem? |
This comment has been minimized.
This comment has been minimized.
Does it work? ;) The "faulty" path rewrite is already happening in getLocalPath. Trigger this faulty part / path is a bit tricky 🙈
I don't think it has too. All other methods / cases in the above switch-case are working with the local filesystem directly. No Nextcloud Api is used to access a image file.
|
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 🐘
Hey @ChristophWurst can this be backported? ;) |
/backport to stable22 |
/backport to stable21 |
/backport to stable20 |
The backport to stable20 failed. Please do this backport manually. |
The backport to stable21 failed. Please do this backport manually. |
Try to fix #28370 and fix #27441
Avoid pre-pends the
$path
to the user's own storage.