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

Ignore unrecognized messages in bridge counter processes #3623

Merged
merged 1 commit into from
Feb 12, 2021

Conversation

vbaranov
Copy link
Member

Motivation

2021-02-10T09:46:14.726 [error] GenServer Explorer.Counters.Bridge terminating
** (FunctionClauseError) no function clause matching in Explorer.Counters.Bridge.handle_info/2
    (explorer 0.0.1) lib/explorer/counters/bridge.ex:92: Explorer.Counters.Bridge.handle_info({:ssl_closed, {:sslsocket, {:gen_tcp, #Port<0.2302>, :tls_connection, :undefined}, [#PID<0.10152.0>, #PID<0.10150.0>]}}, %{consolidate?: true})
    (stdlib 3.13) gen_server.erl:680: :gen_server.try_dispatch/4
    (stdlib 3.13) gen_server.erl:756: :gen_server.handle_msg/6
    (stdlib 3.13) proc_lib.erl:226: :proc_lib.init_p_do_apply/3
Last message: {:ssl_closed, {:sslsocket, {:gen_tcp, #Port<0.2302>, :tls_connection, :undefined}, [#PID<0.10152.0>, #PID<0.10150.0>]}}
State: %{consolidate?: true}

Changelog

Add default function clause for handle_info

Checklist for your Pull Request (PR)

@vbaranov vbaranov force-pushed the vb-bridge-counters-handle-info-fix branch from 46197dd to 22bb48c Compare February 12, 2021 13:21
@vbaranov vbaranov force-pushed the vb-bridge-counters-handle-info-fix branch from 22bb48c to 4d0bf35 Compare February 12, 2021 13:26
@coveralls
Copy link

coveralls commented Feb 12, 2021

Pull Request Test Coverage Report for Build 4d0bf3566aabcceca521e0c9e909f1059f008bc8-PR-3623

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-2.07%) to 65.017%

Totals Coverage Status
Change from base Build 392317e258fa9d432b45f1be3492e637c8e8fbfb: -2.07%
Covered Lines: 2102
Relevant Lines: 3233

💛 - Coveralls

@vbaranov vbaranov merged commit 482c14e into master Feb 12, 2021
@vbaranov vbaranov deleted the vb-bridge-counters-handle-info-fix branch February 12, 2021 14:22
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.

None yet

2 participants