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

Authentication token issues after CDash upgrade #2678

Closed
marc-legendre opened this issue Jan 27, 2025 · 2 comments · Fixed by #2679
Closed

Authentication token issues after CDash upgrade #2678

marc-legendre opened this issue Jan 27, 2025 · 2 comments · Fixed by #2679
Assignees
Milestone

Comments

@marc-legendre
Copy link

Bug report

After upgrading from CDash 3.1.0 to 3.7.0, we noticed that existing CDash API tokens stopped working (I tested a few API calls, they seem to return a 401 error with body {'requirelogin': 1}).

Also, we cannot delete existing tokens. Indeed, in the "My Authentication Tokens" of the CDash UI, the "revoke" (trash can) button does not work, and in the browser devtools I can see that the request to DELETE /api/authtokens/delete/f977...3de675e62b915409d returns a 500 error with

{"error":"Invalid scope listed for auth token with hash f977...409d","code":500}

(We created new API tokens, and we have had no other issue since then.)

Let me know if I can provide any other useful information.

@williamjallen
Copy link
Collaborator

It looks like this was caused by having no default value for the scope column added here. I'll create a PR to add a default value.

@williamjallen williamjallen added this to the v3.8 milestone Jan 28, 2025
@williamjallen williamjallen self-assigned this Jan 28, 2025
github-merge-queue bot pushed a commit that referenced this issue Jan 30, 2025
Fixes #2678 by defaulting all
pre-3.2 tokens to be full access tokens.
@marc-legendre
Copy link
Author

Indeed, I manually updated our cdash database to add the missing scope value, and the issues are gone.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants
@marc-legendre @williamjallen and others