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

Fix /cdns/{name}/federations?id=# to search for CDN #6125

Closed
mattjackson220 opened this issue Aug 19, 2021 · 1 comment · Fixed by #6287
Closed

Fix /cdns/{name}/federations?id=# to search for CDN #6125

mattjackson220 opened this issue Aug 19, 2021 · 1 comment · Fixed by #6287
Assignees
Labels
bug something isn't working as intended low impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops
Milestone

Comments

@mattjackson220
Copy link
Contributor

This Bug Report affects these Traffic Control components:

  • Traffic Ops

Current behavior:

The API endpoint /cdns/{name}/federations?id=# will return the federations information as long as the id is valid, even if the CDN is not valid or does not exist or the federation is not part of that CDN.

Expected behavior:

That endpoint should only return the information for federations within the CDN provided

Steps to reproduce:

Go to the API at /cdns/nonsense/federations?id=# with a valid federation id and see that it returns the information even though nonsense is not a valid CDN

@mattjackson220 mattjackson220 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 labels Aug 19, 2021
@srijeet0406
Copy link
Contributor

@mattjackson220 I can take this one.

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 impact affects only a small portion of a CDN, and cannot itself break one Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants