search

nextcloud / mail

💌 Mail app for Nextcloud
https://apps.nextcloud.com/apps/mail
GNU Affero General Public License v3.0
851 stars 263 forks source link

fix(contactsintegration): Limit number of search results #10321

Closed ChristophWurst closed 3 weeks ago

ChristophWurst commented 3 weeks ago

Searching with few characters in large address books or system address books can yield lots of results. This can exhaust the memory and a user likely will search more specifically anyway.

Fixes Allowed memory size of 1073741824 bytes exhausted (tried to allocate 4096 bytes) at /var/www/html/nextcloud/apps/dav/lib/CardDAV/CardDavBackend.php

Fixes https://github.com/nextcloud/mail/issues/9624

ChristophWurst commented 3 weeks ago

/backport to stable4.0

ChristophWurst commented 3 weeks ago

/backport to stable3.8

ChristophWurst commented 3 weeks ago

/backport to stable3.7

backportbot[bot] commented 3 weeks ago

The backport to stable3.8 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable3.8
git pull origin stable3.8

# Create the new backport branch
git checkout -b backport/10321/stable3.8

# Cherry pick the change from the commit sha1 of the change against the default branch
# This might cause conflicts, resolve them
git cherry-pick b02921e5

# Push the cherry pick commit to the remote repository and open a pull request
git push origin backport/10321/stable3.8

Error: Failed to clone repository: Failed to checkout branches: fatal: ambiguous argument 'origin/stable3.8': unknown revision or path not in the working tree. Use '--' to separate paths from revisions, like this: 'git [...] -- [...]'

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.