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

use Identifier format instead of string format #296

Merged
merged 15 commits into from
Aug 15, 2024
Merged

Conversation

ChuckHend
Copy link
Contributor

Hopefully addresses #295

@vrmiguel vrmiguel requested a review from theory August 14, 2024 19:17
@ChuckHend ChuckHend marked this pull request as ready for review August 14, 2024 20:27
Copy link
Contributor

@theory theory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Big improvement, left a couple of suggestions.

pgmq-extension/sql/pgmq.sql Outdated Show resolved Hide resolved
pgmq-extension/sql/pgmq.sql Outdated Show resolved Hide resolved
@ChuckHend ChuckHend mentioned this pull request Aug 14, 2024
@ChuckHend ChuckHend requested a review from theory August 15, 2024 00:22
Copy link
Contributor

@theory theory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@ChuckHend ChuckHend merged commit 04a06b6 into main Aug 15, 2024
13 checks passed
@ChuckHend ChuckHend deleted the fix-sql-injection branch August 15, 2024 03:18
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

Successfully merging this pull request may close these issues.

2 participants