Skip to content

fix: update OAuth token retrieval to use JWT instead of bearer token#339

Merged
Akanshu-2u merged 2 commits into
masterfrom
ttak/BOMS-488-1
May 6, 2026
Merged

fix: update OAuth token retrieval to use JWT instead of bearer token#339
Akanshu-2u merged 2 commits into
masterfrom
ttak/BOMS-488-1

Conversation

@ttak-apphelix
Copy link
Copy Markdown
Member

@ttak-apphelix ttak-apphelix commented May 6, 2026

Description:

Updates the Jenkins retired_user_cert_remover script’s authentication flow to request and use a JWT (instead of a Bearer token) when calling the LMS retire_certs_s3 endpoint.

JIRA ticket:

https://2u-internal.atlassian.net/browse/BOMS-488

Copilot AI review requested due to automatic review settings May 6, 2026 06:48
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Jenkins retired_user_cert_remover script’s authentication flow to request and use a JWT (instead of a Bearer token) when calling the LMS retire_certs_s3 endpoint.

Changes:

  • Request token_type=jwt from the LMS OAuth token endpoint to obtain a JWT access token.
  • Switch API request authentication header from Authorization: Bearer … to Authorization: JWT ….
  • Update logging/docstring text around token retrieval to reflect JWT usage.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread util/jenkins/retired_user_cert_remover/retired_user_cert_remover.py
Comment thread util/jenkins/retired_user_cert_remover/retired_user_cert_remover.py Outdated
@Akanshu-2u Akanshu-2u merged commit d055ffc into master May 6, 2026
1 of 3 checks passed
@Akanshu-2u Akanshu-2u deleted the ttak/BOMS-488-1 branch May 6, 2026 08:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants