Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: socketio/engine.io
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 6.4.1
Choose a base ref
...
head repository: socketio/engine.io
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 6.4.2
Choose a head ref
  • 7 commits
  • 7 files changed
  • 3 contributors

Commits on Apr 19, 2023

  1. fix(typings): make clientsCount public (#675)

    Related: #672
    tyilo authored Apr 19, 2023
    Configuration menu
    Copy the full SHA
    bd6d471 View commit details
    Browse the repository at this point in the history

Commits on May 1, 2023

  1. refactor: return HTTP 400 upon invalid request overlap

    In both cases, the error comes from the client as it should not send
    multiple concurrent requests, so a HTTP 4xx code is mandated.
    
    Related: #650
    darrachequesne committed May 1, 2023
    Configuration menu
    Copy the full SHA
    911d0e3 View commit details
    Browse the repository at this point in the history
  2. 1 Configuration menu
    Copy the full SHA
    9395782 View commit details
    Browse the repository at this point in the history
  3. fix(uws): prevent crash when using with middlewares

    The class used to accumulate the response headers did not expose the
    exact same API as its wrapped type, which could lead to the following
    error in some rare cases:
    
    > TypeError: Cannot read properties of undefined (reading 'end')
    >    at Polling.onDataRequest (build/transports-uws/polling.js:109:53)
    >    at Polling.onRequest (build/transports-uws/polling.js:47:18)
    >    at callback (build/userver.js:94:56)
    >    at uServer.verify (build/server.js:152:9)
    
    Related: socketio/socket.io#4643
    darrachequesne committed May 1, 2023
    Configuration menu
    Copy the full SHA
    8b22162 View commit details
    Browse the repository at this point in the history
  4. refactor(types): ensure compatibility with Express middlewares

    In order to prevent issues like:
    
    > error TS2345: Argument of type 'RequestHandler<ParamsDictionary, any, any, ParsedQs, Record<string, any>>' is not assignable to parameter of type 'Middleware'.
    >  Types of parameters 'req' and 'req' are incompatible.
    >  Type 'IncomingMessage' is missing the following properties from type 'Request<ParamsDictionary, any, any, ParsedQs, Record<string, any>>': get, header, accepts, acceptsCharsets, and 29 more.
    >
    >  io.engine.use(sessionMiddleware);
                     ~~~~~~~~~~~~~~~~~
    
    Related: socketio/socket.io#4644
    
    We could also have use the RequestHandler type from the
    @types/express-serve-static-core package, but that would add 5 new
    dependencies.
    
    See also: https://github.com/socketio/engine.io/issues/673
    darrachequesne committed May 1, 2023
    Configuration menu
    Copy the full SHA
    0141951 View commit details
    Browse the repository at this point in the history
  5. fix: prevent crash when provided with an invalid query param

    A specially crafted request could lead to the following exception:
    
    > TypeError: Cannot read properties of undefined (reading 'handlesUpgrades')
    >    at Server.onWebSocket (build/server.js:515:67)
    
    This bug was introduced in [1], released in version 5.1.0 and included
    in version 4.1.0 of the `socket.io` parent package. Older versions are
    not impacted.
    
    [1]: 7096e98
    darrachequesne committed May 1, 2023
    Configuration menu
    Copy the full SHA
    fc480b4 View commit details
    Browse the repository at this point in the history
  6. chore(release): 6.4.2

    darrachequesne committed May 1, 2023
    Configuration menu
    Copy the full SHA
    95e2153 View commit details
    Browse the repository at this point in the history
Loading