-
Notifications
You must be signed in to change notification settings - Fork 16
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
Ensure namespace and collection names are validated #149
Comments
➤ Mahesh Rajamani commented: Ivan Senic I know you have done the validation for namespace and collection name, do we need to check for C* reserved word? |
@maheshrajamani what reserved words are you referring to, I was checking this:
|
The issue states "Only allow names that C* would allow. ". I was checking if more validation is needed with cassandra reserved words. But with the change done to enclose collection names with quotes this should be handled. Think we can close this issue. |
I still need to enforce validation on the path parameters. Will do this and resolve it. |
Only allow names that C* would allow. Add validation to the commands. Improve URI path patterns to include the validation as well, or maybe simply validate
CommandContext
that's being passed to theCommandProcessor
.The text was updated successfully, but these errors were encountered: