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

Off by one error in queue name length validation #349

Open
axelfontaine opened this issue Dec 3, 2024 · 0 comments
Open

Off by one error in queue name length validation #349

axelfontaine opened this issue Dec 3, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@axelfontaine
Copy link

Code: https://github.com/tembo-io/pgmq/blob/main/pgmq-extension/sql/pgmq.sql#L593

  IF length(queue_name) >= 48 THEN
    RAISE EXCEPTION 'queue name is too long, maximum length is 48 characters';
  END IF;

Length of 48 triggers exception that the max length should be 48.

Either the check should be changed from >= to > or the rror message should say 47.

@ChuckHend ChuckHend added the bug Something isn't working label Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants