summaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
...
* 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
| * Move DateTime::RFC2822 to DateTimeInterface::2822Christoph Wurst2021-06-232-2/+2
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
| * Move DateTime::ATOM to DateTimeInterface::ATOMChristoph Wurst2021-06-232-2/+2
| | | | | | | | Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | Merge pull request #26344 from J0WI/fs-early-rootLukas Reschke2021-06-231-8/+8
|\ \ | | | | | | Return early if path is root
| * | Return early if path is rootJ0WI2021-04-051-8/+8
| | | | | | | | | | | | Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* | | Merge pull request #26346 from J0WI/clean-auth-regexblizzz2021-06-231-1/+1
|\ \ \ | |_|/ |/| | Cleaner removePassword regex
| * | Cleaner removePassword regexJ0WI2021-03-271-1/+1
| | | | | | | | | | | | Signed-off-by: J0WI <J0WI@users.noreply.github.com>
* | | mail-template - don't show hyphen if slogan is emptyszaimen2021-06-221-1/+5
| | | | | | | | | | | | Signed-off-by: szaimen <szaimen@e.mail.de>
* | | Fix typing error in AppFetcher.phpLouis Chemineau2021-06-211-1/+3
| | | | | | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | Merge pull request #27540 from nextcloud/getstoragerootid-failed-storageJulius Härtl2021-06-181-1/+7
|\ \ \ | | | | | | | | handle case where storage can't be created in getStorageRootId
| * | | handle case where storage can't be created in getStorageRootIdRobin Appelman2021-06-171-1/+7
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Merge pull request #27532 from ↵Morris Jobke2021-06-182-0/+63
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/fix/noid/properly-cleanup-webauthn-user-deletion Properly cleanup entries of WebAuthn on user deletion
| * | | | Properly cleanup entries of WebAuthn on user deletionMorris Jobke2021-06-182-0/+63
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | Merge pull request #24318 from ↵Christoph Wurst2021-06-172-28/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | nextcloud/techdebt/noid/remove-oc_user-getDisplayName Use proper methods for display name retrieval
| * | | | | Use proper methods for display name retrievalMorris Jobke2021-06-172-28/+12
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | | Merge pull request #26939 from nextcloud/enh/app-icon-notification-bubbleblizzz2021-06-171-1/+11
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Let apps toggle an unread counter on app icons
| * | | | | Let apps toggle an unread counter on app iconsJulius Härtl2021-06-161-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-authored-by: Louis Chemineau <louis@chmn.me> Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | | | Merge pull request #22628 from Sp1l/simplify-htaccess-rewritesLukas Reschke2021-06-161-13/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | Coalesce RewriteCond lines in .htaccess
| * | | | | | Coalesce RewriteCond lines in .htaccessBernard Spil2021-06-021-13/+6
| | | | | | | | | | | | | | | | | | | | | And escape some '.' where not yet escaped
* | | | | | | Merge pull request #27098 from nextcloud/fix/noid/missing-key-in-addInfoJohn Molakvoæ2021-06-161-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | force 'name' key in array
| * | | | | | force 'name' key in arrayMaxence Lange2021-05-251-1/+1
| | | | | | | | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | | | | Merge pull request #26319 from nextcloud/files_external_app.phpMorris Jobke2021-06-162-15/+24
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | Move files_external to IBootstrap and remove some deprecated stuff in the process
| * | | | | | remove depricated methods from MountConfigRobin Appelman2021-06-162-15/+24
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* / | | | | Fix default product name for old themesMorris Jobke2021-06-161-0/+9
|/ / / / / | | | | | | | | | | | | | | | Signed-off-by: Morris Jobke <hey@morrisjobke.de>
* | | | | Use product name in places where it is appropriate rather than the instance nameJulius Härtl2021-06-162-2/+3
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Julius Härtl <jus@bitgrid.net>
* | | | | Merge pull request #27514 from nextcloud/fix/share_activityJulius Härtl2021-06-161-0/+5
|\ \ \ \ \ | | | | | | | | | | | | Fix share activity dispatch