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

Frank/#165 fix #166

Merged
merged 6 commits into from
Mar 1, 2021
Merged

Frank/#165 fix #166

merged 6 commits into from
Mar 1, 2021

Conversation

shagamemnon
Copy link
Contributor

Fixes #165 :)

src/index.ts Show resolved Hide resolved
Copy link

@harrishancock harrishancock left a comment

Choose a reason for hiding this comment

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

My one substantive comment would be that a test would be nice. That might be a bit of a headache, though, from looking at our testing code...

src/index.ts Show resolved Hide resolved
src/index.ts Show resolved Hide resolved
@shagamemnon shagamemnon marked this pull request as draft February 7, 2021 15:42
@shagamemnon
Copy link
Contributor Author

Issue cloudflare/workers-sdk#4975 needs to be investigated before proceeding

@koeninger
Copy link

@shagamemnon would we be in better shape than we are today if this PR was merged in the meantime?

@shagamemnon shagamemnon marked this pull request as ready for review February 27, 2021 00:20
@shagamemnon
Copy link
Contributor Author

@koeninger this is ready now. Can you squash the commits?

@koeninger koeninger merged commit 682cba0 into cloudflare:master Mar 1, 2021
@koeninger
Copy link

@shagamemnon cool thanks, squashed + merged

@caass caass added this to the 0.1.1 milestone Mar 15, 2021
xiaolanglanglang pushed a commit to xiaolanglanglang/kv-asset-handler that referenced this pull request May 18, 2021
* defined response status on cached responses. fixes 165

* updated mocks and added tests for range requests

* fixed range request test
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kv-asset-handler can translate 206 responses to 200 responses
5 participants