Skip to content

Conversation

lforst
Copy link

@lforst lforst commented Aug 5, 2024

No description provided.

@lforst lforst requested a review from szokeasaurusrex August 5, 2024 09:01
@lforst
Copy link
Author

lforst commented Aug 5, 2024

@szokeasaurusrex anything we should look out for when bumping the CLI?

@szokeasaurusrex
Copy link
Member

szokeasaurusrex commented Aug 5, 2024

I cannot think of anything specific @lforst. Maybe you can call out some of the important changes since then:

  • (Soft) validation of auth tokens (warn when invalid format recognized)
  • CLI should now be fully compatible with hybrid cloud EU region
  • A security fix (fix: Redact auth tokens when logging CLI args sentry-cli#2115 and fix: Improve token redaction in CLI arg logging sentry-cli#2118) to prevent auth tokens from being logged to console in certain situations. This bug might not be applicable to the bundler plugins, though. The tokens only got logged when both of the following were true:
    • The token was passed in via the --auth-token argument, not via an environment variable or configuration file, and
    • The CLI's log level was set to info or debug.

@lforst lforst merged commit a11470f into main Aug 5, 2024
18 checks passed
@lforst lforst deleted the lforst-bump-sentry-cli branch August 5, 2024 09:32
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