Skip to content

FCM handle Mint connection closed case - #309

Closed
mbarekTrismegistus wants to merge 2 commits into
codedge-llc:mainfrom
mbarekTrismegistus:main
Closed

mbarekTrismegistus wants to merge 2 commits into
codedge-llc:mainfrom
mbarekTrismegistus:main

Conversation

@mbarekTrismegistus

Copy link
Copy Markdown

I've runned into this error when i switched to the latest version that uses mint and not kadabra
it's basically straightforward, the closed connection case is not handled so im handling it

14:50:59.542 [error] GenServer #PID<0.2806.0> terminating Jul 11 14:50:59 instance-20250220-1457 zawajonlypheonix[2408916]: ** (MatchError) no match of right hand side value: {:error, %Mint.HTTP2{transport: Mint.Core.Transport.SSL, socket: {:sslsocket, {:gen_tcp, #Port<0.157>, :tls_connection, :undefined}, [#PID<0.2810.0>, #PID<0.2809.0>]}, mode: :active, hostname: "fcm.googleapis.com", port: 443, scheme: "https", authority: "fcm.googleapis.com", state: :closed, buffer: "", send_window_size: 1048576, receive_window_size: 16777216, receive_window_remaining: 16777216, receive_window_update_threshold: 160000, encode_table: %HPAX.Table{protocol_max_table_size: 4096, max_table_size: 4096, huffman_encoding: :never, entries: [], size: 0, length: 0, pending_minimum_resize: nil}, decode_table: %HPAX.Table{protocol_max_table_size: 4096, max_table_size: 4096, huffman_encoding: :never, entries: [], size: 0, length: 0, pending_minimum_resize: nil}, ping_queue: {[], []}, client_settings_queue: {[], []}, next_stream_id: 3, streams: %{}, open_client_stream_count: 0, open_server_stream_count: 0, reserved_server_stream_count: 0, ref_to_stream_id: %{}, server_settings: %{max_frame_size: 16384, initial_window_size: 1048576, max_concurrent_streams: 100, max_header_list_size: 65536, enable_connect_protocol: false, enable_push: true}, client_settings: %{max_frame_size: 16777215, initial_window_size: 2147483647, max_concurrent_streams: 100, max_header_list_size: 262144, enable_push: true}, headers_being_processed: nil, proxy_headers: [], private: %{}, log: false}, %Mint.HTTPError{reason: :closed, module: Mint.HTTP2}}

this is just my approach to make it work for my own use for now as for version 2.1 is not published yet, maybe this is fixed in the said version

@hpopp

hpopp commented Sep 9, 2026

Copy link
Copy Markdown
Member

Thanks for this. I ended up addressing the issue in my own #311 but left you partial credit in the changelog

@hpopp hpopp closed this Sep 9, 2026
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.

2 participants