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

8f9d508

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

smarcet

committed

1664a15

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

smarcet

committed

210f2cd

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

smarcet

committed

801669b

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

smarcet

committed

9a33863

docs(adr): nested transaction rollback safety decision record

smarcet

committed

48ab18c

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

smarcet

committed

af0f739

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

smarcet

committed

6123534

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

smarcet

committed

fbd08d0

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

smarcet

committed

d5388d0

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

smarcet

committed

006e3f0

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

smarcet

committed

d0a3f2a

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

smarcet

committed

ef5f96e

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

smarcet

committed

ef92755

fix(transactions): surface ambiguous commit failures as AmbiguousCommitException

smarcet

committed

9e83515

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

smarcet

committed

531ceae

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

smarcet

committed

1042e94

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

smarcet

committed

6b25453

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

smarcet

committed

ff25ef1

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

smarcet

committed

39aa137

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

smarcet

committed

95c589b

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

smarcet

committed

40fdbcc

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

smarcet

committed

15efcd1

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

smarcet

committed

5a2ff78

chore(ci): run MemberServiceTest in the integration matrix

smarcet

committed

7aa806a

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

smarcet

committed

b5c0c2b

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

smarcet

committed

aedd59e

fix(registration): normalize owner email comparison in ReserveOrderTask

smarcet

committed

3b796f5

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

smarcet

committed

3abcca3

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

smarcet

committed

8dc9062

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

smarcet

committed

9291321

docs(adr): move SponsorUserSync pair to unreachable after #582 group gates

smarcet

committed

c452755

Navigation Menu

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

Commits on Aug 31, 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

docs(adr): move SponsorUserSync pair to unreachable after #582 group gates