Allow "pgx" driver to be used with SavePoint
sammy-tam opened this issue · comments
Currently the SAVEPOINT function checks for driver name == "postgres". Since "pgx" driver is now supported by sqlx as well, it'd be great and we allow both.
Hello (and sorry for the extra late reply!)
This is definitely a change we can include
Would you care to open a pull request that includes tests? Thanks in advance
Hi @skateinmars !
Just made a PR to fix this issue earlier today. Would you mind reviewing/merging it, although it's not a security fix? 🙏
We use sqalx extensively and just started using nested transactions, so this change really would go a long way for us!