Skip to content

fix(LockWrapper): gracefully handle null user in copyFromStorage()#1097

Merged
mejo- merged 2 commits into
mainfrom
fix/copy_from_storage_null_user
May 26, 2026
Merged

fix(LockWrapper): gracefully handle null user in copyFromStorage()#1097
mejo- merged 2 commits into
mainfrom
fix/copy_from_storage_null_user

Conversation

@mejo-
Copy link
Copy Markdown
Member

@mejo- mejo- commented May 26, 2026

$this->userSession->getUser() can be null, e.g. when called from occ command.

`$this->userSession->getUser()` can be null, e.g. when called from `occ`
command.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- requested a review from ArtificialOwl May 26, 2026 10:40
@mejo- mejo- self-assigned this May 26, 2026
@mejo- mejo- added bug Something isn't working regression labels May 26, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team May 26, 2026
@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented May 26, 2026

/backport to stable34

@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented May 26, 2026

/backport to stable33

@mejo-
Copy link
Copy Markdown
Member Author

mejo- commented May 26, 2026

/backport to stable32

@benjaminfrueh benjaminfrueh moved this from 🧭 Planning evaluation (don't pick) to 👀 In review in 📝 Productivity team May 26, 2026
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/copy_from_storage_null_user branch from 1300a37 to e3bb8ef Compare May 26, 2026 14:27
@mejo- mejo- merged commit 2e3d756 into main May 26, 2026
47 of 48 checks passed
@mejo- mejo- deleted the fix/copy_from_storage_null_user branch May 26, 2026 20:01
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team May 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working regression

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants