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.

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

Feature request: JSON string placeholder #20

Open
whisper-bye opened this issue Oct 27, 2022 · 1 comment
Open

Feature request: JSON string placeholder #20

whisper-bye opened this issue Oct 27, 2022 · 1 comment

Comments

@whisper-bye
Copy link

From this

JsonbDSL.field("{\"id\": \"1337\"}")

To this

JsonbDSL.field("{\"id\":{0}}", "1337")
@t9t
Copy link
Owner

t9t commented Oct 30, 2022

Thanks for the suggestion! Unfortunately maintaining this project is a bit on the backburner for me because I don't use it myself anymore (I have no projects where it's applicable).

Having said that, with this particular feature I also have some worries as a bad implementation could lead to the potential for SQL injection. If possible, existing JOOQ features should be used for the string placeholder functionality.

I might have some time to look into it this week but I can't promise that unfortunately. If you feel up to creating a merge request that would be greatly appreciated.

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

No branches or pull requests

2 participants