Skip to content

Inconsistent casing for websocketPort option #6419

@kj

Description

@kj

What version of Remix are you using?

1.16.1

Are all your remix dependencies & dev-dependencies using the same version?

  • Yes

Steps to Reproduce

Try to set the websocket port via the future.unstable_dev.websocketPort config option.

Expected Behavior

The browser connects to the specified websocket port.

Actual Behavior

The browser still tries to connects to the default port 3001, ignoring the config.

This is because there is inconsistency in the source code. Some files refer to the config key as websocketPort (as documented in release notes), while others refer to it as webSocketPort (with an uppercase 'S' for Socket). I think this inconsistency was introduced in 1.16.1, as if memory serves it was working in 1.16.0.

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions