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

Client Controls Fixes #194

Closed
wants to merge 4 commits into from
Closed

Client Controls Fixes #194

wants to merge 4 commits into from

Conversation

areynold
Copy link
Collaborator

@areynold areynold commented Mar 6, 2014

Replaces #185.

Fixes #184, #189

To test:

  1. In your browser, go to Admin > Client Controls and change the Time Until Welcome Page is Shown to 1 hour.
  2. "Seconds" should not be an available option.
  3. Click Save, then Save & Apply.
  4. SSH in and run ndsctl status. If you see actual nodogsplash data, the fix worked. If the fix didn't work you will see ndsctl: nodogsplash probably not started (Error: Connection refused).
  5. Open /etc/nodogsplash/nodogsplash.conf. ClientIdleTimeout and ClientForceTimeout should each be set to 60.
  6. In your browser, set Time Until Welcome Page to 1 day. ClientIdleTimeout/ClientForceTimeout should be 1440.

@areynold areynold added this to the 1.1 milestone Mar 6, 2014
@jheretic jheretic self-assigned this Mar 6, 2014
@jheretic
Copy link
Member

jheretic commented Mar 6, 2014

The correct values seem to be generated in nodogsplash.conf, but there seem to be several other problems. One is that running /etc/init.d/nodogsplash restart actually often causes nodogsplash to crash if it's already running. Also, committing values to its LuCI page also causes errors similar to this:

[3][Thu Mar 6 21:47:34 2014]6971 Can't find target ndsINC in mangle table

shortly before nodogsplash crashes.

@dismantl
Copy link
Contributor

dismantl commented Mar 7, 2014

Regarding NDS crashing when running the init.d restart command, we might want to modify the init script. The problem with restarting is that it tries to start a new NDS process before the old one is finished shutting down. I've only had success with /etc/init.d/nodogsplash stop; sleep 2; /etc/init.d/nodogsplash start or something similar.

@jheretic
Copy link
Member

This has been superseded by opentechinstitute/luci-commotion-splash#26 and related pull requests.

@jheretic jheretic closed this Mar 10, 2014
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 this pull request may close these issues.

Nodogsplash does not restart after changing lease time
3 participants