summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #16502 from nextcloud/bugfix/16474Morris Jobke2019-07-292-8/+20
|\ | | | | Check the if we can actually access the storage cache for recent files
| * Add additional check for read permissionsJulius Härtl2019-07-241-1/+2
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
| * Check the if we can actually access the storage cache for recent filesJulius Härtl2019-07-242-7/+18
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | Merge pull request #16591 from nextcloud/fix/noid/typo-in-commentMorris Jobke2019-07-291-1/+1
|\ \ | | | | | | Fix typo in comment
| * | Fix typo in commentMorris Jobke2019-07-291-1/+1
|/ / | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | Merge pull request #16582 from nextcloud/enh/split_up_security_middlewareRoeland Jago Douma2019-07-297-140/+238
|\ \ | | | | | | Split up security middleware
| * | Split up security middlewareRoeland Jago Douma2019-07-277-140/+238
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With upcoming work for the feature policy header. Splitting this in smaller classes that just do 1 thing makes sense. I rather have a few small classes that are tiny and do 1 thing right (and we all understand what is going on) than have big ones. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #16563 from nextcloud/enh/lostcontroller/better_exceptionsMorris Jobke2019-07-295-21/+53
|\ \ \ | | | | | | | | Use proper exception in lostController
| * | | Use proper exception in lostControllerRoeland Jago Douma2019-07-275-21/+53
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | There is no need to log the expcetion of most of the stuff here. We should properly log them but an exception is excessive. This moves it to a proper exception which we can catch and then log. The other exceptions will still be fully logged. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #16581 from nextcloud/dep/strict_csps_can_goRoeland Jago Douma2019-07-293-0/+3
|\ \ \ | | | | | | | | No need to have these classes we tighten the default CSP from time to time
| * | | No need to have these classes we tighten the default CSP from time toRoeland Jago Douma2019-07-273-0/+3
| |/ / | | | | | | | | | | | | | | | time Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | Merge pull request #16570 from nextcloud/enh/supress_touch_errorRoeland Jago Douma2019-07-291-2/+2
|\ \ \ | | | | | | | | Supress warnings touch can generate
| * | | Supress warnings touch can generateRoeland Jago Douma2019-07-261-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We already catch the result value. Having the warning being logged explicitly doesn't help and polutes the log. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #16571 from nextcloud/enh/update_preview_controllerMorris Jobke2019-07-291-6/+1
|\ \ \ \ | | | | | | | | | | Update PreviewController
| * | | | Update PreviewControllerRoeland Jago Douma2019-07-261-6/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The constructor is called with the userId. However if a user is not logged in this is null. Which means that we get an exception instead of this being handled gracefully in the middleware. There are cleaner solutions. But this is the solution that is the easiest to apply without lots of work and risk of breaking things (handling the logged in middleware before initializing the controller etc). Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | [tx-robot] updated from transifexNextcloud bot2019-07-298-2/+8
| | | | |
* | | | | Merge pull request #16573 from nextcloud/fix-default-timeouts-in-oc-notificationRoeland Jago Douma2019-07-288-16/+118
|\ \ \ \ \ | | | | | | | | | | | | Fix default timeouts in OC.Notification
| * | | | | Add unit tests for "OC.Notification.hide()"Daniel Calviño Sánchez2019-07-261-0/+56
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | | Fix default timeouts in OC.NotificationDaniel Calviño Sánchez2019-07-268-11/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When no timeout was given "show()" used the default timeout of "OCP.Toast", which is 7 seconds instead of indefinitely as stated in the documentation of "show()". "showHtml()" should also indefinitely show the notification if no timeout is given, but due to the strict comparison the notification was indefinitely shown only when a timeout of 0 was explicitly given. Now both methods show the notification indefinitely (or until it is explicitly hidden) when no timeout is given. The unit tests did not catch this error because "showHtml()" had no tests (as before the move to Toastify it was called from "show()" and thus implicitly tested), and because "show()" verified that "hide()" was not called after some time; "hide()" is no longer called from "show()" since "OCP.Toast" is used internally, so the test always passed even if the notification was indeed hidden. Now the test is based on whether the element is found or not, and explicit tests were added too for "showHtml()". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | | Fix mixed test for "show" and "showTemporary"Daniel Calviño Sánchez2019-07-261-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "showTemporary()" when a timeout was given was being tested along with the "show()" tests; now there are two separate tests when a timeout is given, one for "showTemporary()" and one for "show()". Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
| * | | | | Check number of elements instead of if the jQuery object is definedDaniel Calviño Sánchez2019-07-261-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tje jQuery object created through "$('#testArea .toastify')" will be always defined even if no elements were found, so the check does not really work; instead, it should be checked the number of elements found. Signed-off-by: Daniel Calviño Sánchez <danxuliu@gmail.com>
* | | | | | Merge pull request #16583 from nextcloud/dependabot/npm_and_yarn/webpack-4.38.0Roeland Jago Douma2019-07-282-4/+4
|\ \ \ \ \ \ | | | | | | | | | | | | | | Bump webpack from 4.36.1 to 4.38.0
| * | | | | | Bump webpack from 4.36.1 to 4.38.0dependabot-preview[bot]2019-07-282-4/+4
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [webpack](https://github.com/webpack/webpack) from 4.36.1 to 4.38.0. - [Release notes](https://github.com/webpack/webpack/releases) - [Commits](https://github.com/webpack/webpack/compare/v4.36.1...v4.38.0) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* / | | | | [tx-robot] updated from transifexNextcloud bot2019-07-2864-56/+38
|/ / / / /
* | | | | Merge pull request #16560 from nextcloud/bugfix/noid/fix_cutype_reportingRoeland Jago Douma2019-07-271-11/+16
|\ \ \ \ \ | | | | | | | | | | | | fix calendar-user-type reporting
| * | | | | fix CUType reportingGeorg Ehrke2019-07-261-11/+16
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Georg Ehrke <developer@georgehrke.com>
* | | | | | Merge pull request #16558 from nextcloud/enh/less_verbose_locked_loggingRoeland Jago Douma2019-07-271-3/+3
|\ \ \ \ \ \ | | | | | | | | | | | | | | Do not log all locked exceptions
| * | | | | | Do not log all locked exceptionsRoeland Jago Douma2019-07-261-3/+3
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This can happen for valid reasons (multiple users writing at the same time) with for example the text app. Apps should properly handle it. No reason to log it by default. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | | [tx-robot] updated from transifexNextcloud bot2019-07-27126-212/+218
| |_|/ / / |/| | | |
* | | | | Merge pull request #16556 from ↵Morris Jobke2019-07-261-0/+25
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/feature/16554_adminAuditShareByMail/wiswedel log email shares in admin_audit log
| * | | | | log email shares in admin_audit logSascha Wiswedel2019-07-261-0/+25
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Sascha Wiswedel <sascha.wiswedel@nextcloud.com>
* | | | | Merge pull request #16557 from nextcloud/enh/do_not_log_locked_filesMorris Jobke2019-07-261-1/+8
|\ \ \ \ \ | |_|_|/ / |/| | | | Do not log locked files
| * | | | Do not log locked filesRoeland Jago Douma2019-07-261-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the code doing its job. There is no need to spam the log file with this. Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | | Merge pull request #16555 from nextcloud/fix/16529/mask-keysMorris Jobke2019-07-262-4/+7
|\ \ \ \ \ | | | | | | | | | | | | use a pattern to identify sensitive config keys
| * | | | | treat sensitive config keys by patternArthur Schiwon2019-07-262-4/+7
| | |/ / / | |/| | | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | | Merge pull request #16551 from nextcloud/fix/12735/displayname-emailMorris Jobke2019-07-262-1/+227
|\ \ \ \ \ | | | | | | | | | | | | supresses disclosing the userid for LDAP users in the welcome mail
| * | | | | supresses disclosing the userid for LDAP users in the welcome mailArthur Schiwon2019-07-262-1/+227
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The userid is not relevant here, and by default cannot be used to login with. Typically, there is a common type of login names in organizations (LDAP username or email most often) that does not need to be disclosed. Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | | Merge pull request #16456 from nextcloud/dep/searchByTagRoeland Jago Douma2019-07-2610-306/+0
|\ \ \ \ \ | |_|/ / / |/| | | | Remove deprecated searchByTag
| * | | | Remove deprecated searchByTagRoeland Jago Douma2019-07-2610-306/+0
| |/ / / | | | | | | | | | | | | Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>
* | | | Merge pull request #16461 from nextcloud/fix/noid/pgsql-versionRoeland Jago Douma2019-07-261-2/+2
|\ \ \ \ | |/ / / |/| | | fixes the check for postgresql
| * | | fixes the check for postgresqlArthur Schiwon2019-07-181-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | Merge pull request #16542 from ↵Roeland Jago Douma2019-07-261-17/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/dependabot/npm_and_yarn/build/fstream-1.0.12 [Security] Bump fstream from 1.0.11 to 1.0.12 in /build
| * | | | [Security] Bump fstream from 1.0.11 to 1.0.12 in /builddependabot-preview[bot]2019-07-251-17/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [fstream](https://github.com/npm/fstream) from 1.0.11 to 1.0.12. **This update includes a security fix.** - [Release notes](https://github.com/npm/fstream/releases) - [Commits](https://github.com/npm/fstream/compare/v1.0.11...v1.0.12) Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
* | | | | [tx-robot] updated from transifexNextcloud bot2019-07-2612-54/+56
| | | | |
* | | | | Merge pull request #16532 from nextcloud/bugfix/14776/maxcontrast-fixMorris Jobke2019-07-251-3/+6
|\ \ \ \ \ | |/ / / / |/| | | | Fix max contrast retrieval to limit minimum color for relative time
| * | | | Trigger fallback code to get max contrast value and use integer thereMorris Jobke2019-07-251-2/+5
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
| * | | | Fix max contrast retrieval to limit minimum color for relative timeMorris Jobke2019-07-241-1/+1
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | Merge pull request #16503 from ↵Roeland Jago Douma2019-07-251-0/+5
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/5504/create_new_birthday_calendars_with_VEVENT_only allow to provide supported calendar component set internally as a string
| * | | | | allow to provide supported calendar component set internally as a stringGeorg Ehrke2019-07-221-0/+5
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Georg Ehrke <developer@georgehrke.com>
* | | | | [tx-robot] updated from transifexNextcloud bot2019-07-2514-12/+50
| | | | |
generated by cgit v1.2.3 (git 2.39.1) at 2025-01-24 01:45:11 +0000