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

Problem with WAN Detection OPENWRT #5

Closed
drewgarth opened this issue May 3, 2015 · 4 comments
Closed

Problem with WAN Detection OPENWRT #5

drewgarth opened this issue May 3, 2015 · 4 comments

Comments

@drewgarth
Copy link

Thanks for updating this script,
Compared to the Master branch, the AWK branch seems to have a problem with the way it writes out the detected WAN into the DB file.
For debugging purposes i just ran the Master branch and then the AWK branch with the following
MASTER
(WAN),NA,eth1,46508,22835,0,0,69343,04-05-2015_08:53:33,04-05-2015_08:53:34

AWK
eth1,NA,eth1,32,1672,,,1704,04-05-2015_09:47:25,04-05-2015_09:47:25

The extra three columns result in the quotes being in the wrong placed in the array when published, and the entire table then fails to load.
Array in Javascript in AWK generated HTML
new Array("eth1",
32.000000,1672,1704,04-05-2015_09:47:25,04-05-2015_09:47:25,"",""),

Hope this helps

cheers

@pyrovski
Copy link
Owner

pyrovski commented May 4, 2015

I think I fixed it. Would you mind testing the latest code in the awk branch? You'll have to delete your old data file.

@drewgarth
Copy link
Author

Thanks for that. It fixed it for me.

One other question, per your instructions you haven't specified the need for a cron job to keep running setup and therefore refresh the connected MAC/hostnames? Is this correct that i dont need to continually run setup in cron?

thanks for your work

@drewgarth drewgarth reopened this May 4, 2015
@drewgarth
Copy link
Author

Oops, hit the wrong button, while i think it can be closed, I will allow you to do that! :)

@pyrovski
Copy link
Owner

pyrovski commented May 4, 2015

You should only need to run setup once after booting the machine, unless you change the list of interfaces to monitor. New hosts are automatically recognized in the update command.

@pyrovski pyrovski closed this as completed May 4, 2015
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