-
Notifications
You must be signed in to change notification settings - Fork 125
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
SNOW-761744 Added URL Validator and URL Encoder #757
SNOW-761744 Added URL Validator and URL Encoder #757
Conversation
CLA Assistant Lite bot: I have read the CLA Document and I hereby sign the CLA 1 out of 2 committers have signed the CLA. |
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
So we just add some util functions without using them at this moment? |
Description
Added URL Validator to ensure that the SSO URL being used in external browser is valid. Also added URL Encoder to encode base64 strings to url safe format.
Checklist
make fmt
to fix inconsistent formatsmake lint
to get lint errors and fix all of them