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

acme_accounts doesn't validate email or uri #6369

Closed
ocket8888 opened this issue Nov 21, 2021 · 0 comments · Fixed by #6624
Closed

acme_accounts doesn't validate email or uri #6369

ocket8888 opened this issue Nov 21, 2021 · 0 comments · Fixed by #6624
Labels
bug something isn't working as intended low difficulty the estimated level of effort to resolve this issue is low low impact affects only a small portion of a CDN, and cannot itself break one needs more info more information or a better description is needed to proceed Traffic Ops related to Traffic Ops

Comments

@ocket8888
Copy link
Contributor

This Bug Report affects these Traffic Control components:

  • Traffic Ops

Current behavior:

You can submit a POST or PUT to /acme_accounts with an email that isn't an email address, and/or a uri that isn't a valid uri.

Expected behavior:

I think the API should verify those fields, but it's possible I just don't understand some use-case for them being invalid.

Steps to reproduce:

topost -kpa 3.1 acme_accounts '{"email": "not an email", "uri": "not a URI", "provider": "some provider", "privateKey": "some private key"}'

@ocket8888 ocket8888 added bug something isn't working as intended Traffic Ops related to Traffic Ops low impact affects only a small portion of a CDN, and cannot itself break one needs more info more information or a better description is needed to proceed low difficulty the estimated level of effort to resolve this issue is low labels Nov 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug something isn't working as intended low difficulty the estimated level of effort to resolve this issue is low low impact affects only a small portion of a CDN, and cannot itself break one needs more info more information or a better description is needed to proceed Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant