summaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #28256 from nextcloud/enh/21045/stable22-quota-restrictionsPytal2021-08-231-0/+12
|\ | | | | [stable22] Add quota restrictions options
| * refs #21045 add app config to disable unlimited quota and to set max quotaChristopher Ng2021-08-201-0/+12
| | | | | | | | | | | | | | | | | | | | avoid unlimited quota as default_quota fallback value if unlimited quota is not allowed avoid getting/setting/displaying unlimited default quota if not allowed implement tests for unlimited quota restrictions Signed-off-by: Julien Veyssier <eneiluj@posteo.net> Signed-off-by: Christopher Ng <chrng8@gmail.com> Signed-off-by: nextcloud-command <nextcloud-command@users.noreply.github.com>
* | Properly log errors in Movie previews generationJohn Molakvoæ (skjnldsv)2021-08-192-2/+8
| | | | | | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* | Merge pull request #28499 from nextcloud/backport/28485/stable22John Molakvoæ2021-08-191-6/+14
|\ \
| * | Fix #20913: Check image resource before attempting to preserve alphaSimon Spannagel2021-08-181-6/+14
| | | | | | | | | | | | Signed-off-by: Simon Spannagel <simonspa@kth.se>
* | | Merge pull request #28454 from nextcloud/backport/28303/stable22John Molakvoæ2021-08-191-0/+7
|\ \ \ | |_|/ |/| |
| * | Use IURLGenerator function to get value of `\OC::$WEBROOT` globalJonas Meurer2021-08-161-0/+7
| | | | | | | | | | | | Signed-off-by: Jonas Meurer <jonas@freesources.org>
* | | Merge pull request #28494 from nextcloud/backport/28481/stable22Pytal2021-08-181-2/+3
|\ \ \ | |_|/ |/| | [stable22] Hash cache key
| * | Hash cache keyChristopher Ng2021-08-181-2/+3
| | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | Emit an error log when the app token login name does not matchChristoph Wurst2021-08-181-1/+6
|/ / | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* / Only trap E_ERROR in session handlingJulius Härtl2021-08-171-1/+3
|/ | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* fix Folder->getById() when a single storage is mounted multiple timesRobin Appelman2021-08-091-15/+29
| | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* dont apply jail search filter is on the rootRobin Appelman2021-07-291-11/+15
| | | | | | the extra '/' breaks things and the filter wouldn't do anything anyway except making the databases job harder Signed-off-by: Robin Appelman <robin@icewind.nl>
* Admin Audit - Sharing: createShare - report the full pathszaimen2021-07-261-0/+1
| | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* Allow empty Redis configVincent Petry2021-07-231-4/+4
| | | | | | | | When Redis config is empty, use defaults. However when a Redis cluster config is specified, the "seeds" attribute is mandatory. Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Properly support RedisClusterJohn Molakvoæ (skjnldsv)2021-07-221-16/+22
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Support redis user password auth and tls encryptionJohn Molakvoæ (skjnldsv)2021-07-221-22/+84
| | | | Signed-off-by: John Molakvoæ (skjnldsv) <skjnldsv@protonmail.com>
* Sanitize more functions from the encryption appLukas Reschke2021-07-211-1/+57
| | | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Disable HEIC image preview provider for performance concernstobiasKaminsky2021-07-211-2/+1
| | | | Signed-off-by: tobiasKaminsky <tobias@kaminsky.me>
* Mask password for Redis and RedisCluster on connection failureDaniel Kesselberg2021-07-191-0/+6
| | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* Directly return cloud id from userJulius Härtl2021-07-141-1/+1
| | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* Merge pull request #27964 from nextcloud/backport/27761/stable22Joas Schilling2021-07-141-1/+1
|\ | | | | [stable22] Also hide group from direct matches
| * Also hide group from direct matchesJoas Schilling2021-07-141-1/+1
| | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | Correctly skip suppressed errors in PHP 8.0Chih-Hsuan Yen2021-07-141-1/+1
|/ | | | | | | | | | | | | | | | | | | | | | | Applies the suggested transformation mentioned in https://www.php.net/manual/en/migration80.incompatible.php, > The @ operator will no longer silence fatal errors (E_ERROR, > E_CORE_ERROR, E_COMPILE_ERROR, E_USER_ERROR, E_RECOVERABLE_ERROR, > E_PARSE). Error handlers that expect error_reporting to be 0 when > @ is used, should be adjusted to use a mask check instead The new code still works on PHP 7, as error_reporting() already returns 0 when diagnostics are suppressed. This fixes https://github.com/nextcloud/server/issues/25807 in PHP 8.0. For PHP 7.x, https://github.com/nextcloud/server/pull/22243 suppresses the E_NOTICE message from the second session_start() call with the error suppression operator @, and thus those E_NOTICE messages are still logged in PHP 8.0. See also https://github.com/nextcloud/server/issues/25806 Signed-off-by: Chih-Hsuan Yen <yan12125@gmail.com>
* Check for !== false instead Lukas Reschke2021-07-121-1/+1
| | | Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Check if dns_get_record returns non-falseLukas Reschke2021-07-121-1/+1
| | | | | | | `dns_get_record` can return false which results in exceptions such as the ones shown in https://github.com/nextcloud/server/issues/27870. Signed-off-by: Lukas Reschke <lukas@statuscode.ch>
* Merge pull request #27825 from nextcloud/backport/27801/stable22MichaIng2021-07-091-13/+28
|\ | | | | [stable22] Ignore subdomain for soa queries
| * Ignore subdomain for soa queriesDaniel Kesselberg2021-07-061-13/+28
| | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | Merge pull request #27848 from sanpii/fix-dns-pinning-max-recursionkesselb2021-07-071-1/+1
|\ \ | | | | | | Fixes recursion count incrementation
| * | Fixes recursion count incrementationSanpi2021-07-071-1/+1
| |/ | | | | | | Signed-off-by: Sanpi <sanpi@homecomputing.fr>
* / Fix in locking cache checkMichaIng2021-07-061-1/+1
|/ | | | | The intention obviously was to check whether $lockingCacheClass is defined, and existing class, and available. It was however checked whether the $distributedCacheClass is an existing class, which would have caused an exception already in the previous distributed cache check. Signed-off-by: MichaIng <micha@dietpi.com>
* Fix DnsPinMiddleware resolve pinning bugAaron Ball2021-07-051-5/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | Libcurl expects the value of the CURLOPT_RESOLVE configurations to be an array of strings, those strings containing a comma delimited list of resolved IPs for each host:port combination. The original code here does create that array with the host:port:ip combination, but multiple ips for a single host:port result in additional array entries, rather than adding them to the end of the string with a comma. Per the libcurl docs, the `CURLOPT_RESOLVE` array entries should match the syntax `host:port:address[,address]`. This creates a function-scoped associative array which uses `host:port` as the key (which are supposed to be unique and this ensures that), and the value is an array containing IP strings (ipv4 or ipv6). Once the associative array is populated, it is then set to the CURLOPT_RESOLVE array, imploding the ip arrays using a comma delimiter so the array syntax matches the expected by libcurl. Note that this reorders the "foreach ip" and "foreach port" loops. Rather than looping over ips then ports, we now loop over ports then ips, since ports are part of the unique host:port map, and multiple ips can exist therein. Signed-off-by: Aaron Ball <nullspoon@oper.io>
* Merge pull request #27752 from nextcloud/backport/27586/stable22John Molakvoæ2021-07-051-0/+1
|\ | | | | [stable22] Reset checksum when writing files to object store
| * Reset checksum when writing files to object storeJulius Härtl2021-07-011-0/+1
| | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | dont include folder being search in in the resultsRobin Appelman2021-07-021-1/+7
|/ | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* Merge pull request #27719 from nextcloud/backport/27474/stable22John Molakvoæ2021-07-015-283/+337
|\
| * cleanupArthur Schiwon2021-06-301-7/+3
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * adjust internal data handling logic to fix store and loadArthur Schiwon2021-06-303-135/+155
| | | | | | | | | | | | - format as stored previously in oc_accounts table is kept Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * adjust email verification checkerArthur Schiwon2021-06-301-16/+18
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * fix code styleArthur Schiwon2021-06-302-2/+2
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * adjust verification state updater methodArthur Schiwon2021-06-301-70/+33
| | | | | | | | | | | | - also fixes scope of internal methods Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * make AccountManager actually write multi value propertiesArthur Schiwon2021-06-301-98/+145
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * accounts event handler to use eventdispatcher, DI and Accounts APIArthur Schiwon2021-06-301-39/+28
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * prov api to be able to edit multivalue propertiesArthur Schiwon2021-06-301-0/+13
| | | | | | | | | | | | | | - adding as usual - deleting and scope setting via additional endpoint Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
| * prov api reports additional emails on getUserArthur Schiwon2021-06-301-2/+26
| | | | | | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | ensure that factoryClass exisits before instantiationArthur Schiwon2021-06-301-1/+1
|/ | | | Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* Revert "First attempt to check against core routes before loading all app ↵Vincent Petry2021-06-291-14/+12
| | | | | | routes" Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* Harden bootstrap context registrations when apps are missingChristoph Wurst2021-06-251-11/+51
| | | | | | | | | | | It's not expected that an app would be unavailable when the app container is created but when services are registered, but Sentry tells me on Nextcloud 21 there is an edge case where this can happen. Therefore this patch hardens the code a bit to log a meaningful error message and skipping the next code instead of logging a php notice for the undefined index and an exception for calling a method on null. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Unshift crash reports when they are loaded, to break the recusionChristoph Wurst2021-06-251-3/+2
| | | | | | | | | | | | | | | If, for whatever reason, during the loading of a crash reporter a new log entry is generated, then the lazy loading mechanism will be invoked *again* while it's already executed. This doesn't result in an endless recursion, but means that the crash reporters will be built and registered many times. This then means any further log entry will be logged x times instead of once. Unshift makes sure to take the class off the registration list right away, so another invokation of the same method won't try to do the same job. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* Merge pull request #27635 from nextcloud/fix/datetime-constantsPytal2021-06-234-4/+4
|\ | | | | Fix usage of DateTime constants
generated by cgit v1.2.3 (git 2.39.1) at 2025-03-23 23:39:52 +0000