Skip to content

Conversation

@achingbrain
Copy link
Member

If an empty ndjson response has been cached, the response body can be undefined so handle this case.

Fixes regression introduced in #174

Change checklist

  • I have performed a self-review of my own code
  • I have made corresponding changes to the documentation if necessary (this includes comments as well)
  • I have added tests that prove my fix is effective or that my feature works

If an empty ndjson response has been cached, the response body can
be undefined so handle this case.
@achingbrain achingbrain requested a review from a team as a code owner December 3, 2025 06:25
@achingbrain achingbrain changed the title fix: cached response body can be undefined fix: cached ndjson response body can be undefined Dec 3, 2025
@achingbrain achingbrain merged commit 7e3c88a into main Dec 3, 2025
21 checks passed
@achingbrain achingbrain deleted the fix/cached-body-can-be-missing branch December 3, 2025 06:31
github-actions bot pushed a commit that referenced this pull request Dec 3, 2025
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🎉 This PR is included in version @helia/delegated-routing-v1-http-api-client-v5.1.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants