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

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 19 commits into
mainfrom
hotfix/doctrine-tx-manager

Commits on Jul 10, 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

Commits on Jul 11, 2026

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