| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| | |
[stable9.1] Fix user casing in initMountPoints
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| | |
owncloud/analyse-dependencies-on-app-enable-as-well
App dependencies are now analysed on app enable as well - not only on app install.
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
* Adjust unit test execution after folder rename
* Adjust login controller tests to match current behavior
* Fix broken unit tests
|
|/
|
|
|
|
|
|
|
|
| |
#25603 (#25968)
* Don't parse info.xml but reuse already cached app infos - fixes #25603
* Use === in InfoParser. Fixes test
* InfoParser should not depend on UrlGenerator - fixes issue with session being closed too early
|
|
|
|
|
|
|
| |
The check for two factor enforcement would return true for non-existing
users. This fix makes it return false in order to be able to perform
the regular login which will then fail and return false.
This prevents throwing PasswordLoginForbidden for non-existing users.
|
|
|
|
|
|
| |
* Log the reason of occ controller failure
* Allow requests from SERVER_ADDR
|
|\
| |
| |
| |
| | |
owncloud/stable9.1-c97f24efbaec7756667368568b0c5d7b506d864f
[stable9.1] Redirect to challenge page when only one 2FA provider
|
| |
| |
| |
| |
| | |
If only one two factor provider exists, spare the user from having to
select it and redirect directly to its challenge page.
|
|/ |
|
|
|
|
|
|
|
|
|
| |
The DefaultShareProvider now does a DB-level check to find out whether
file_source is accessible at all (deleted file) or whether it's in the
trashbin of a home storage.
One small corner case where the home storage id is in md5 form cannot
be covered properly with this approach.
|
| |
|
|
|
|
|
| |
* Don't log credentials of LoginController::tryLogin - fixes #25895
* Don't log password in loginWithPassword
|
| |
|
|
|
|
|
|
| |
* Take from https://github.com/nextcloud/server/pull/608/commits/351cab6bce41b53f9efd4ba9aed4e7435f843691 - THX @schiessle
* Fix unit tests
|
| |
|
|
|
|
|
|
|
| |
Whenever a group share is created after a direct share, the stime order
needs to be properly considered in the repair routine, considering that
the direct user share is appended to the $subShares array and breaking
its order.
|
|
|
|
|
|
|
| |
Pick the most recent subshare that has no parenthesis from duplication
which should match whichever name the user picked last. If all
subshares have duplicate parenthesis names, use the least recent group
share's target instead.
|
|\
| |
| |
| |
| | |
owncloud/stable9.1-99611d277521b84a943f68d48fe05f290127db7e
[stable9.1] getJailedPath expects $path to have a trailing / (#25703)
|
| |
| |
| |
| |
| |
| | |
* getJailedPath expects $path to have a trailing / - fixes #25464
* Adding test case for getPathById including a jailed cache where root is just empty
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* [stable9.1] Next step jenkinsfile (#25622)
* Adding timestamper and evaluation of test results even in case of failure
* Adding build timeout
* use fixed value 120 minutes as timeout for each test executing for now
* Terminate the build as soon as test execution fails
* Adjust external testing as well
* Finalize use of executeAndReport
* [stable9.1] Fix discoverymanager tests (#25769)
* Allow testing of the DiscoveryManager
* Array sort order is of no relevance
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Group shares with same source and target
Fixes #24575
Note that this is a very limited solution and eventually we want smarter
merging!
* Add integration tests for merging received shares
* Improved share grouping readability + fixed test
* Add repair step for unmerged shares (WIP)
* Added more tests for sharing's MountProvider
* Group incoming shares for resharing in JS
* Adjust repair version check for unmerged shares
* Fix RepairUnmergedShares to not skip valid repair cases
The repair step was a bit overeager to skip repairing so it missed the
case where a group share exists without subshares but with an
additional direct user share.
* Add integration tests for double shares with rename in between
* Make share target consistent when grouping group share with user share
In some situations, a group share is created before a user share, and
the recipient renamed the received share before the latter is created.
In this situation, the "file_target" was already modified and the second
created share must align to the already renamed share.
To achieve this, the MountProvider now groups only by "item_source"
value and sorts by share time. This makes it so that the least recent
share is selected as super-share and its "file_target" value is then
adjusted in all grouped shares.
This fixes the issue where this situation would have different
"file_target" values resulting in two shared folders appearing instead
of one.
|
|
|
|
| |
(#25517) (#25595)
|
| |
|
|
|
|
|
|
|
|
| |
certificate is revoked (#25468)
* Adding certificate revocation list and validate if the app certificate is revoked
* Check integrity of a signed app in any case on installation
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Revert "Merge pull request #25240 from owncloud/remove-svg"
This reverts commit 8b8d2b679a12f703141de9da71340f1f02151f3d, reversing
changes made to a35747b6fa8f7704bf7333f16a2b867b76acb187.
* Revert "Merge pull request #25253 from owncloud/users-fixotherquotadropdown"
This reverts commit a35747b6fa8f7704bf7333f16a2b867b76acb187, reversing
changes made to a573b6863cbb4eb21feb7fe5a17a9b8fc60cf059.
* Revert "Merge pull request #25314 from owncloud/files_external-backends-config"
This reverts commit a573b6863cbb4eb21feb7fe5a17a9b8fc60cf059, reversing
changes made to 8147eefaeba3822c9331283e973006ad3aeac1f5.
* Revert "Add all properties while creating a subscription (#25318)"
This reverts commit aaf4c3073af2511ab895e982cadae8dc6a143e55.
* Revert "Merge pull request #25276 from owncloud/delete-own-session-token"
This reverts commit e42ce62ce2855c95861eeae669508e5c20f99be4, reversing
changes made to aaf4c3073af2511ab895e982cadae8dc6a143e55.
* Revert "Merge pull request #25262 from owncloud/fed-sharing-error"
This reverts commit 027715f9acba4dc314a7e4c63ac41a58d4e33f22, reversing
changes made to e42ce62ce2855c95861eeae669508e5c20f99be4.
|
|
|
|
| |
some browsers (#25382) (#25397)
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Wait for socket to be open
* Fix call on null
* Allow DB access for MountProviderTest
Makes unit tests pass when using object store, since their FS access is
actually oc_filecache DB access. It is currently not possible to mock
or bypass the logic from "SharedMount::verifyMountPoint()" triggered by
this test.
|
|\
| |
| | |
prevent users from deleting their own session token
|
| | |
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
* String columns with a length higher then 4000 are converted into a CLOB columns automagically - we have to respect this when migrating
* Adding schema migration tests to prevent unnecessary and non-sense migration steps
Fix Oracle autoincrement and unsigned handling
* Fix sqlite integer type for autoincrement
* Use lower case table names - fixes pg
* Fix postgres with default -1 - this only affect pg 9.4 servers - 9.5 seems to work fine
|
|
|
|
|
|
|
|
|
|
|
|
| |
* fix login hooks
* adjust user session tests
* fix login return value of successful token logins
* trigger preLogin hook earlier; extract method 'loginWithPassword'
* call postLogin hook earlier; add PHPDoc
|
|\
| |
| | |
Add explicit delete permission to link shares
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Link shares always allowed deletion, however internally the permissions
were stored as 7 which lacked delete permissions. This created an
inconsistency in the Webdav permissions.
This fix makes sure we include delete permissions in the share
permissions, which now become 15.
In case a client is still passing 7 for legacy reasons, it gets
converted automatically to 15.
|
|/ |
|
|\
| |
| | |
Token login validation
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
* Initial web executor
* Fix PHPDoc
Fix broken integration test
OccControllerTests do not require database access - moch them all!
Kill unused sprintf
|
| | |
|
|/ |
|
| |
|
| |
|