aboutsummaryrefslogtreecommitdiffstats
path: root/lib
Commit message (Collapse)AuthorAgeFilesLines
* do not write htaccess file if disk space is too lowSimon L2023-11-171-0/+8
| | | | Signed-off-by: Simon L <szaimen@e.mail.de>
* Fix(l10n): Update translations from TransifexNextcloud bot2023-11-172-4/+4
| | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* Merge pull request #41541 from nextcloud/fix/noid/serve-flac-filesMarcel Müller2023-11-161-1/+1
|\ | | | | Serve .flac audio files
| * Serve .flac audio filesMarcel Müller2023-11-161-1/+1
| | | | | | | | Signed-off-by: Marcel Müller <marcel-mueller@gmx.de>
* | Merge pull request #41395 from nextcloud/hello-emfChristoph Wurst2023-11-166-45/+121
|\ \ | | | | | | feat: add preview provider for emf files based on office
| * | fix: use png as preview right awayDaniel Kesselberg2023-11-132-37/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The initial office preview implementation converted an office document with LibreOffice to PDF, used ImageMagick to extract the first page as JPEG, and passed it OC_Image. https://github.com/nextcloud/server/pull/10198 changed the implementation to use PNG rather than PDF. OC_Image can use a PNG as a preview right away, so the ImageMagick step is unnecessary. The registration code was updated to not ask ImageMagick if PDF is supported, as PDFs are no longer used to create office document previews. Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
| * | feat: allow multiple libreoffice invocationsDaniel Kesselberg2023-11-131-11/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | LibreOffice only allows one invocation per user profile.[^1] The office provider set the user profile to /tmp/owncloud-instanceid and therefore only one invocation per instance is allowed. This was introduced a while ago, yet it's unclear if this was intentionally or just a side effect.[^2] The limitation on one invocation leads to the situation that the preview generation only works for a couple of files if you upload a whole folder of emf or word files. This commit removes the limitation by using a new user profile for each preview. That's done by using instance id plus file id as postfix for getTemporaryFolder. This has some drawbacks: - Overload protection: If you upload 100 emf files, you may end up with 100 LibreOffice invocations. Though, you can use preview_concurrency_new to limit the number of previews that can be generated concurrently when php-sysvsem is available. - New profile: I assume it takes a few bits to generate a fresh LibreOffice user profile. It appears that there is no way to ask LibreOffice to not create a profile and just work with the defaults. The profile will be cleaned after use by our temp manager. - Remove the configuration option preview_office_cl_parameters: This is not strictly necessary yet, but if you set the configuration option, the generated path for the user profile is also missing. The configuration option is not well documented (e.g., it's unclear that the last option needs to be --outdir) and actually, there should be no reason to change it after all. [^1]: https://wiki.documentfoundation.org/UserProfile [^2]: https://github.com/owncloud/core/pull/9784 Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
| * | feat: add preview provider for emf files based on officeDaniel Kesselberg2023-11-135-0/+47
| | | | | | | | | | | | Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
* | | Merge pull request #41057 from nextcloud/registermounts-optBenjamin Gaussorgues2023-11-164-57/+69
|\ \ \ | | | | | | | | optimize UserMountCache::registerStorage
| * | | optimize UserMountCache::registerStorageRobin Appelman2023-11-164-57/+69
| | |/ | |/| | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl> Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | Merge pull request #41511 from nextcloud/enh/advanced_search_backendJoas Schilling2023-11-164-11/+27
|\ \ \ | |/ / |/| | Enhancements for advanced search API
| * | Introduce constants for builtin search filtersBenjamin Gaussorgues2023-11-152-9/+24
| | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| * | Undeprecate getTermBenjamin Gaussorgues2023-11-151-1/+0
| | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
| * | Change person type separatorBenjamin Gaussorgues2023-11-151-1/+3
| | | | | | | | | | | | | | | | | | `/` isn't allowed in UID, it will avoid conflicts Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | Merge pull request #41465 from nextcloud/artonge/migrate_metadataLouis2023-11-164-0/+49
|\ \ \ | | | | | | | | Migrate old metadata table to the new ones
| * | | Add migration from the old to new metadata tablesLouis Chemineau2023-11-164-0/+49
| | | | | | | | | | | | | | | | Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | | Reverse X-Forwarded-For list to read the correct proxy remote addressJoas Schilling2023-11-161-2/+8
| | | | | | | | | | | | | | | | Signed-off-by: Joas Schilling <coding@schilljs.com>
* | | | Merge pull request #41486 from nextcloud/fix/41470/invalid_user_groupBenjamin Gaussorgues2023-11-162-4/+6
|\ \ \ \ | | | | | | | | | | Fix invalid users/groups handling in advanced search
| * | | | Fix invalid users/groups handling in advanced searchBenjamin Gaussorgues2023-11-152-4/+6
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | | | Fix(l10n): Update translations from TransifexNextcloud bot2023-11-162-0/+22
| |/ / / |/| | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | Merge pull request #41501 from nextcloud/feat/dav/show-ooo-ui-by-defaultChristoph Wurst2023-11-152-0/+20
|\ \ \ \ | | | | | | | | | | feat(dav): Enable OOO UI and expose enabled via OCP
| * | | | feat(dav): Enable OOO UI and expose enabled via OCPChristoph Wurst2023-11-152-0/+20
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Makes the feature opt-out now that we have meaningful integrations based on OOO data. Allows instances still to turn the feature off. For apps like Mail that build on top of this feature we need to know if the instance has the feature turned on or off. This is exposed as OCP API, too. Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
* | | | Merge pull request #37959 from nextcloud/ob-procSimon L2023-11-151-2/+8
|\ \ \ \ | |/ / / |/| | |
| * | | Us strpos() onlyGit'Fellow2023-11-151-9/+3
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
| * | | Check for open_basedir before reading /procGit'Fellow2023-11-151-2/+14
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | | Merge pull request #41495 from ↵Christoph Wurst2023-11-151-0/+4
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/fix/ocp/register-availability-coordinator fix(ocp): register IAvailabilityCoordinator service alias
| * | | | fix(ocp): register IAvailabilityCoordinator service aliasRichard Steinmetz2023-11-151-0/+4
| | |/ / | |/| | | | | | | | | | Signed-off-by: Richard Steinmetz <richard@steinmetz.cloud>
* | | | Merge pull request #40935 from nextcloud/local-case-insensitiveBenjamin Gaussorgues2023-11-151-1/+26
|\ \ \ \ | |/ / / |/| | | add some support for rename on case insensitive local filesystems
| * | | add some support for rename on case insensitive local filesystemsRobin Appelman2023-10-201-1/+26
| | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | Merge pull request #41366 from ir0nhide/fix/multipart-copy-ssecJulius Härtl2023-11-151-1/+7
|\ \ \ \
| * | | | fix(s3): support SSE-C headers for the MultipartCopy callTobias Zimmerer2023-11-141-1/+7
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Tobias Zimmerer <3228193+ir0nhide@users.noreply.github.com>
* | | | | Merge pull request #38622 from nextcloud/ifElseReturnMatchGit'Fellow2023-11-151-15/+8
|\ \ \ \ \ | |_|_|/ / |/| | | | Replace if/else with return match
| * | | | Replace if/else with return matchGit'Fellow2023-11-011-15/+8
| | | | | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | | | fix(AppManager): Ensure that app manager can handle empty app keys on the ↵Ferdinand Thiessen2023-11-142-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | navigation entries Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | fix: Allow to set custom app order on navigation entries added by closures ↵Ferdinand Thiessen2023-11-142-7/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | to NavigationManager Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | fix(NavigationManager): Set default app in `add` function to allow entries ↵Ferdinand Thiessen2023-11-141-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | added as closures Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | fix(NavigationManager): Check for `appManager` before using itFerdinand Thiessen2023-11-141-4/+4
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | | fix(INavigationManager): Add psalm typing for `add` parameterFerdinand Thiessen2023-11-141-1/+7
| |/ / / |/| | | | | | | | | | | Signed-off-by: Ferdinand Thiessen <opensource@fthiessen.de>
* | | | Merge pull request #41432 from nextcloud/fix/white-user-icon-searchBenjamin Gaussorgues2023-11-141-0/+6
|\ \ \ \ | | | | | | | | | | Fix search providers icons
| * | | | Fix user search provider iconBenjamin Gaussorgues2023-11-141-0/+6
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | | | metadata must be set as editable for PROPPATCHMaxence Lange2023-11-147-14/+155
|/ / / / | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | fix psalmMaxence Lange2023-11-141-5/+1
| | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | ignore metadata if table is emptyMaxence Lange2023-11-146-10/+34
| | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | Merge pull request #41436 from ↵Arthur Schiwon2023-11-144-0/+38
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/feat/introduce-inAppSearch-method-for-advanced-search Add `inAppSearch` for advanced search providers
| * | | | Add new method `inAppSearch` for advanced search providersBenjamin Gaussorgues2023-11-144-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows to indicates if a search provider also support in app search Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
* | | | | Merge pull request #41446 from nextcloud/fix/noid/moving-metadataqueryArthur Schiwon2023-11-149-13/+14
|\ \ \ \ \ | | | | | | | | | | | | moving IMetadataQuery
| * | | | | moving metadataqueryMaxence Lange2023-11-139-13/+14
| |/ / / / | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* / / / / Fix(l10n): Update translations from TransifexNextcloud bot2023-11-142-4/+4
|/ / / / | | | | | | | | | | | | Signed-off-by: Nextcloud bot <bot@nextcloud.com>
* | | | fix(workflowengine): use andWhere() not second where()Arthur Schiwon2023-11-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | - an unconditional where() condition is added right before - fixes also wrongly named methods in the Exception message Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
* | | | Merge pull request #41406 from ↵Joas Schilling2023-11-131-1/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | nextcloud/bugfix/noid/log-exception-class-when-message-is-empty fix(log): Log exception class when message is empty