Skip to content

Disable OIDC#1601

Merged
MarcAstr0 merged 1 commit intomainfrom
disable-oidc
Mar 20, 2026
Merged

Disable OIDC#1601
MarcAstr0 merged 1 commit intomainfrom
disable-oidc

Conversation

@pn-tams
Copy link
Collaborator

@pn-tams pn-tams commented Mar 20, 2026

Updated permissions and authentication steps for npm registry.

Description

Changes

Checks

  • Project Builds
  • Project passes tests and checks
  • Updated documentation accordingly

Updated permissions and authentication steps for npm registry.
@pn-tams pn-tams requested a review from MarcAstr0 March 20, 2026 12:22
@what-the-diff
Copy link

what-the-diff bot commented Mar 20, 2026

PR Summary

  • Simplified GitHub Actions workflow
    The unnecessary 'permissions' section for 'id-token' and 'pull-requests' have been removed from the GitHub Actions workflow. This streamlining will lead to not only a cleaner code but also faster execution of the workflows.

  • Reinstated npm registry authentication
    The steps originally meant for npm registry authentication have been restored and are now active in the workflow. This would allow continuous integration/delivery processes to proceed more securely by interacting with the npm registry.

  • Optimized usage of NPM_TOKEN
    The 'NPM_TOKEN' is now properly utilized across both authentication steps and the command to publish. This enhances security by ensuring that the process to publish packages to the npm registry uses the correct secret.

@MarcAstr0 MarcAstr0 merged commit 259f888 into main Mar 20, 2026
3 checks passed
@MarcAstr0 MarcAstr0 deleted the disable-oidc branch March 20, 2026 12:26
MarcAstr0 added a commit that referenced this pull request Mar 23, 2026
The @azure/identity security bump (PR #1600) was never published because
the publish workflow had broken OIDC auth at the time. The workflow is
now fixed (PR #1601), so this cosmetic change triggers a new publish run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
MarcAstr0 added a commit that referenced this pull request Mar 23, 2026
* Trigger CI to publish pending version bump

The @azure/identity security bump (PR #1600) was never published because
the publish workflow had broken OIDC auth at the time. The workflow is
now fixed (PR #1601), so this cosmetic change triggers a new publish run.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add rush changeset for CI trigger

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* Add rush changeset for CI trigger (via rush change)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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.

2 participants