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.

Sign up for GitHub

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

Jump to bottom

Make xhr transports resilient to onmessage errors #564

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Make xhr transports resilient to onmessage errors #564

wants to merge 4 commits into from

Commits on Sep 8, 2021

  1. Make xhr transports resilient to onmessage errors 

    Fixes sockjs#563
    @jcheng5
    jcheng5 authored Sep 8, 2021
    Configuration menu
    Copy the full SHA
    767b921 View commit details
    Browse the repository at this point in the history

Commits on Sep 22, 2021

  1. Add unit test for xhr unhandled errors fix

    @jcheng5
    jcheng5 committed Sep 22, 2021
    Configuration menu
    Copy the full SHA
    082e6dd View commit details
    Browse the repository at this point in the history

  2. Allow jsonp receiver to survive errors in message handlers 

    Without this fix, once a jsonp-polling message handler throws an
uncaught exception, the receiver stops polling.
    @jcheng5
    jcheng5 committed Sep 22, 2021
    Configuration menu
    Copy the full SHA
    e10d608 View commit details
    Browse the repository at this point in the history

Commits on Sep 23, 2021

  1. Use explicit try/finally instead of careful statement ordering

    @jcheng5
    jcheng5 committed Sep 23, 2021
    Configuration menu
    Copy the full SHA
    d4e5c2f View commit details
    Browse the repository at this point in the history