Skip to content
This repository has been archived by the owner on May 21, 2022. It is now read-only.

exceeds_window_size from Mint.HTTP2 when using Mojito.post #54

Closed
dch opened this issue Apr 6, 2020 · 3 comments · Fixed by #68
Closed

exceeds_window_size from Mint.HTTP2 when using Mojito.post #54

dch opened this issue Apr 6, 2020 · 3 comments · Fixed by #68
Labels
bug Something isn't working

Comments

@dch
Copy link

dch commented Apr 6, 2020

{:error, %Mojito.Error{message: nil, reason:
  %Mint.HTTPError{module: Mint.HTTP2,
    reason: {:exceeds_window_size, :request, 65536}}}}

I am seeing this bubble up occasionally from Mojito.post (when POSTing to Wordpress); it looks like Mojito needs to renegotiate the window size after initial setup of the connection, or when this error is returned, and then send chunked data over the stream.

@jayjun
Copy link

jayjun commented Jun 16, 2020

One workaround is downgrading to HTTP/1.x. #41

Tesla’s Mint adapter has the same issue. I’ve described why the error happens in detail elixir-tesla/tesla#394. The fix is to receive frames occasionally while sending, which could mean rearchitecting the library.

@gamache
Copy link
Contributor

gamache commented Jun 17, 2020

@dch and @jayjun thank you for documenting this. I can't work on this in the next couple of days but I will look into this.

@gamache gamache added the bug Something isn't working label Jun 18, 2020
@gamache
Copy link
Contributor

gamache commented Jun 26, 2020

This is confirmed. A workaround is to force HTTP/1 operation by passing protocols: [:http1] in the Mojito request options.

gamache added a commit that referenced this issue Dec 10, 2020
* upgrade to Cowboy 2.8

* upgrade Mint to 1.1; add breaking test for #54

* checkpoint

* tests pass

* removed redundant apply_resp implementations

* moved put_chunk into Mojito.Response

* test timeouts during stream_request_body

* remove unused variable

* update Travis config to use Elixir 1.9 and 1.10

* update OTP in Travis

* fix booboo in request/single.ex
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants