Skip to content

feat: login with browser#9951

Open
owenrumney wants to merge 3 commits intoaquasecurity:mainfrom
owenrumney:feat/login-with-browser
Open

feat: login with browser#9951
owenrumney wants to merge 3 commits intoaquasecurity:mainfrom
owenrumney:feat/login-with-browser

Conversation

@owenrumney
Copy link
Contributor

Description

After having discussions internally, the logic for working with the Trivy Cloud integration is going to be simplified greatly. This involves removing the Cloud specific config file and bringing the config into the existing Trivy config. The cloud config commands will be removed as they aren't needed anymore.

  • Add standard config entries to scan commands to configure the pro integration.
    • in CI, the TRIVY_PRO_TOKEN env var can be used to login
  • Add login/logout flow to get a token from the API using the browser. Login with trivy login, if providing --token, that will be stored in the keyring, if not, the browser will be opened up to login and generate a token and callback to the cli before saving to the keyring.
  • Some small updates to the log prefix
  • Generate the docs for the new commands
Kapture.2025-11-26.at.12.40.34.mp4

Related issues

Closes #9693
Closes #9746

Checklist

  • I've read the guidelines for contributing to this repository.
  • I've followed the conventions in the PR title.
  • I've added tests that prove my fix is effective or that my feature works.
  • I've updated the documentation with the relevant information (if needed).

After having discussions internally, the logic for working with the
Trivy Cloud integration is going to be simplified greatly. This involves
removing the Cloud specific config file and bringing the config into the
existing Trivy config. The `cloud config` commands will be removed as
they aren't needed anymore.

As the focus is going to be on CI/CD integration with the Trivy Cloud
platform, there isn't currently a need to have the Login and Logout
mechanisms.
Copilot AI review requested due to automatic review settings December 16, 2025 13:42
@owenrumney owenrumney requested a review from knqyf263 as a code owner December 16, 2025 13:42
Copy link
Contributor

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Add login/logout flow to get a token from the API using the browser.

Login with trivy login, if providing --token, that will be stored in the
keyring, if not, the browser will be opened up to login and generate a
token and callback to the cli before saving to the keyring.

Some small updates to the log prefix

Generate the docs for the new commands
@owenrumney owenrumney force-pushed the feat/login-with-browser branch from b066a56 to 5d0aaff Compare December 16, 2025 14:30
@owenrumney owenrumney requested a review from Copilot December 16, 2025 14:47
@owenrumney owenrumney marked this pull request as draft December 16, 2025 14:47
Copy link
Contributor

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

Copilot reviewed 26 out of 27 changed files in this pull request and generated 3 comments.


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

@owenrumney owenrumney marked this pull request as ready for review December 16, 2025 15:08
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.

feat(cli): Add login to pro through the browser refactor(cloud): rework the cloud integration to be more CI focused

1 participant