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

Returning authenticated cookie when using TO Client Cert Auth #7984

Merged
merged 1 commit into from
Apr 11, 2024

Conversation

rimashah25
Copy link
Contributor

@rimashah25 rimashah25 commented Apr 9, 2024

Traffic Ops session returned after logging in withcerts fails to authenticate in subsequent requests


Which Traffic Control components are affected by this PR?

  • Traffic Ops

What is the best way to verify this PR?

Run traffic ops and set it up to work with client certs as explained in #7392
Run the tests and make sure everything passes
Manual testing:
Write a script that logs in to Traffic Ops using the LoginWithCert method
Now, using this session, make a request to an authenticated endpoint and make sure you get no errors

If this is a bugfix, which Traffic Control versions contained the bug?

master

PR submission checklist

@rimashah25 rimashah25 marked this pull request as ready for review April 9, 2024 16:14
@zrhoffman zrhoffman merged commit e4729e8 into apache:master Apr 11, 2024
80 of 81 checks passed
@zrhoffman zrhoffman added bug something isn't working as intended Traffic Ops related to Traffic Ops TO Client (Go) related to the Go implementation of a TC client authentication Relating to login, registration, passwords, tokens, etc. labels Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
authentication Relating to login, registration, passwords, tokens, etc. bug something isn't working as intended TO Client (Go) related to the Go implementation of a TC client Traffic Ops related to Traffic Ops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants