summaryrefslogtreecommitdiffstats
path: root/tests
Commit message (Collapse)AuthorAgeFilesLines
* Clear appstore cache on version upgradeRoeland Jago Douma2016-12-153-7/+131
| | | | | | | | | * Add version to cached json * Compare version * Updated calls * Updated tests Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #2665 from nextcloud/remove-db-class-and-interfaceRoeland Jago Douma2016-12-141-2/+0
|\ | | | | Remove IDb interface which was deprecated for 3 years already
| * Remove IDb interface which was deprecated for 3 years alreadyJoas Schilling2016-12-141-2/+0
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Use the mocked config versionJoas Schilling2016-12-141-5/+6
|/ | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Add a unit testJoas Schilling2016-12-131-0/+132
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Don't use cached informations for app versionLukas Reschke2016-12-091-0/+4
| | | | | | | | | | | | | When installing an app from the appstore the `\OC_App::getAppVersion` code is triggered twice: - First when the downloader tries to compare the current version to the new version on the appstore to check if there is a newer version. This protects against downgrade attacks and is implemented in `\OC\Installer::downloadApp`. - Second, when the app is actually installed the current version is written to the database. (`\OC\Installer::installApp`) This fails however when the version is actually cached. Because in step 1 the cached version will be set to "0" and then be reused in the second step. While this is probably not the cleanest version I assume this is an approach that is least invasive. Feedback and suggestions welcome :) Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Save the timezone on login againJoas Schilling2016-12-081-1/+14
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Document updater channel & check for correct PHP version in updaterMorris Jobke2016-12-061-1/+1
| | | | | | * see https://github.com/nextcloud/updater/issues/53 Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #2505 from nextcloud/sudo-mode-provisioning-apiMorris Jobke2016-12-051-0/+101
|\ | | | | Require sudo mode on the provisioning API
| * Set last-login-check on basic authRoeland Jago Douma2016-12-051-0/+101
| | | | | | | | | | | | | | | | | | Else the last-login-check fails hard because the session value is not set and thus defaults to 0. * Started with tests Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | Merge pull request #2484 from nextcloud/fix-wrong-update-of-email-addressChristoph Wurst2016-12-051-7/+26
|\ \ | | | | | | make sure that we only update the email address if it really changed
| * | make sure that we only update the email address if it really changedBjoern Schiessle2016-12-021-7/+26
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | Merge pull request #2411 from nextcloud/fix-encryption-home-storageMorris Jobke2016-12-051-0/+94
|\ \ \ | |_|/ |/| | check if the file should really be encrypted before we update the file cache
| * | check if the file should really be encrypted before we update the file cacheBjoern Schiessle2016-11-291-0/+94
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | update testRobin Appelman2016-12-021-1/+1
| |/ |/| | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Add testsLukas Reschke2016-12-011-0/+71
| | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | LanguageMorris Jobke2016-11-301-68/+0
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #2152 from nextcloud/preview_cleanupjobMorris Jobke2016-11-302-0/+371
|\ \ | | | | | | Adds background job to cleanup all previews.
| * | Add test if repair step is already doneRoeland Jago Douma2016-11-191-1/+43
| | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
| * | Adds background job to cleanup all previews.Roeland Jago Douma2016-11-192-0/+329
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * A repair step that inserts a background job for each user * Each background job will delete for 15 seconds if it takes longer we reschedule. This is done so instances that don't use the system cron won't time out. * Added tests Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #2066 from nextcloud/fix-redirect-double-encodingMorris Jobke2016-11-291-6/+6
|\ \ \ | |_|/ |/| | do not double encode the redirect url
| * | do not double encode the redirect urlChristoph Wurst2016-11-091-6/+6
| | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | Merge pull request #2351 from nextcloud/remember-session-defaultLukas Reschke2016-11-281-3/+43
|\ \ \ | | | | | | | | do not remember session tokens by default
| * | | fix parameter orderChristoph Wurst2016-11-281-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
| * | | do not remember session tokens by defaultChristoph Wurst2016-11-271-3/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have to respect the value of the remember-me checkbox. Due to an error in the source code the default value for the session token was to remember it. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | Merge pull request #2353 from nextcloud/renew-session-token-rememberLukas Reschke2016-11-281-1/+11
|\ \ \ \ | | | | | | | | | | copy remember-me value when renewing a session token
| * | | | copy remember-me value when renewing a session tokenChristoph Wurst2016-11-271-1/+11
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On renew, a session token is duplicated. For some reason we did not copy over the remember-me attribute value. Hence, the new token was deleted too early in the background job and remember-me did not work properly. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | Merge pull request #2327 from nextcloud/exclude-pre-releasesChristoph Wurst2016-11-281-18/+38
|\ \ \ \ | |/ / / |/| | | Exclude pre-release versions as per SemVer
| * | | Exclude pre-release versions as per SemVerLukas Reschke2016-11-251-18/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As SemVer can be used apps could define a release like "10.0.0-alpha". This is something that we don't support at the moment in the server and we should filter all prereleases. Ref https://github.com/nextcloud/server/pull/2307#issuecomment-262911588 Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | Merge pull request #2276 from nextcloud/update-email-addressMorris Jobke2016-11-253-35/+484
|\ \ \ \ | |/ / / |/| | | Update email address
| * | | bring back setEmailAddress for the user managementBjoern Schiessle2016-11-252-9/+218
| | | | | | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
| * | | update accounts table if email address or display name changes from outsideBjoern Schiessle2016-11-251-15/+24
| | | | | | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
| * | | update email address correctlyBjoern Schiessle2016-11-231-20/+251
| | | | | | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | | Add Identityproof testsRoeland Jago Douma2016-11-243-1/+250
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add tests for Key * Add tests for Manager * Add tests for Signer * Removed URLGenerator from Signer Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Loop for newest version in appstore responseLukas Reschke2016-11-242-4/+1861
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The current implementation when fetching apps from the appstore is to assume that the first element is the newest version, this is now always applicable and leads to the fact that for some apps (e.g. nextant) the newest version is not delivered. This can be easily tested by comparing the version of the downloaded Nextant version. This change will loop over all releases delivered by the appstore and chooses the newest compatible one. While not the cleanest solution, it does its job. Most of the code are actually unit tests. Whereas I have copied the whole original response from the appstore and also have performed the transformation. So that's why the diff looks so huge. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | | Harden cookies more appropriateLukas Reschke2016-11-231-0/+70
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the __Host- prefix to the same-site cookies. This is a small but yet nice security hardening. See https://googlechrome.github.io/samples/cookie-prefixes/ for the implications. Fixes https://github.com/nextcloud/server/issues/1412 Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | Merge pull request #1972 from nextcloud/invalid-files-from-scannerRoeland Jago Douma2016-11-222-1/+27
|\ \ \ | | | | | | | | Make sure we don't scan files that can not be accessed
| * | | Move the validation into one place onlyJoas Schilling2016-11-211-1/+1
| | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
| * | | Make sure we don't scan files that can not be accessedJoas Schilling2016-11-211-0/+26
| | |/ | |/| | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | Merge pull request #2214 from nextcloud/remove-loggingRobin Appelman2016-11-213-181/+0
|\ \ \ | | | | | | | | remove old logging section
| * | | update testsRobin Appelman2016-11-213-181/+0
| |/ / | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | Adjust unit testsLukas Reschke2016-11-211-2/+6
| | | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | Fix changing display names for subadminsLukas Reschke2016-11-211-7/+42
| | | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | Add a signer class for signingLukas Reschke2016-11-212-0/+167
| | | | | | | | | | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* | | fix unit testsBjoern Schiessle2016-11-211-74/+7
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | push public user data to the lookup serverBjoern Schiessle2016-11-211-0/+4
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | remove old test classBjoern Schiessle2016-11-211-173/+0
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | fix unit testsBjoern Schiessle2016-11-211-0/+202
| | | | | | | | | | | | Signed-off-by: Bjoern Schiessle <bjoern@schiessle.org>
* | | allow to change display names in the user settings againBjörn Schießle2016-11-211-0/+1
| | | | | | | | | | | | | | | | | | keep display name and email address in sync with the accounts table Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | introduce accounts table and keep it up-to-date with the data added to the ↵Björn Schießle2016-11-211-0/+173
|/ / | | | | | | | | | | personal settings Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>