Skip to content
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

Fix response event not being emitted on cache verify request #1305

Merged
merged 7 commits into from
Jul 4, 2020
Merged

Fix response event not being emitted on cache verify request #1305

merged 7 commits into from
Jul 4, 2020

Conversation

Giotino
Copy link
Collaborator

@Giotino Giotino commented Jun 5, 2020

That's only a workaround, a real solution is still WIP (this PR will be updated).

Checklist

  • I have read the documentation.
  • I have included a pull request description of my changes.
  • I have included some tests.

Fixes #1287

test/cache.ts Outdated Show resolved Hide resolved
@Giotino Giotino marked this pull request as ready for review June 6, 2020 09:41
source/core/index.ts Outdated Show resolved Hide resolved
@adityapatadia
Copy link

Any thoughts on merging this one? #1287 is pending due to this I think?

@szmarczak szmarczak changed the title ETag related cache problem Fix response event not being emitted on cache verify request Jul 4, 2020
@szmarczak szmarczak merged commit da4769e into sindresorhus:master Jul 4, 2020
@szmarczak
Copy link
Collaborator

Good job @Giotino! 🙌

@adityapatadia
Copy link

I think this is still not available in 11.4.0. Is this planned for future release?

@szmarczak
Copy link
Collaborator

Is this planned for future release?

Yes. I'll try to make another one ASAP.

@Giotino Giotino deleted the issue-1287 branch July 5, 2020 10: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.

Request hangs up with GOT v11
3 participants