fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error by smarcet · Pull Request #533 · OpenStackweb/summit-api

fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error

smarcet

committed

fix(transactions): harden root/nested split against phantom writes and masked errors

smarcet

committed

fix(tx): remove DBAL savepoints, propagate native isRollbackOnly guard

smarcet

committed

test: nested-transaction rollback coverage for SummitOrderService, SummitService, SpeakerService, PresentationService, SummitPromoCodeService

smarcet

committed

test: HTTP-level exception-branch coverage for order/attendee ticket creation

smarcet

committed

docs(adr): nested transaction rollback safety decision record

smarcet

committed

test: nested-transaction rollback coverage for remaining ADR-003 gaps

smarcet

committed

docs(adr): update nested-tx rollback ADR with remaining test coverage

smarcet

committed

docs(adr): remove docs/plans references from ADR-003

smarcet

committed

chore: add nested-tx rollback test classes to push.yml CI matrix

smarcet

committed

docs(adr): clarify origin/main's actual pre-branch behavior in ADR-003

smarcet

committed

fix(transactions): never retry ambiguous commits, refuse closed-EM re-entry

smarcet

committed

fix(transactions): discard broken manager/connection when rollback fails

smarcet

committed

refactor(transactions): deduplicate failure-cleanup paths in DoctrineTransactionService

smarcet

committed

fix(transactions): surface ambiguous commit failures as AmbiguousCommitException

smarcet

committed

fix(registration): log-and-skip failing rows in CSV ticket data import

smarcet

committed

fix(transactions): never classify rollback-only commit failures as ambiguous

smarcet

committed

docs(adr): log AmbiguousCommitException consumer wiring as known gap

smarcet

committed

fix(registration): keep import file when a CSV row commit outcome is unknown

smarcet

committed

fix(summit): re-fetch summit per row in processEventData CSV import

smarcet

committed

test(imports): volume coverage for every CSV import service method

smarcet

committed

fix(registration): support guest buyers on the reserve saga

smarcet

committed

test(orders): revive reserve API tests and cover the order cancel endpoint

smarcet

committed

fix(auth): flush email invalidation before reassigning a colliding email

smarcet

committed

chore(ci): run MemberServiceTest in the integration matrix

smarcet

committed

chore(ci): wire Stripe test credentials from repository secrets

smarcet

committed

fix(summit): keep import files when a row commit outcome is unknown

smarcet

committed

fix(registration): normalize owner email comparison in ReserveOrderTask

smarcet

committed

refactor(imports): drop AmbiguousCommitException special-casing from CSV importers

smarcet

committed

chore(ci): read all Stripe test credentials from repository secrets

smarcet

committed

test(payments): keep Stripe webhook creation out of CI runs

smarcet

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error#533

fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error#533
smarcet wants to merge 31 commits into
mainfrom
hotfix/doctrine-tx-manager

Commits on Jul 15, 2026

fix(transactions): prevent nested transaction from destroying outer EntityManager on connection error

fix(transactions): harden root/nested split against phantom writes and masked errors

fix(tx): remove DBAL savepoints, propagate native isRollbackOnly guard

test: nested-transaction rollback coverage for SummitOrderService, SummitService, SpeakerService, PresentationService, SummitPromoCodeService

test: HTTP-level exception-branch coverage for order/attendee ticket creation

docs(adr): nested transaction rollback safety decision record

test: nested-transaction rollback coverage for remaining ADR-003 gaps

docs(adr): update nested-tx rollback ADR with remaining test coverage

docs(adr): remove docs/plans references from ADR-003

chore: add nested-tx rollback test classes to push.yml CI matrix

docs(adr): clarify origin/main's actual pre-branch behavior in ADR-003

fix(transactions): never retry ambiguous commits, refuse closed-EM re-entry

fix(transactions): discard broken manager/connection when rollback fails

refactor(transactions): deduplicate failure-cleanup paths in DoctrineTransactionService

fix(transactions): surface ambiguous commit failures as AmbiguousCommitException

fix(registration): log-and-skip failing rows in CSV ticket data import

fix(transactions): never classify rollback-only commit failures as ambiguous

docs(adr): log AmbiguousCommitException consumer wiring as known gap

fix(registration): keep import file when a CSV row commit outcome is unknown

fix(summit): re-fetch summit per row in processEventData CSV import

test(imports): volume coverage for every CSV import service method

fix(registration): support guest buyers on the reserve saga

test(orders): revive reserve API tests and cover the order cancel endpoint

fix(auth): flush email invalidation before reassigning a colliding email

chore(ci): run MemberServiceTest in the integration matrix

chore(ci): wire Stripe test credentials from repository secrets

fix(summit): keep import files when a row commit outcome is unknown

fix(registration): normalize owner email comparison in ReserveOrderTask

refactor(imports): drop AmbiguousCommitException special-casing from CSV importers

chore(ci): read all Stripe test credentials from repository secrets

test(payments): keep Stripe webhook creation out of CI runs