Skip to content

Conversation

@xPaw
Copy link
Contributor

@xPaw xPaw commented Feb 20, 2022

While the code already checks for SELECT statements only, this should enforce it on the database level.

For mysqli driver, it will throw "Cannot execute statement in a READ ONLY transaction" error, while in PDO in worse case it will roll back any of the changed data.

@staabm staabm merged commit 99a43d0 into staabm:main Feb 20, 2022
@staabm
Copy link
Owner

staabm commented Feb 20, 2022

Thank you

@xPaw xPaw deleted the transactions branch February 20, 2022 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants