summaryrefslogtreecommitdiffstats
path: root/build
Commit message (Collapse)AuthorAgeFilesLines
* Fix transfering ownership of a share to user with same id as receiverDaniel Calviño Sánchez2018-08-071-0/+13
| | | | | | | | | When the ownership of a user share is transfered to the receiver the share is removed, as the receiver now owns the original file. However, due to a missing condition, any share with a group, link or remote with the same id as the user was removed, not only the user shares. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix the behat version to 3.3.*Joas Schilling2018-07-111-1/+1
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Check all paths for own files and sharesDaniel Calviño Sánchez2018-06-291-1/+7
| | | | | | | | Add "PARENT (2)" and its subdirectories to the paths to be checked, as before only the own "PARENT" folder was being checked, but not the shared one. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Fix filenames in sharing integration testsDaniel Calviño Sánchez2018-06-291-3/+3
| | | | | | | | | | | | | When a file is shared and the receiver of the share already has a file with the same name that file is left untouched, and "(2)" is appended to the name of the shared file. As "textfile0.txt" is included in the user folder skeleton all the users in the integration test have that file, so when it is shared the receiver sees the share as "/textfile0 (2).txt", and her own file as "/textfile0.txt". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Limit Sinon version to 5.0.7 at mostDaniel Calviño Sánchez2018-05-281-1/+1
| | | | | | | | | | | | | | | | | | | | | When using fake servers with Sinon.JS, the JavaScript test framework, the XHR objects are also fake. In Sinon 5.0.8 the "setRequestHeader" of XMLHttpRequest was modified to normalize the header values (as requested by the spec), but since then only string values are accepted; null or integer values can no longer be passed to "setRequestHeader", as it expects the "replace" function to be available in the object. However, in the tests null and integer values are passed to "setRequestHeader", which causes them to fail. Both Firefox and Chromium accept passing non-string values to their "setRequestHeader" implementation, and it is done, for example, in davclient.js; it is not clear yet whether Sinon got too restrictive or the code calling "setRequestHeader" was too loose. Given that davclient.js is an external dependency, as a temporary measure Sinon version is forced to be 5.0.7 at most until either Sinon or davclient.js are updated. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* backport of #8355 to stable13Arthur Schiwon2018-03-022-9/+9
| | | | | | | | | | | | | | | | | | test creating comments with numeric user ids Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> fix creating comments when file is accessible to users with numeric ids Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> tests for systemtags related to numeric user ids Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de> fix systemtags event with numeric user ids Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Remove old perl script to update l10n filesMorris Jobke2018-02-082-61/+1
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix CI after Code of Conduct mergeRoeland Jago Douma2018-01-101-0/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Merge pull request #7533 from ↵Morris Jobke2018-01-032-0/+37
|\ | | | | | | | | nextcloud/oc-28545-handle-oc-total-length-in-new-chunking [oc] Handle OC-Total-Length in new chunking
| * Add integration tests for length header on new chunkingThomas Müller2017-12-152-0/+37
| | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Add unit tests for the navigation bar slide gestureDaniel Calviño Sánchez2017-12-141-0/+1
|/ | | | | | | | The slide gesture is enabled or disabled depending on the width of the browser window. In order to easily control that width the karma-viewport plugin is now used in the unit tests. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* Some tests for the remote cloud apiRobin Appelman2017-12-084-1/+201
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #6437 from nextcloud/support-mail-send-in-shareMorris Jobke2017-11-271-1/+1
|\ | | | | Add support for `\OCP\Share\IShare::getMailSend` back
| * Federation actually sends an email via Share 2.0Morris Jobke2017-11-271-1/+1
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Harden phan checks to catch more errorsMorris Jobke2017-11-271-5/+5
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Merge pull request #7047 from ↵Björn Schießle2017-11-201-6/+6
|\ | | | | | | | | nextcloud/add-support-for-files-with-no-permissions Add support for files with no permissions
| * Return whether the file is readable or not in the DAV permissionsDaniel Calviño Sánchez2017-11-021-6/+6
| | | | | | | | | | | | | | | | | | | | Until now it was safe to assume that every file was readable by its owner, so there was no need to return whether the file was readable or not. However, with the introduction of end to end encryption that is no longer the case, and it is now necessary to explicitly provide that information. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | Remove brackets for invalid mail addresses when updating the license headerJulius Härtl2017-11-071-0/+9
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Change @georgehrke's emailMorris Jobke2017-11-061-0/+1
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Update license headersMorris Jobke2017-11-0621-17/+58
|/ | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Update phan configRoeland Jago Douma2017-10-241-0/+18
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* DAV AutoloaderRoeland Jago Douma2017-10-241-0/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Comments AutoloaderRoeland Jago Douma2017-10-241-0/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Admin Audit autoloaderRoeland Jago Douma2017-10-241-0/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Ignore composer stuff in phanRoeland Jago Douma2017-10-171-0/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Update autoload checkerRoeland Jago Douma2017-10-171-5/+14
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Phan is moved to a new repoRoeland Jago Douma2017-10-161-1/+1
| | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* show diff in autoload checkerRobin Appelman2017-09-181-0/+1
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* No more root fileJoas Schilling2017-07-251-1/+0
| | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* Move image-optimization.sh to build dirRoeland Jago Douma2017-07-241-0/+20
| | | | | | * We should not ship this! Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* Add Phan plugin to check for SQL injectionsLukas Reschke2017-07-204-0/+255
| | | | | | | | | | This adds a phan plugin which checks for SQL injections on code using our QueryBuilder, while it isn't perfect it should already catch most potential issues. As always, static analysis will sometimes have false positives and this is also here the case. So in some cases the analyzer just doesn't know if something is potential user input or not, thus I had to add some `@suppress SqlInjectionChecker` in front of those potential injections. The Phan plugin hasn't the most awesome code but it works and I also added a file with test cases. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add workaround for https://github.com/etsy/phan/issues/1033Lukas Reschke2017-07-191-1/+0
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Run phan over code baseLukas Reschke2017-07-197-0/+2346
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add test to check if new files are added to the root of the repositoryMorris Jobke2017-07-121-0/+109
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Prevent sending second WWW-Authenticate headerLukas Reschke2017-06-131-2/+2
| | | | | | | | Overrides \Sabre\DAV\Auth\Backend\AbstractBearer::challenge to prevent sending a second WWW-Authenticate header which is standard-compliant but most DAV clients simply fail hard. Fixes https://github.com/nextcloud/server/issues/5088 Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Bearer comes first on the new endpointLukas Reschke2017-05-181-1/+1
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Adjust realm from SabreDAV to NextcloudLukas Reschke2017-05-181-2/+2
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Make legacy DAV backend use the BearerAuth backend as wellLukas Reschke2017-05-181-1/+5
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add additional test for accessing DAV using Bearer AuthLukas Reschke2017-05-181-0/+4
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Use a standardized Bearer nowLukas Reschke2017-05-181-2/+2
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Add app to autoenabled provisioning API scenarioLukas Reschke2017-05-181-0/+1
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Extract app name and navigation entries for l10nMorris Jobke2017-05-081-0/+60
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Split up sharing-v1-part2.feature to avoid timeoutsMorris Jobke2017-05-022-388/+396
| | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* Fix forbidden backslash DAV integration testsVincent Petry2017-04-261-12/+12
|
* Added test cases from core 16825Sergio Bertolin2017-04-261-0/+41
|
* Simplified new endpoint move testVincent Petry2017-04-261-9/+4
|
* Added case when final chunk move must not change file idVincent Petry2017-04-261-0/+13
|
* Added test about checking file id after a moveSergio Bertolin2017-04-262-2/+45
|
* Merge pull request #4494 from nextcloud/fix-jsunitMorris Jobke2017-04-251-1/+1
|\ | | | | Fix JS unit failure because of Jasmine 2.6 and the phantomjs launcher
| * Fix JS unit failure because of Jasmine 2.6 and the phantomjs launcherMorris Jobke2017-04-251-1/+1
| | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>