Skip to content

websocket port #1767

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

Closed
kiralikbeyin opened this issue Jun 9, 2019 · 3 comments
Closed

websocket port #1767

kiralikbeyin opened this issue Jun 9, 2019 · 3 comments
Labels
duplicate This issue or pull request already exists question

Comments

@kiralikbeyin
Copy link

I can get half of http server data but i cant get WS datas from remote.

If i connect to my device which is forward to ...duckdns.org with the same wifi network everything is ok.

Maybe i have to open more port in my router for ws?
(using 80 as default)

@mcspr
Copy link
Collaborator

mcspr commented Jun 11, 2019

Does this sound like #1610 ? If yes, see the lwip patch comments there and the attached PR. Or you can try using recent Core releases instead of the 2.3.0

In the future, please write out the version you are using (release bin, self-built) and if it is not from releases - which Core version was used and what build method was used (PlatformIO, ArduinoIDE, Makefile etc.). As it was specified in the template.
It was an annoying bug, so it was a simple guess to make.

@kiralikbeyin
Copy link
Author

I am now using 2.5.2 and ws works fine for remote. (before 2.3.0 which makes trouble) Thanks @mcspr

Is there any disabilty using 2.5.2?

@mcspr
Copy link
Collaborator

mcspr commented Jun 13, 2019

Not really, besides total firmware size.

You may experience some issues regarding active async webserver and doing anything with wificlient (per #1716 (comment) and #1620), as I've seen some weird timeout problems there. It is solved by scheduler patches in current esp8266/Arduino master, you can pick those patches individually if that becomes a problem.

@mcspr mcspr closed this as completed Jun 13, 2019
@mcspr mcspr added the duplicate This issue or pull request already exists label Jun 13, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists question
Projects
None yet
Development

No branches or pull requests

2 participants