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

feat(ai-prompt-guard): increase the maximum length of regex expression #12731

Merged
merged 2 commits into from
Mar 26, 2024

Conversation

liverpool8056
Copy link
Contributor

Summary

increase the maximum length of regex expression to 500 for both allow and deny parameter

Checklist

  • The Pull Request has tests
  • A changelog file has been created under changelog/unreleased/kong or skip-changelog label added on PR if changelog is unnecessary. README.md
  • There is a user-facing docs PR against https://github.com/Kong/docs.konghq.com - PUT DOCS PR HERE

Issue reference

FTI-5767

@@ -0,0 +1,4 @@
message: |
**AI-Prompt-Guard**: increase the maximum length of regex expression for both allow and deny parameter
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
**AI-Prompt-Guard**: increase the maximum length of regex expression for both allow and deny parameter
**AI-Prompt-Guard**: increase the maximum length of regex expression to 500 characters for both allow and deny parameter

@chronolaw chronolaw added the cherry-pick kong-ee schedule this PR for cherry-picking to kong/kong-ee label Mar 19, 2024
@locao locao requested a review from flrgh March 19, 2024 20:34
@liverpool8056
Copy link
Contributor Author

Hi @ttyS0e, do you have a moment to give this a review? Thanks!

@flrgh flrgh merged commit 27949b5 into master Mar 26, 2024
26 checks passed
@flrgh flrgh deleted the increase-ai-anthropic-expression-length branch March 26, 2024 20:45
@team-gateway-bot
Copy link
Collaborator

Successfully created cherry-pick PR for master:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants