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

[#1551] Handle external exits of processes #1514

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Apr 23, 2021

  1. [1551] Handle external exits of processes

    Adds handler to capture external exits of processes, and tests to verify
    this is indeed happening.
    
    Problem: I don't know how to silence crash reports in this case. When I
    run the tests, I can see the crash reports I added, but I would like to
    silence them in tests. ct_helper:ignore() doesn't seem to cut it the way
    I use it, which is probably wrong. I see on the console:
    
    Testing ninenines.cowboy.metrics_SUITE: Starting test, 96 test cases
    
    =ERROR REPORT==== 23-Apr-2021::11:22:09 ===
    Ranch listener http, connection process <0.266.0>, stream 1 had its
    request process <0.275.0> exit with reason external_exit
    
    which we probably want to silence but I'm not sure how. Help?
    hubertlepicki committed Apr 23, 2021
    Configuration menu
    Copy the full SHA
    dbc02ed View commit details
    Browse the repository at this point in the history