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

"Apply algorithm to response" is recursive with no base case #63

Closed
jyasskin opened this issue Feb 6, 2017 · 4 comments · Fixed by #97
Closed

"Apply algorithm to response" is recursive with no base case #63

jyasskin opened this issue Feb 6, 2017 · 4 comments · Fixed by #97

Comments

@jyasskin
Copy link
Member

jyasskin commented Feb 6, 2017

Apply algorithm to response says:

Let result be the result of §3.4.1 Apply algorithm to response to the representation data without any content-codings applied, except when the user agent intends to consume the content with content-encodings applied. In the latter case, let result be the result of applying algorithm to the representation data.

If the user agent intends to consume the content without content-encodings, then that just loops infinitely.

I also note that response is never used. And that this algorithm is never called.

@mozfreddyb
Copy link
Collaborator

Can you take a look, @fmarier?

@jyasskin
Copy link
Member Author

jyasskin commented Feb 8, 2017

It is used after all. I needed to search for a direct link to the anchor, instead of relying on Bikeshed's dfn-panels.

@fmarier
Copy link
Member

fmarier commented Feb 8, 2017

So is there an issue after all?

@fmarier fmarier removed their assignment Feb 8, 2017
@jyasskin
Copy link
Member Author

jyasskin commented Feb 8, 2017

Yes, it's still recursive and doesn't use response.

annevk added a commit that referenced this issue Feb 19, 2021
Corresponding Fetch PR: whatwg/fetch#1172.

Closes #63. Helps with #83.
annevk added a commit that referenced this issue Feb 22, 2021
Corresponding Fetch PR: whatwg/fetch#1172.

Closes #63. Helps with #83.
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 a pull request may close this issue.

3 participants