-
Notifications
You must be signed in to change notification settings - Fork 26
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
Change port state using SAVE button not working on IO control page #67
Comments
Ps . eg. http://192.168.0.17/03 and /04 method is working |
I will roll back a test device and check this in case I missed something, but I know this is working OK in the current version (20210413 1254). |
I just compiled 20210413 1254 - not ok, and roll back to 20210221 1826 - ok |
Since you indicated your are compiling your own, have you tried using the pre-compiled .stp and .sx files? That might give a clue whether the problem is due to a difference in compiler environment. |
yours .sx and .stp also not work (windows 10 64bit, Edge 90.0.818.56 - latest and Chrome 90.0.4430.93 ) |
Well, this is really odd. No one else is reporting an issue like this. Are you using REST and GUI commands at the same time? A bug I just fixed (but haven't released yet) addresses an issue where using REST commands from the GUI while also trying to use GUI radio buttons can cause confusing results. But if you ONLY use the IOControl page ON/OFF radio buttons (no REST commands at the same time) it should work just fine. Can you tell me more about what steps you take? And maybe send screen shots of your Configuration and IOControl pages to my email address [email protected] |
The main anomaly I see is that the Gateway address should match your IP address. Since you are using 192.168.0.17 as your Network Module address the Gateway address should be 192.168.0.1. If you can give that a try please let me know the result. |
I should add that in my understanding of routers it shouldn't matter that the Gateway address upper octets don't match ... as the Gateway (or default router) address shouldn't be used unless you are expecting routing to the internet (not just your intranet). BUT - there was one user where this made a difference. So, I suspect it may be router dependent. |
"Gateway address should match your IP address" - 17 is no gate to internet, and if I routing localy eg. form 192.168.0.60 to 192.168.0.17 this should be irrelevant if the gate is set to 192.168.0.1 eg. my computer have also this gate and there is no problem with connection to other local ip addresses. ps. I have a wrong entry there, it should be 192.168.0.1 not 1.1 but in this case it does not matter |
Remember that in the older version it works with these settings :) |
I am facing with exactly the same issue: Version 20210221 1826 is working fine with exactly the same deploy process and same settings = No changes in MAC or IP address. Only enabling some pins as input and output. |
This is really a puzzle. It is working for me here, I'm using Firefox and Chrome for testing. I'm trying to think of how I can alter my testing to try to get this issue to show up ... |
I just got a freeze reproduction with Edge ... so now I have something to start tracking down. |
I am not sure what is happening, but I can see the EEPROM is being overwritten with garbage when I attempt a SAVE with Edge. Investigating. |
Tested the latest firmware on Win 10 x64 with fresh instal off Edge Versão 90.0.818.56 (64 bits) no problem the port´s change fine the retain work´s, could´t it bee a cache issue? |
Maybe ... but right now I don't think so. I just upgraded Chrome to the latest and now it is showing the issue also. So, it has something to do with the latest versions of Chromium (used in both Chrome and Edge). But having said that, I don't think the browser is at fault. I think the latest versions of these browsers have exposed some pointer issue in my POST processing routine. I now have a test routine that is repeatable, and I see that the EEPROM is being overwritten with an incoming string from these browsers. That should not be possible unless my code has a defect. So, I am pursuing that. |
By the way, if you hit the defect you can reset to factory defaults (with the reset button) and re-enter your settings. But clicking Save on IOControl will just cause the problem again. I am not sure why Save does not cause a problem in the Configuration page ... it runs the same code but parses different POST variables. So that alone is a big clue. |
OK - found it and fixed it. The problem was provoked by a change in the pre-amble in the Chromium based browsers, but the defect was in my POST processing code. I wasn't properly handling the case where a packet ended during the first POST component, and that POST component was continued in the next packet. The browser pre-amble change shifted the POST data just enough to make this packet boundary crossing occur in just the right place. I will get a new build out within an hour. |
Fixed in 20210511 1317 |
Just published Release 20210511 1317 with the fix. |
I will check it at the evening |
Works fine. Thanks |
Code Revision 20210221 1826 - is ok, next version not (hangs if I press this button)
The text was updated successfully, but these errors were encountered: