[stable9] Properly convert OCS params + more tests#26695
Closed
PVince81 wants to merge 757 commits into
Closed
Conversation
…sed-for-fallback-languages-stable9 [stable9] Do not use the language name, when it's not the one from the language…
A share can only be requested by 3 'types' of people * owner * initiator * recipient So we have to get the path as the current user. Since that is the only path that has any meaning to the user. - Fixed tests
…-installing-on-oracle-via-cli [stable9] Allow empty host when installing on oracle via CLI (#25034)
…orkaround [stable9] decrease initial users to load to 50
…cesslist [stable9] Use capped cache for encryption's user access list
[stable9] DAV now returns file name with Content-Disposition header
[stable9] Capped cache for cache info in UserMountCache
[9.0] fix grouped input fields, make sure they take precedence
[stable9] Convert Dropbox Forbidden exception to StorageNotAvailableException
[stable9] load authentication apps first
�emit correct signal when disabling an app
* Might fix an issue on oracle
Do not recurse link share fetching
This moves registerMountProviders until after the sharing backends were registered. In some situations registerMountProviders will trigger listeners which might require filesystem access which itself would mount shares, which itself requires the sharing backends to be initialized.
…ementation is loaded - fixes owncloud/objectstore#38
This prevents a validation failure where the code checks whether a file is already shared with another user, but the check disables itself when the owner is the same. Processing the groups first prevents the check to kick in too early when the group share still has the old owner while we try transferring the user share.
…e_info-patch [stable9] Fixed disappearing of share info in file view
…storagefix [stable9] Transfer ownership only analyze home storage
When creating link shares from external storage, the filesystem cannot find an owner in some scenarios (ex: system-wide mounts). In such cases, fall back to using the current user's trashbin which happens to also be the user who created the link share. Fixes an issue where this scenario made deletion impossible due to missing user information.
…271a98dc3f6d19f1a71 [stable9] We need to ensure that the app holding the objectstore impl…
…792c485f728925c2469 [stable9] Fallback to share link owner when no owner found (#26587)
…-master [stable9] behat will only only pass if all tests pass
[stable9] Add repair step to fix file share permissions
Conflicts: build/integration/features/bootstrap/BasicStructure.php
Stable9 integration tests qa 222
…ceptions [stable9] Skip broken shares when transferring ownership
* Properly expire ext storage versions System-wide external storages have no real owner so the current user is used as owner. However when running cron.php there is no current user, so no expiry can be done. This fix adds an user argument to the expire() function to tell for which user to expire files. This information is anyway always available now through the expire command job. * Move version expire setupFS into the expire function * Add comment about not tearing down in version Storage::expire()
…sions [stable9] Properly expire ext storage versions (#26601)
Added read and share permissions instead of all (default ones) Added positive case Check also permissions update Added tests for resharing with public links when the user doesn't have enough permisssions
Contributor
Author
|
ohhhhhh... |
|
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. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
18 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport of #26691 to stable9.
Retested manually, still works.
Note that I had to change the perms in the test from 15 to 7 because in OC <= 9.0 the public upload perms were missing the DELETE permission (known case that was adjusted in 9.1+).
@jvillafanez @SergioBertolinSG @DeepDiver1975