Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

mysqlclient 2.1 no longer provides escape_string #6978

Closed
chrisburr opened this issue Apr 23, 2023 · 1 comment
Closed

mysqlclient 2.1 no longer provides escape_string #6978

chrisburr opened this issue Apr 23, 2023 · 1 comment

Comments

@chrisburr
Copy link
Member

chrisburr commented Apr 23, 2023 •
edited
Loading

PyMySQL/mysqlclient#511

Once dealt with we should revert DIRACGrid/DIRACOS2@4ca5a1b
@fstagni
Copy link
Contributor

fstagni commented Jan 30, 2024

This would mean using parametrized queries (I have not found other solutions, at least not after a quick search).
Unfortunately, this would mean for DIRAC a lot of changes and we are not, I think, willing to embark on that for not obvious gains. Therefore I close this, and we'll "solve" it with sqlalchemy in DiracX. Feel free to disagree.

@fstagni fstagni closed this as completed Jan 30, 2024
@fstagni fstagni mentioned this issue Feb 7, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@fstagni @chrisburr