-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Move public webdav auth to share manager #23736
Move public webdav auth to share manager #23736
Conversation
Mmmm now the other unit tests fall appart... but why... |
2445414
to
12d938e
Compare
The infamous incognito mode strikes again... fixed.. |
Ok so apparently I broke one of the intergration tests... will take care. |
12d938e
to
bb58642
Compare
Ok all should be fine now! Review please |
Calling once again on the review team: @PVince81 @schiesbn @DeepDiver1975 @nickvergessen |
// Set old user | ||
\OC_User::setUserId($this->oldUser); | ||
\OC_Util::setupFS($this->oldUser); | ||
} |
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.
missing parent::tearDown
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.
fixed
code looks good beside the stuff mentioned by @nickvergessen 👍 |
bb58642
to
03f9636
Compare
@nickvergessen fixed your comments |
Works, can still access password protected shares and also mount them as remote shares 👍 |
If we do not allow public upload we should limit the permissions on links shares upon retrieval. * Added unit test * Allow fetching federated shares by token as well
The public webdav auth should use the shiny new share manager.
058d80f
to
eda7124
Compare
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
For #22209
This moves over the webdav public auth to the sharemanager.
As a bonus we now finally have (at least some) unit tests for the webdav public auth!
CC: @PVince81 @schiesbn @DeepDiver1975 @nickvergessen