summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Fix bigint handling on postgresJoas Schilling2017-09-291-0/+74
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Merge pull request #6544 from nextcloud/fix/duplicate-session-tokenMorris Jobke2017-09-251-0/+8
|\ | | | | Fix duplicate session token after remembered login
| * Fix duplicate session token after remembered loginChristoph Wurst2017-09-201-0/+8
| | | | | | | | | | | | | | | | | | On a remembered login session, we create a new session token in the database with the values of the old one. As we actually don't need the old session token anymore, we can delete it right away. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Handle SameSiteCookie check for index.php in AppFramework MiddlewareRoeland Jago Douma2017-09-242-2/+145
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #6605 from nextcloud/oc_ocs_response_is_depMorris Jobke2017-09-211-4/+4
|\ \ | | | | | | OC_OCS_Response is deprecated
| * | OC_OCS_Response is deprecatedRoeland Jago Douma2017-09-211-4/+4
| |/ | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* / Fix testsRoeland Jago Douma2017-09-211-0/+9
|/ | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #5462 from nextcloud/add-frameancestor-supportLukas Reschke2017-09-181-0/+41
|\ | | | | Add CSP frame-ancestors support
| * Add CSP frame-ancestors supportThomas Citharel2017-09-151-0/+41
| | | | | | | | | | | | Didn't set the @since annotation yet. Signed-off-by: Thomas Citharel <tcit@tcit.fr>
* | Add filter for `shareapi_allow_share_dialog_user_enumeration`Lukas Reschke2017-09-151-32/+125
| | | | | | | | | | | | | | | | This adjusts the contacts menu to also support searching by email address which is relevant in scenarios where no UID is known such as LDAP, etc. Furthermore, if `shareapi_allow_share_dialog_user_enumeration` is disabled only results are shown that match the full user ID or email address. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | Add testsTobia De Koninck2017-09-151-0/+129
| | | | | | | | Signed-off-by: Tobia De Koninck <tobia@ledfan.be>
* | Fix testsTobia De Koninck2017-09-151-1/+19
|/ | | | Signed-off-by: Tobia De Koninck <tobia@ledfan.be>
* Fix AppPassword 2FA authRoeland Jago Douma2017-09-121-1/+16
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #6414 from nextcloud/share-notification-wrong-languageMorris Jobke2017-09-081-0/+11
|\ | | | | Use the language of the recipient for the share notification
| * Use the language of the recipient for the share notificationJoas Schilling2017-09-081-0/+11
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Improve text: 'you have now' -> 'you now have'Leon Klingele2017-09-089-15/+15
|/ | | | Signed-off-by: Leon Klingele <leon@struktur.de>
* Merge pull request #6247 from nextcloud/oauth-state-undefinedMorris Jobke2017-09-061-0/+12
|\ | | | | Fix undefined index oauthState
| * Fix testsMorris Jobke2017-09-061-0/+12
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #6364 from nextcloud/fix_login_loopMorris Jobke2017-09-061-1/+5
|\ \ | |/ |/| Fix login with basic auth
| * Fix login with basic authRoeland Jago Douma2017-09-051-1/+5
| | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #6358 from nextcloud/fix-mixup-of-id-and-nameMorris Jobke2017-09-053-3/+7
|\ \ | |/ |/| Set the meta data before everything
| * Fix testsMorris Jobke2017-09-052-2/+6
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * Fix unit testsJoas Schilling2017-09-041-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Merge pull request #6254 from nextcloud/async-bus-splitMorris Jobke2017-09-043-28/+82
|\ \ | | | | | | Allow configuring different command bus backends
| * | Add proper EOLMorris Jobke2017-09-041-1/+1
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * | split async test bus for easier subclassingRobin Appelman2017-08-243-28/+82
| | |
* | | Fix broken testsMorris Jobke2017-09-041-1/+6
| |/ |/| | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #6296 from nextcloud/improve_2faRoeland Jago Douma2017-08-311-17/+189
|\ \ | | | | | | Improve 2FA
| * | Improve 2FARoeland Jago Douma2017-08-291-17/+189
| | | | | | | | | | | | | | | | | | | | | | | | * Store the auth state in the session so we don't have to query it every time. * Added some tests Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Return the user id in case of an errorJoas Schilling2017-08-291-1/+1
| | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #6177 from nextcloud/properly-add-slo-urlMorris Jobke2017-08-262-5/+15
|\ \ \ | |/ / |/| | Properly allow \OCP\Authentication\IApacheBackend to specify logout URL
| * | Inject \OCP\IURLGenerator to make tests workLukas Reschke2017-08-181-1/+11
| | | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
| * | Properly allow \OCP\Authentication\IApacheBackend to specify logout URLLukas Reschke2017-08-182-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any `\OCP\Authentication\IApacheBackend` previously had to implement `getLogoutAttribute` which returns a string. This string is directly injected into the logout `<a>` tag, so returning something like `href="foo"` would result in `<a href="foo">`. This is rather error prone and also in Nextcloud 12 broken as the logout entry has been moved with 054e161eb5f4a5c5c13ee322ae8e93ce66f01b13 inside the navigation manager where one cannot simply inject attributes. Thus this feature is broken in Nextcloud 12 which effectively leads to the bug described at nextcloud/user_saml#112, people cannot logout anymore when using SAML using SLO. Basically in case of SAML you have a SLO url which redirects you to the IdP and properly logs you out there as well. Instead of monkey patching the Navigation manager I decided to instead change `\OCP\Authentication\IApacheBackend` to use `\OCP\Authentication\IApacheBackend::getLogoutUrl` instead where it can return a string with the appropriate logout URL. Since this functionality is only prominently used in the SAML plugin. Any custom app would need a small change but I'm not aware of any and there's simply no way to fix this properly otherwise. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | Merge pull request #5870 from nextcloud/generate-system-wide-key-pairBjörn Schießle2017-08-241-15/+66
|\ \ \ | | | | | | | | extend the identity proof manager to allow system wide key pairs
| * | | add prefix to user and system keys to avoid name collisionsBjoern Schiessle2017-08-101-5/+5
| | | | | | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
| * | | extend the identity proof manager to allow system wide key pairsBjoern Schiessle2017-08-101-11/+62
| | | | | | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | | Merge pull request #6164 from nextcloud/dont-show-error-message-when-sql-failedRoeland Jago Douma2017-08-231-1/+20
|\ \ \ \ | |_|_|/ |/| | | Prevent SQL error message in case of error
| * | | Prevent SQL error message in case of errorLukas Reschke2017-08-171-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `\OC\User\Database::createUser` can throw a PHP exception in case the UID is longer than permitted in the database. This is against it's PHPDocs and we should cast this to `false`, so that the regular error handling triggers in. The easiest way to reproduce is on MySQL: 1. Create user `aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa` in admin panel 2. Create user `aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa` in admin panel again 3. See SQL exception as error message Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | No password reset for disabled usersJoas Schilling2017-08-181-0/+29
| |_|/ |/| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #6161 from nextcloud/drone-s3Morris Jobke2017-08-172-355/+1
|\ \ \ | | | | | | | | Enable S3 tests on drone again
| * | | Enable object storage tests againMorris Jobke2017-08-171-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * | | Remove windows config settings in testsMorris Jobke2017-08-172-354/+0
| |/ / | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* / / Always log cron executionMorris Jobke2017-08-171-8/+2
|/ / | | | | | | | | | | | | | | | | | | | | There was a setting to disable the last execution of cron. There is no known problem with this write access and it was also questioned when this feature was build in https://github.com/owncloud/core/pull/7689#issuecomment-38264707 Recently there was also a bug report about a non-visible last cron execution (#6088) - let's better remove this. Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Cleanup unused methodsMorris Jobke2017-08-153-15/+15
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #6123 from nextcloud/cleanup-shareItemMorris Jobke2017-08-153-22/+22
|\ \ | | | | | | Cleanup usage of shareItem in internal code base
| * | Cleanup usage of shareItem in internal code baseMorris Jobke2017-08-153-22/+22
| | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | Merge pull request #6121 from nextcloud/cleanup-registerBackendMorris Jobke2017-08-153-4/+4
|\ \ \ | | | | | | | | Use proper methods for registerBackend
| * | | Use proper methods for registerBackendMorris Jobke2017-08-153-4/+4
| |/ / | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* / / Use proper methods for checkPasswordProtectedShareMorris Jobke2017-08-151-1/+1
|/ / | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Cleanup unused methodsMorris Jobke2017-08-151-880/+0
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>