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.

Sign up for GitHub

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

Jump to bottom

[stable27] fix(twofactor): avoid DB error on Twofactor (en/dis)abled event #41212

Merged
merged 1 commit into from
Nov 15, 2023
Merged

[stable27] fix(twofactor): avoid DB error on Twofactor (en/dis)abled event #41212

merged 1 commit into from
Nov 15, 2023

Conversation

Altahrim
Copy link
Collaborator

@Altahrim Altahrim commented Oct 31, 2023 •
edited
Loading

Manual backport of #40472

@Altahrim Altahrim added the 3. to review Waiting for reviews label Oct 31, 2023
@Altahrim Altahrim requested a review from a team October 31, 2023 10:04
@Altahrim Altahrim self-assigned this Oct 31, 2023
@Altahrim Altahrim requested review from ArtificialOwl, icewind1991 and nfebe and removed request for a team October 31, 2023 10:04
@solracsf solracsf added this to the Nextcloud 27.1.4 milestone Oct 31, 2023
@Altahrim
fix(twofactor): avoid error in pgsql for duplicate entry
234e8d2 
Signed-off-by: Benjamin Gaussorgues <benjamin.gaussorgues@nextcloud.com>
@Altahrim Altahrim force-pushed the backport/fix/aborted_pgsql_transaction_on_error/stable27 branch from 6d4cd0c to 234e8d2 Compare October 31, 2023 13:44
@Altahrim Altahrim changed the title [stable27] fix(twofactor): avoid DB error on Twofactor (en/dis)abled event #40472 [stable27] fix(twofactor): avoid DB error on Twofactor (en/dis)abled event Oct 31, 2023
@Altahrim Altahrim enabled auto-merge October 31, 2023 14:50
@blizzz blizzz mentioned this pull request Nov 13, 2023
Merged
@Altahrim Altahrim merged commit 11764ff into stable27 Nov 15, 2023
@Altahrim Altahrim deleted the backport/fix/aborted_pgsql_transaction_on_error/stable27 branch November 15, 2023 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes

@AndyScherzinger AndyScherzinger AndyScherzinger approved these changes

@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl was automatically assigned from nextcloud/server-backend

@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 was automatically assigned from nextcloud/server-backend

@nfebe nfebe Awaiting requested review from nfebe nfebe was automatically assigned from nextcloud/server-backend

@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst

Assignees

@Altahrim Altahrim

Labels
3. to review Waiting for reviews
Projects
None yet
Milestone
Nextcloud 27.1.4
Development

Successfully merging this pull request may close these issues.
4 participants
@Altahrim @nickvergessen @AndyScherzinger @solracsf