Skip to content

Migrate to use OIDC#234

Merged
lfarrel6 merged 1 commit intomasterfrom
liam/inf-31-move-evervault-node-to-use-oidc
Mar 3, 2026
Merged

Migrate to use OIDC#234
lfarrel6 merged 1 commit intomasterfrom
liam/inf-31-move-evervault-node-to-use-oidc

Conversation

@lfarrel6
Copy link
Member

@lfarrel6 lfarrel6 commented Mar 3, 2026

Why

Node SDK publishing should be restricted to OIDC tokens from this repo

How

  • Split up release workflow into changeset and publish steps
  • Remove references to access tokens
  • Add OIDC permissions to workflow

@changeset-bot
Copy link

changeset-bot bot commented Mar 3, 2026

⚠️ No Changeset found

Latest commit: c4cf2a9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@socket-security
Copy link

@socket-security
Copy link

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
License policy violation: npm chai-as-promised under WTFPL

License: WTFPL - the applicable license policy does not allow this license (4) (npm metadata)

License: WTFPL - the applicable license policy does not allow this license (4) (package/LICENSE.txt)

License: WTFPL - the applicable license policy does not allow this license (4) (package/package.json)

From: package-lock.jsonnpm/chai-as-promised@7.1.1

ℹ Read more on: This package | This alert | What is a license policy violation?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Find a package that does not violate your license policy or adjust your policy to allow this package's license.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/chai-as-promised@7.1.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn High
License policy violation: npm typescript under CC-BY-4.0

License: CC-BY-4.0 - the applicable license policy does not allow this license (4) (package/ThirdPartyNoticeText.txt)

License: LicenseRef-W3C-Community-Final-Specification-Agreement - the applicable license policy does not allow this license (4) (package/ThirdPartyNoticeText.txt)

License: MIT-Khronos-old - the applicable license policy does not allow this license (4) (package/ThirdPartyNoticeText.txt)

From: package-lock.jsonnpm/typescript@5.3.3

ℹ Read more on: This package | This alert | What is a license policy violation?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Find a package that does not violate your license policy or adjust your policy to allow this package's license.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/typescript@5.3.3. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@lfarrel6 lfarrel6 marked this pull request as ready for review March 3, 2026 20:31
@lfarrel6 lfarrel6 requested a review from CianT123 March 3, 2026 20:32
@lfarrel6 lfarrel6 merged commit 8df0b64 into master Mar 3, 2026
8 checks passed
@lfarrel6 lfarrel6 deleted the liam/inf-31-move-evervault-node-to-use-oidc branch March 3, 2026 20:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants