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

User can update parent tenant to have their child tenant as a new parent #5405

Closed
dpham692 opened this issue Jan 6, 2021 · 2 comments · Fixed by #5474
Closed

User can update parent tenant to have their child tenant as a new parent #5405

dpham692 opened this issue Jan 6, 2021 · 2 comments · Fixed by #5474
Assignees
Labels
bug something isn't working as intended high impact impacts the basic function, deployment, or operation of a CDN Traffic Ops related to Traffic Ops

Comments

@dpham692
Copy link
Contributor

dpham692 commented Jan 6, 2021

I'm submitting a ...

  • bug report

Traffic Control components affected ...

  • Traffic Portal
  • Traffic Ops

Current behavior:

User can update parent tenant to have their child tenant as a new parent and the result is it will delete all the tenant from parent to the child

Expected behavior:

System should not let user update parent tenant to have their child tenant as a new parent tenant.

Minimal reproduction of the problem with instructions:

  1. Create a tenant with few child tenant.
  2. Update parent tenant to have child tenant and it will wipe out all the tenant.

Anything else:

@dpham692 dpham692 added the bug something isn't working as intended label Jan 6, 2021
@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops high impact impacts the basic function, deployment, or operation of a CDN labels Jan 6, 2021
@mitchell852
Copy link
Member

mitchell852 commented Jan 6, 2021

good catch! yeah, obviously a tenant's parent cannot be a child of that tenant. 👍 - needs a simple parent check on PUT and if child return 400 with message.

@tcfdev
Copy link
Collaborator

tcfdev commented Jan 20, 2021

@mitchell852 I'll take this one. Could you assign it to me?

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 high impact impacts the basic function, deployment, or operation of a CDN Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants