php / php/php-src

No exception on `mysqli::rollback` when transaction is no longer active

Offen
#12,576 3 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Extension: mysqli Status: Needs Triage
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.2k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

Repro steps:

  1. have an open mysqli connection
  2. start transaction using mysqli::begin_transaction
  3. run some query like analyze table xx, this is "administrative statement" [1] which implicit autocommit
  4. rollback transaction using mysqli::rollback and notice no exception is thrown

Since PHP 8.0, PDO checks if a transaction is active and throws an exception: PDOException: There is no active transaction

mysqli must throw the same exception as well.

Related MariaDB tickets:

[1] docs https://dev.mysql.com/doc/refman/8.0/en/implicit-commit.html and repro: https://dbfiddle.uk/coUjimLq

PHP Version

7.4 - master

Operating System

any

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginnen Sie am Einstiegspunkt mysqli::rollback und reproduzieren Sie die in diesem Issue beschriebene Sequenz aus begin_transaction, administrative-statement und rollback. Vergleichen Sie das Verhalten des Transaktionsstatus mit dem Ausnahmeverhalten von PDO in PHP 8 und fügen Sie Tests hinzu, die zeigen, dass rollback eine Exception meldet, wenn die Transaktion nicht mehr aktiv ist.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c, php
Bereich
databases
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.