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

Mikrotik HAP ACL doesn't render entire first screen below the buttons N-1948 to N-1955 #558

Closed
k1ky opened this issue Dec 6, 2022 · 11 comments · Fixed by #559
Closed

Mikrotik HAP ACL doesn't render entire first screen below the buttons N-1948 to N-1955 #558

k1ky opened this issue Dec 6, 2022 · 11 comments · Fixed by #559

Comments

@k1ky
Copy link

k1ky commented Dec 6, 2022

Some (not all) Mikrotik HAP ACL will not render the entire first GUI screen below the Buttons. All other screens appear to be fine. So far this has occurred on 3 nodes out of probably more than 15 that have been upgraded. Same behavior Google Chrome, Microsoft Edge, Palemoon 32 and Firefox. Mesh RF is disabled seems to be the key. See below.

@k1ky
Copy link
Author

k1ky commented Dec 6, 2022

image

@k1ky
Copy link
Author

k1ky commented Dec 6, 2022

@AJ6GZ
Copy link

AJ6GZ commented Dec 6, 2022

I noticed an unclosed </nobr tag on line 287 of /www/cgi-bin/status but fixing it on the affected node didn't help. Also that typo looks to have been there at least since 3.22.8.

Output html just seems to stop at this point on his node:

image

@k1ky
Copy link
Author

k1ky commented Dec 6, 2022

It appears that having Mesh RF disabled is the key. Enabling Mesh RF - even with no connections available seems to fix the issue.

@k1ky
Copy link
Author

k1ky commented Dec 6, 2022

MESH RF=Enabled - I realize this is a different node from above, but same result.
image

@AJ6GZ
Copy link

AJ6GZ commented Dec 6, 2022

Tried on a hAP with Mesh RF disabled, but with WAN/active tunnel present and the page showed properly. Also tried SXT with disabled Mesh RF and WAN only and page was ok.

@ab7pa
Copy link
Contributor

ab7pa commented Dec 6, 2022

Thank you @k1ky Tom and @AJ6GZ Ian for finding this. I made a fix for this issue and the new /www/cgi-bin/status file is available here if you want to download it and test it:
https://raw.githubusercontent.com/aredn/aredn/f7420ee0629236a0198cf013f11293bb37e56f8b/files/www/cgi-bin/status
I'd appreciate your feedback. Many thanks!

@k1ky
Copy link
Author

k1ky commented Dec 6, 2022

You betcha Steve - I may wait for the nightly fix. Maybe Ian is up for it!? One of our guys just applied and it works fine.

@dman776
Copy link
Contributor

dman776 commented Dec 6, 2022 via email

@kc2cbd
Copy link

kc2cbd commented Dec 6, 2022

That fixed the issue Steve.
Thanks

@AJ6GZ
Copy link

AJ6GZ commented Dec 6, 2022

Reproduced and fixed with file above.

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

Successfully merging a pull request may close this issue.

5 participants