aboutsummaryrefslogtreecommitdiffstats
path: root/lib/private
Commit message (Collapse)AuthorAgeFilesLines
* Merge pull request #34073 from nextcloud/login-email-filterVincent Petry2022-09-161-0/+3
|\ | | | | dont try email login if the provider username is not a valid email
| * dont try email login if the provider username is not a valid emailRobin Appelman2022-09-141-0/+3
| | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | Merge pull request #24596 from kofemann/dcache-wormVincent Petry2022-09-161-0/+14
|\ \ | | | | | | make NextCloud WORM file system friendly
| * | config: add a switch to control truncate before updateTigran Mkrtchyan2022-09-161-6/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | To avoid extra truncate on non WORM file systems, add a new config option `localstorage.unlink_on_truncate`, which defaults to false. The OC\Files\Storage\Local is update to respect that option. Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
| * | files: remove destination file before copying new content (WORM)Tigran Mkrtchyan2022-09-161-0/+2
| | | | | | | | | | | | Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
| * | files: make OC\Files\Storage\Local WORM friendlyTigran Mkrtchyan2022-08-251-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some filesystems run as a Write-Once-Read-Many storages. This makes them impossible to use with NexeCloud, as the file system layers uses `truncate` syscall (through file_put_contents function). As Nextcloud is never updates existing files, removing the old entry and creatint a new one on update will allow NextCoud to update on such file systems. Update Local#fopen and Local#file_put_contents to remote existing file before truncating. Signed-off-by: Tigran Mkrtchyan <tigran.mkrtchyan@desy.de>
* | | Merge pull request #34020 from nextcloud/logical-opVincent Petry2022-09-161-18/+12
|\ \ \ | | | | | | | | Switch logical operators (and|or)
| * | | Switch logical operators (and|or)Git'Fellow2022-09-121-18/+12
| | | | | | | | | | | | Signed-off-by: Git'Fellow <12234510+solracsf@users.noreply.github.com>
* | | | l10n: Correct error messagesValdnet2022-09-161-3/+3
| | | | | | | | | | | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* | | | l10n: Correct error messagesValdnet2022-09-161-4/+4
| | | | | | | | | | | | Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* | | | Merge pull request #34072 from nextcloud/enh/noid/default-enabled-2Vincent Petry2022-09-152-4/+21
|\ \ \ \ | | | | | | | | | | set defaultEnabled in shipped.json
| * | | | set defaultEnabled in shipped.jsonMaxence Lange2022-09-152-4/+21
| | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | | Merge pull request #33718 from nextcloud/seekable-http-fseek-endVincent Petry2022-09-151-21/+60
|\ \ \ \ \ | | | | | | | | | | | | fix using FSEEK_END with SeekableHttpStream to get file size
| * | | | | fix using FSEEK_END with SeekableHttpStream to get file sizeRobin Appelman2022-09-151-21/+60
| |/ / / / | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | Merge pull request #34009 from andrey-utkin/safe-config-updateCôme Chilliet2022-09-151-25/+29
|\ \ \ \ \ | | | | | | | | | | | | Make config file saving safe against write failures
| * | | | | Make config file saving safe against write failuresAndriy Utkin2022-09-091-25/+29
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In case of disk space depletion, writing a new version of config fails, leaving the config file empty. This patch improves safety of updates to config.php by saving the new version of the config into a randomly-named temporary file in the same directory, and then renaming it to config.php, which renaming is atomic in POSIX-compliant filesystems and shouldn't involve disk space allocation. If writing the new config version is impossible, the current config remains unchanged. This patch drops the use of file locking as unnecessary. File locking merely established order in which the concurrent independent processes attempted file writing. In the end, the process which was last to update the file "wins" - their changes persist and the changes of previous writers are dropped as there's no conflict detection or change merging mechanism anyway. With the current change, there is still some resulting ordering, and the last writer still wins in the same way. Readers don't need file locking as well, as opening config.php for reading always provides a certain consistent version of the file. Signed-off-by: Andriy Utkin <dev@autkin.net>
* | | | | Merge pull request #33793 from nextcloud/fix/noid/rtrim-cloud-idMaxence Lange2022-09-151-2/+2
|\ \ \ \ \ | |_|/ / / |/| | | | rtrim cloudId url earlier
| * | | | rtrim url earlierMaxence Lange2022-09-011-2/+2
| | | | | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | | Add accessibility entry in user menuVincent Petry2022-09-141-5/+16
| |_|_|/ |/| | | | | | | | | | | Signed-off-by: Vincent Petry <vincent@nextcloud.com>
* | | | Merge pull request #34019 from nextcloud/fix/quota-typeVincent Petry2022-09-122-4/+4
|\ \ \ \ | | | | | | | | | | Fix quota type to int
| * | | | Fix quota type to intCarl Schwan2022-09-122-4/+4
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change the quota to int from float, since the quota is a number of bits and a bits can not be splitted. Fix #34010 Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Merge pull request #33921 from nextcloud/fix/fix-config-file-emptiedCôme Chilliet2022-09-121-0/+4
|\ \ \ \ | |/ / / |/| | | Do not empty config.php file if reading failed for any reason
| * | | Do not empty config.php file if reading failed for any reasonCôme Chilliet2022-09-061-0/+4
| | | | | | | | | | | | | | | | Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
* | | | Update avatars on updateCarl Schwan2022-09-092-19/+13
| | | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Dark theme for guest avatarCarl Schwan2022-09-093-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | And better caching policy Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Avatar new styleCarl Schwan2022-09-092-30/+53
| | | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | Merge pull request #33788 from nextcloud/improve-recentCarl Schwan2022-09-081-27/+55
|\ \ \ \ | | | | | | | | | | Improve getting recent files performance
| * | | | Filter out old files when trying to get recent filesCarl Schwan2022-09-051-27/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only do so when asking for less than 100 files and having an offset equal to 0. Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | | Merge pull request #33906 from nextcloud/fix/fix-calendar-tests-getTimestampblizzz2022-09-081-1/+16
|\ \ \ \ \ | | | | | | | | | | | | Fix Calendar tests mocking a non-existant method.
| * | | | | Fix various errors in iMIP logicAnna Larch2022-09-081-1/+16
| | |/ / / | |/| | | | | | | | | | | | | Signed-off-by: Anna Larch <anna@nextcloud.com>
* | | | | Merge pull request #33600 from nextcloud/lazy-quotablizzz2022-09-072-24/+55
|\ \ \ \ \ | | | | | | | | | | | | get quota for user only when needed
| * | | | | get quota for user only when neededRobin Appelman2022-09-012-24/+55
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | always apply the wrapper but have the wrapper get the quota lazily only when an operation where quota is applied is performed Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | Merge pull request #33813 from nextcloud/fix/infinite_email_change_notificationLouis2022-09-061-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | Compare lowercase email when updating from ldap
| * | | | | | I dug into it again, and the issue is much simpler than I previously though.Louis Chemineau2022-09-061-1/+1
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - LDAP has an email address with capital letters - NC store this address in lower case - When the user logs in, we compare the [stored email with the new lower case email](https://github.com/nextcloud/server/blob/master/lib/private/AllConfig.php#L259-L261) before storing it. Here, both email will be the same, so we won't store the new email address with upper case letters. Which is what we want. - We then [compare emails as they are before triggering an event](https://github.com/nextcloud/server/blob/master/lib/private/User/User.php#L202-L204), they won't match, so the user will receive an email signaling an email change every time he logs in. The fix is to compare the old email with the new lower case email before sending the event. Signed-off-by: Louis Chemineau <louis@chmn.me>
* | | | | | Merge pull request #29881 from nextcloud/fix/setting/accessibility-titleCarl Schwan2022-09-061-0/+7
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Improve accessibility of the title of the settings
| * | | | | Merge branch 'master' into fix/setting/accessibility-titleCarl Schwan2022-09-02178-1496/+3293
| |\ \ \ \ \ | | | | | | | | | | | | | | Signed-off-by: Carl Schwan <carl@carlschwan.eu>
| * | | | | | Improve accessibility of the title of the settingsCarl Schwan2022-05-161-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Before every setting page had the same title and this is causing issues for screenreaders since they can't differenciate the title of page. Now a new variable is available for apps to declare the page subtitle. This new variable is implemented for the settings app and while at it I added a bit more type hinting to the stuff I touched :) Signed-off-by: Carl Schwan <carl@carlschwan.eu>
* | | | | | | Merge pull request #33863 from nextcloud/hash-wrapper-catch-errorsblizzz2022-09-021-2/+7
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | handle errors from hash_final
| * | | | | | | handle errors from hash_finalRobin Appelman2022-09-021-2/+7
| | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
* | | | | | | Merge pull request #33692 from nextcloud/update-dirblizzz2022-09-021-2/+2
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | Document and use the new updateDirectory if set
| * | | | | | | Introduce $updateDir to move updates outside $dataDirGit'Fellow2022-08-251-2/+2
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | To be merged after https://github.com/nextcloud/updater/pull/420 Signed-off-by: Git'Fellow <carlos@reendex.com>
* | | | | | | Merge pull request #33790 from nextcloud/Valdnet-patch-2Simon L2022-09-021-1/+1
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | l10n: Spelling unification
| * | | | | | l10n: Spelling unificationValdnet2022-09-021-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Spelling unification in Transifex. Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com> l10n: Change to administration Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com> l10n: Spelling unification Spelling unification in Transifex. Match the menu name to the header name of the settings page. Signed-off-by: Valdnet <47037905+Valdnet@users.noreply.github.com>
* | | | | | | Merge pull request #33625 from nextcloud/fix/33572/add-userVincent Petry2022-09-012-1/+6
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | Fix creation of new user and display the correct error message
| * | | | | | Fix creation of new user and display the correct error messageChristopher Ng2022-08-222-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Christopher Ng <chrng8@gmail.com>
* | | | | | | Merge pull request #33780 from nextcloud/enh/noid/cloud-id-cliblizzz2022-09-011-0/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | filter index.php from cloudId
| * | | | | | | filter index.php from cloudIdMaxence Lange2022-08-311-0/+3
| | |_|_|_|_|/ | |/| | | | | | | | | | | | Signed-off-by: Maxence Lange <maxence@artificial-owl.com>
* | | | | | | Merge pull request #33764 from nextcloud/cloudid-cacheVincent Petry2022-09-012-7/+80
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | cache cloud id data in CloudIdManager
| * | | | | | | adjusts tests for CloudIdManagerRobin Appelman2022-08-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>
| * | | | | | | cache cloud id data in CloudIdManagerRobin Appelman2022-08-312-7/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robin Appelman <robin@icewind.nl>