Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: don't return null for SharedStorage::getWrapperStorage with share recursion #44132

Merged
merged 2 commits into from
Mar 14, 2024

Conversation

icewind1991
Copy link
Member

While this might not fix errors from share recursion it should at least result in better errors.

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Mar 11, 2024
@icewind1991 icewind1991 added this to the Nextcloud 29 milestone Mar 11, 2024
@icewind1991 icewind1991 requested review from a team, ArtificialOwl, Altahrim and nfebe and removed request for a team March 11, 2024 16:08
@icewind1991 icewind1991 force-pushed the fix/share-init-recursion branch from 37023f1 to ba7b89e Compare March 11, 2024 16:41
@Altahrim Altahrim mentioned this pull request Mar 12, 2024
@susnux
Copy link
Contributor

susnux commented Mar 13, 2024

@icewind1991 conflicts

@Altahrim Altahrim mentioned this pull request Mar 14, 2024
…e recursion

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@come-nc come-nc force-pushed the fix/share-init-recursion branch from ba7b89e to f46209a Compare March 14, 2024 10:35
@come-nc
Copy link
Contributor

come-nc commented Mar 14, 2024

@icewind1991 conflicts

Fixed them

@susnux susnux added the bug label Mar 14, 2024
@susnux susnux merged commit 26c37fa into master Mar 14, 2024
160 checks passed
@susnux susnux deleted the fix/share-init-recursion branch March 14, 2024 11:39
@icewind1991
Copy link
Member Author

/backport to stable28

@icewind1991
Copy link
Member Author

/backport to stable27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants