textSQL icon indicating copy to clipboard operation
textSQL copied to clipboard

Published 20 hours ago • verified publisher icon caesarHQ

Update utils.py

Open nrghosh opened this issue 1 year ago • 1 comments

Don't want to let user input mess with table permissions on our side. Alternatively, we could just fix this at the session level with SET SESSION TRANSACTION READ ONLY;

nrghosh avatar Mar 12 '23 11:03 nrghosh

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
text-sql ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 12, 2023 at 11:03AM (UTC)

vercel[bot] avatar Mar 12 '23 11:03 vercel[bot]

@alex-sun-kuo done + merged in separate PR [see: https://github.com/caesarHQ/textSQL/pull/61 ]. It's not costly to filter the generated SQL statement against the blacklisted words so I didn't remove this section (checking, etc) entirely in the other PR -- so we could still merge this.

Don't have write access though but it should be a simple conflict to resolve (if we want to merge).

nrghosh avatar Mar 13 '23 05:03 nrghosh