Feature #411
closedtypo in config file did not show up in a config-test
Description
simple-vhost.default-root = "grisu.home.kneschke.de"
did not generate a warning during the config test (note, root should read host)
Howver, when I started the daemon, I got a warning about an invalid config key (or some such)
simple-vhost.default-root = "grisu.home.kneschke.de"
nero# lighttpd -t -f lighttpd.conf
Syntax OK
nero# ./rc.d/lighttpd.sh restart
Stopping lighttpd.
Starting lighttpd.
nero# 2005-12-09 16:24:57: (server.c.693) WARNING: unknown config-key: simple-vhost.default-root (ignored)
Perhaps keys are not checked during startup?
lighttpd-1.4.8 (ssl) - a light and fast webserver
Build-Date: Dec 9 2005 10:54:27
-- eydaimon
Updated by stbuehler about 16 years ago
- Status changed from New to Fixed
- Resolution set to invalid
As some modules do more than just checking the syntax while loading the config, we won't change that. And the syntax of your file is ok...
Updated by Anonymous about 16 years ago
- Status changed from Fixed to Need Feedback
- Resolution deleted (
invalid)
Then I suggest that:
-t test the config-file, and exitis changed to read:
-t test config-file for syntax errors, and exitUpdated by gstrauss over 8 years ago
- Description updated (diff)
@stbuehler, what would you think of extending the -t flag so that if provided more than once on command line (e.g. "-tt" or "-t -t"), that config processing continues, loads all modules, and exits after checking for unused/deprecated config keys?
Anything that with potentially destructive behavior would have to be skipped, such as overwriting pidfile. daemonizing would be disabled to keep process in foreground and stderr attached. network_init() would need to be passed an extra flags to indicate "check_only" so as not to attempt to bind() to potentially in-use addresses. plugin hooks 'init' and 'set_defaults', called by plugins_call_init() and plugins_call_set_defaults(), would need to take an "check-only" flag, too, so this would be a breaking change to third-party modules, similar to the change in 1.4.38 in r3049 which extended function signature of config_insert_values_internal(). For most modules, they would still perform their complete setup routine, but there might be a few which need to skip a few actions and the "check-only" flag would communicate that.
Updated by gstrauss over 8 years ago
Instead of changing the function signatures, a much less invasive way to make the change would be to add a flag 'preflight_check' to server_config (srv->srvconf) (e.g. right along with 'dont_daemonize' member)
I looked through the code and these places would need to check for srv->srvconf.preflight_check:
server.c:main()
skip pidfile creation
skip daemonizing
skip log_error_open()
network.c:network_server_init()
skip bind() to sockets
Skipping error log and access log opens is not technically required for a startup test, but probably still a good idea to avoid triggering custom behavior by piped loggers
server.c:main()
skip log_error_open()
mod_accesslog.c:log_access_open()
skip open_log_file_or_pipe()
@stbuehler, think you would accept a patch for this if I created one?
Updated by stbuehler over 8 years ago
- Tracker changed from Bug to Feature
- Assignee deleted (
jan)
@gstrauss: No promises on acceptance or review time :) But I'm open to the idea.
Updated by gstrauss over 8 years ago
- Status changed from Need Feedback to Patch Pending
Submitted pull request https://github.com/lighttpd/lighttpd1.4/pull/44
Updated by gstrauss over 8 years ago
Turned out not to be a very large change:
https://github.com/lighttpd/lighttpd1.4/pull/44
src/base.h | 1 + src/mod_accesslog.c | 2 ++ src/network.c | 12 ++++++++++-- src/server.c | 23 ++++++++++++++++++++--- 4 files changed, 33 insertions(+), 5 deletions(-)
Updated by stbuehler over 8 years ago
- Status changed from Patch Pending to Fixed
- % Done changed from 0 to 100
Applied in changeset r3130.
Also available in: Atom