Skip to content
This repository was archived by the owner on Jul 13, 2021. It is now read-only.

beta: https better server support, client-server port config#85

Merged
shellscape merged 4 commits intomasterfrom
next
Jul 6, 2018
Merged

beta: https better server support, client-server port config#85
shellscape merged 4 commits intomasterfrom
next

Conversation

@shellscape
Copy link
Contributor

This PR contains a:

  • bugfix
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

Clients were having trouble connecting to the WebSocket server that webpack-hot-client creates, when an https host was used.

Breaking Changes

None

Additional Info

Resolves #79. This PR would also take precedence over #80 and #83 due to the HTTPS fix needing to refactor how ports and hosts are determined.

@shellscape
Copy link
Contributor Author

/cc @DBosley @JSteunou

@shellscape shellscape changed the title feat: https better server support, client-server port config beta: https better server support, client-server port config Jul 6, 2018
@shellscape
Copy link
Contributor Author

Note that Firefox has special requirements for self-signed certs: https://github.com/webpack-contrib/webpack-hot-client/tree/next#https

@DBosley
Copy link

DBosley commented Jul 6, 2018

Looking good!

@shellscape shellscape merged commit 48f0bc2 into master Jul 6, 2018
@shellscape shellscape deleted the next branch July 6, 2018 18:10
@shellscape shellscape restored the next branch July 6, 2018 18:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add server/client configuration to port option (like with "host" option)

4 participants