Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

fix: check for flushed response headers during proxy request errors #88

Merged
merged 6 commits into from
Jul 21, 2023
Merged

fix: check for flushed response headers during proxy request errors #88

merged 6 commits into from
Jul 21, 2023

Conversation

jcbain
Copy link
Contributor

@jcbain jcbain commented Apr 18, 2023 •
edited
Loading

Resolves #86

Checklist

q0rban
q0rban reviewed Apr 19, 2023
View reviewed changes
index.js Outdated Show resolved Hide resolved
@Fdawgs Fdawgs changed the title check for flushed response headers during proxy request errors (#86) fix: check for flushed response headers during proxy request errors May 6, 2023
@Eomm Eomm requested a review from a team July 19, 2023 10:13
@q0rban q0rban mentioned this pull request Jul 19, 2023
Merged
4 tasks
Uzlopak
Uzlopak reviewed Jul 19, 2023
View reviewed changes
Copy link
Contributor

@Uzlopak Uzlopak left a comment

Choose a reason for hiding this comment

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

I have no clue.

@mcollina mcollina merged commit 2aecb29 into fastify:master Jul 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Uzlopak Uzlopak Uzlopak left review comments

@q0rban q0rban q0rban left review comments

@Eomm Eomm Eomm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Response being written after being sent because res.sent is undefined
5 participants
@jcbain @q0rban @Uzlopak @Eomm @mcollina