Attention is currently required from: dexter.
laforge has posted comments on this change by dexter. ( https://gerrit.osmocom.org/c/pysim/+/41508?usp=email )
Change subject: card_key_provider: add PostgreSQL support ......................................................................
Patch Set 4: Code-Review-1
(1 comment)
File pySim/card_key_provider.py:
https://gerrit.osmocom.org/c/pysim/+/41508/comment/7abd6bed_c71e9487?usp=ema... : PS4, Line 229: cur.execute("SELECT column_name FROM information_schema.columns where table_name = '%s';" % t)
I believe here and in all other cur.execute statements you don't want to use '%". See https://www. […]
yeah. this is actually true in any programming language: Always use the parameter expansion/susbstitution features of the library, never use format strings.