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.

Sign up for GitHub

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

Jump to bottom

Internal Server Error when update root tenant #5396

Closed
dpham692 opened this issue Dec 29, 2020 · 2 comments · Fixed by #5397
Closed

Internal Server Error when update root tenant #5396

dpham692 opened this issue Dec 29, 2020 · 2 comments · Fixed by #5397
Assignees
@srijeet0406
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

Comments

@dpham692
Copy link
Contributor

dpham692 commented Dec 29, 2020 •
edited
Loading

I'm submitting a ...

  • bug report

Traffic Control components affected ...

  • Traffic Portal

Current behavior:

  1. Go to TP
  2. Click on User Admin -> Tenants
  3. Try to update root tenant. Notice that it will generate Internal Server Error message

Expected behavior:

TP should let user update root tenant
When attempting to update the immutable root tenant, the user should get a 400 Bad Request with a message along the lines of "Root tenant is immutable".

Minimal reproduction of the problem with instructions:

Anything else:

ERROR: api.go:209: 2020-12-29T17:08:32.859350184Z: 96.118.16.179:37360 checking tenant authorized: sql: Scan error on column index 0, name "parent_id": converting NULL to int is unsupported
@dpham692 dpham692 added the bug something isn't working as intended label Dec 29, 2020
@mitchell852
Copy link
Member

actually, when attempting to update the immutable root tenant, the user should get a 400 Bad Request instead I believe with a message along the lines of "Root tenant is immutable".

@mitchell852 mitchell852 added Traffic Ops related to Traffic Ops low impact affects only a small portion of a CDN, and cannot itself break one labels Dec 29, 2020
@srijeet0406
Copy link
Contributor

I can take this one

@srijeet0406 srijeet0406 mentioned this issue Dec 30, 2020
Merged
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@srijeet0406 srijeet0406

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
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Return the correct error while trying to update the root tenant srijeet0406/trafficcontrol
3 participants
@mitchell852 @dpham692 @srijeet0406