Skip to content

fix(client-fetch): interceptor exposes fetch errors #2657

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

fix(client-fetch): interceptor exposes fetch errors #2657

wants to merge 1 commit into from
+66 −59

Conversation

MeikelLP
Copy link

@MeikelLP MeikelLP commented Sep 18, 2025 •
edited
Loading

fixes #1994

I recommend disabling whitespace in diff

@bolt-new-by-stackblitz Bolt.new (by StackBlitz)
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Sep 18, 2025

⚠️ No Changeset found

Latest commit: f9033ea

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

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

@vercel Vercel
Copy link

vercel bot commented Sep 18, 2025

@MeikelLP is attempting to deploy a commit to the Hey API Team on Vercel.

A member of the Team first needs to authorize it.

@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. bug 🔥 Something isn't working client Client package related labels Sep 18, 2025
github-advanced-security[bot]
github-advanced-security bot found potential problems Sep 18, 2025
View reviewed changes
examples/openapi-ts-fetch/src/client/client/client.ts
response = await fn(response, request, opts);
}
let error : unknown;
let response

Check notice

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (93% of all statements in
the enclosing function
Loading
have an explicit semicolon).
examples/openapi-ts-fetch/src/client/client/client.ts
try {
response = await _fetch(request);
}catch (e) {
error = e

Check notice

Code scanning / CodeQL

Semicolon insertion Note

Avoid automated semicolon insertion (93% of all statements in
the enclosing function
Loading
have an explicit semicolon).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
bug 🔥 Something isn't working client Client package related size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Client interceptor not working for failed requests?
1 participant
@MeikelLP