Skip to content

fix(dav): Fix avatar size in system address book - #38481

Merged
nickvergessen merged 1 commit into
masterfrom
bugfix/noid/avatar-size-in-sab
May 26, 2023
Merged

fix(dav): Fix avatar size in system address book#38481
nickvergessen merged 1 commit into
masterfrom
bugfix/noid/avatar-size-in-sab

Conversation

@nickvergessen

@nickvergessennickvergessen commented May 26, 2023

Copy link
Copy Markdown
Member

Summary

Before our server went away on a 14MB avatar:

In Connection.php line 123:
SQLSTATE[HY000]: General error: 2006 MySQL server has gone away 

Checklist

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessennickvergessen added the 3. to review Waiting for reviews label May 26, 2023
@nickvergessennickvergessen added this to the Nextcloud 28 milestone May 26, 2023
@nickvergessennickvergessen self-assigned this May 26, 2023
@ChristophWurst

Copy link
Copy Markdown
Member

/backport to stable27

@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels May 26, 2023
@nickvergessen
nickvergessen merged commit 37f387b into masterMay 26, 2023
@nickvergessen
nickvergessen deleted the bugfix/noid/avatar-size-in-sab branch May 26, 2023 09:04
@backportbot-nextcloud

Copy link
Copy Markdown

The backport to stable27 failed. Please do this backport manually.

# Switch to the target branch and update it
git checkout stable27
git pull origin/stable27
# Create the new backport branch
git checkout -b fix/foo-stable27
# Cherry pick the change from the commit sha1 of the change against the default branch# This might cause conflicts. Resolve them.
git cherry-pick abc123
# Push the cherry pick commit to the remote repository and open a pull request
git push origin fix/foo-stable27

More info at https://docs.nextcloud.com/server/latest/developer_manual/getting_started/development_process.html#manual-backport

@nickvergessen

Copy link
Copy Markdown
MemberAuthor

/backport to stable27

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishfeature: carddavRelated to CardDAV internalsfeature: dav

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@nickvergessen@ChristophWurst@GretaD