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.

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

EmonESP unable to set port of EmonCMS server #85

Open
mmallozzi opened this issue Feb 16, 2022 · 2 comments
Open

EmonESP unable to set port of EmonCMS server #85

mmallozzi opened this issue Feb 16, 2022 · 2 comments

Comments

@mmallozzi
Copy link

When I try to enter a custom port in the traditional way for the EmonCMS server, such as 192.168.1.3:27015, the code seems to treat the entire string as the hostname field, and uses hardcoded port 80 to try to connect to. So ultimately it tries to connect to 192.168.1.3:27015:80 and spits out errors accordingly.

This might just require pulling in a fix from upstream: openenergymonitor/EmonESP#102

Also, what are the changes in the EmonESP fork? Is it possible to just use the mainstream EmonESP?

@mmallozzi
Copy link
Author

For anyone looking for a temporary workaround, I modified emoncms.cpp in the emoncms_publish method to pass my desired port (27015) as the optional third parameter to get_http in the "Plain old HTTP" branch. For me (puled down master a couple days ago) that was on line 63.

@CircuitSetup
Copy link
Owner

Thanks for following up with this. I'll take a look at what needs to be changed to fix it.

The version for the 6 channel meter adds calibration fields, and the additional current channels depending on how many add-on boards are used, if any. Unfortunately, the original EmonESP would not work properly with the 6 channel meter.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants