Skip to content

Replace the BASIS token with an updated token#1572

Merged
zlav merged 1 commit into
masterfrom
fix-ci-tokens
Aug 1, 2025
Merged

Replace the BASIS token with an updated token#1572
zlav merged 1 commit into
masterfrom
fix-ci-tokens

Conversation

@zlav

@zlav zlav commented Aug 1, 2025

Copy link
Copy Markdown
Member

Overview

This PR replaces the reference to the BASIS_ACCESS_TOKEN. The token recently expired, and I figured it was a good time to update the name. Basis no longer really exists, and the purpose of the command is to download libraries that have been ripped out from Basis, among other new things like circe. Changing it to ANALYSIS_VENDOR_DOWNLOAD_TOKEN felt appropriate. I'm open to other suggestions.

After this PR merges I will delete the BASIS_ACCESS_TOKEN

Acceptance criteria

This build succeeds.

Testing plan

I validated that the updated token works by replacing the BASIS_ACCESS_TOKEN with the updated token and retrying another failed build.

Risks

The only risk is that the permissions could be different for this token than the BASIS_ACCESS_TOKEN was, but it only has read access, so it should be fine.

References

Failed builds

Checklist

  • I added tests for this PR's change (or explained in the PR description why tests don't make sense).
  • If this PR introduced a user-visible change, I added documentation into docs/.
  • If this PR added docs, I added links as appropriate to the user manual's ToC in docs/README.ms and gave consideration to how discoverable or not my documentation is.
  • If this change is externally visible, I updated Changelog.md. If this PR did not mark a release, I added my changes into an ## Unreleased section at the top.
  • If I made changes to .fossa.yml or fossa-deps.{json.yml}, I updated docs/references/files/*.schema.json AND I have updated example files used by fossa init command. You may also need to update these if you have added/removed new dependency type (e.g. pip) or analysis target type (e.g. poetry).
  • If I made changes to a subcommand's options, I updated docs/references/subcommands/<subcommand>.md.

@zlav zlav requested a review from a team as a code owner August 1, 2025 21:00
@zlav zlav requested review from csasarak and pete-fossa August 1, 2025 21:00

@csasarak csasarak left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

I think we should put this in 1password. Or barring that, make an entry for it in 1password and describe what to do if there are problems in the notes.

@zlav zlav enabled auto-merge (squash) August 1, 2025 21:06
@zlav zlav merged commit e8985df into master Aug 1, 2025
19 of 20 checks passed
@zlav zlav deleted the fix-ci-tokens branch August 1, 2025 21:29
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