Skip to content

fix(cli): allow custom 403 error message #13443

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Jun 16, 2025
Merged

Conversation

epallerols
Copy link
Contributor

Currently, the link command handles all 403 errors from the API with the same generic error message, except when the error contains "missingToken" or "invalidToken". This approach lacks specificity for various error conditions.

This PR modifies the error handling to only use the generic message when the error code is specifically "forbidden" or "team_unauthorized". All other 403 errors will now pass through their original API error messages. This change enables more precise error reporting to users, making troubleshooting more straightforward by exposing the actual underlying issues rather than masking them behind a generic message.

@epallerols epallerols requested a review from a team as a code owner June 12, 2025 11:32
Copy link

changeset-bot bot commented Jun 12, 2025

🦋 Changeset detected

Latest commit: 08521ff

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
vercel Minor

Not sure what this means? Click here to learn what changesets are.

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

@epallerols epallerols requested a review from markandrus June 12, 2025 11:33
@epallerols epallerols added this pull request to the merge queue Jun 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 13, 2025
@epallerols epallerols added this pull request to the merge queue Jun 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jun 13, 2025
@epallerols epallerols added this pull request to the merge queue Jun 16, 2025
Merged via the queue into main with commit d5ae0fe Jun 16, 2025
439 of 443 checks passed
@epallerols epallerols deleted the enric/iam-link-error-handling branch June 16, 2025 08:19
github-merge-queue bot pushed a commit that referenced this pull request Jun 16, 2025
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## vercel@43.2.0

### Minor Changes

- fix(cli): allow custom 403 error message
([#13443](#13443))

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

4 participants