Skip to content

Conversation

@szokeasaurusrex
Copy link
Member

Description

Deprecate the apiKey field, which is used to set the already-deprecated SENTRY_API_KEY environment variable. Also, update the docstrings for apiKey and authToken to clarify that the two options are not interchangeable. apiKey only takes a legacy API key; authToken takes auth tokens.

Issues

@linear
Copy link

linear bot commented Nov 11, 2025

@szokeasaurusrex szokeasaurusrex changed the base branch from master to graphite-base/2937 November 11, 2025 11:42
szokeasaurusrex added a commit that referenced this pull request Nov 11, 2025
### Description
Deprecate the `apiKey` field, which is used to set the already-deprecated `SENTRY_API_KEY` environment variable. Also, update the docstrings for `apiKey` and `authToken` to clarify that the two options are not interchangeable. `apiKey` only takes a legacy API key; `authToken` takes auth tokens.

### Issues
- Resolves #2936
- Resolves CLI-223
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/deprecate-api-key-js branch from a30afca to b4be8d8 Compare November 11, 2025 11:42
@szokeasaurusrex szokeasaurusrex changed the base branch from graphite-base/2937 to szokeasaurusrex/fix-changelog November 11, 2025 11:42
Copy link
Member Author

szokeasaurusrex commented Nov 11, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

### Description
Deprecate the `apiKey` field, which is used to set the already-deprecated `SENTRY_API_KEY` environment variable. Also, update the docstrings for `apiKey` and `authToken` to clarify that the two options are not interchangeable. `apiKey` only takes a legacy API key; `authToken` takes auth tokens.

### Issues
- Resolves #2936
- Resolves CLI-223
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/deprecate-api-key-js branch from b4be8d8 to 0d3b235 Compare November 11, 2025 11:43
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/fix-changelog branch from aece140 to b850dc9 Compare November 11, 2025 11:43
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review November 11, 2025 11:43
@szokeasaurusrex szokeasaurusrex requested review from a team as code owners November 11, 2025 11:43
@szokeasaurusrex szokeasaurusrex requested review from nicohrubec and removed request for a team and nicohrubec November 11, 2025 11:43
Base automatically changed from szokeasaurusrex/fix-changelog to master November 11, 2025 11:47
@szokeasaurusrex
Copy link
Member Author

@andreiborza does this look good from the JS perspective?

Copy link
Member

@andreiborza andreiborza left a comment

Choose a reason for hiding this comment

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

LGTM

@szokeasaurusrex szokeasaurusrex merged commit 7595ba9 into master Nov 11, 2025
29 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/deprecate-api-key-js branch November 11, 2025 13:51
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.

Deprecate apiKey in JS API

4 participants