load environment variables by default

3 files changed, 21 insertions, 13 deletions

diff --git a/README b/README
index 7475c58..20ef34d 100644
--- a/README
+++ b/README
@@ -17,17 +17,21 @@ The tools can be configured in three ways:
 1. Commandline options
    Use --help to get an overview
 
-2. User configuration file
+2. Environment variables
+
+3. User configuration file
    ~/.config/linksys-wrt3g-tools
 
-3. System configuration file
+4. System configuration file
    /etc/linksys-wrt3g-tools
 
 The configuration files are normal ini-style files with key=value
-lines. Supported configuration options in the files are
+lines.
+Supported configuration options in the files, and environment variables, are
  - host
  - user
  - password
+(lowercase)
 
 
 Dependencies
diff --git a/Wrt3g/Config.php b/Wrt3g/Config.php
index a8232df..5036f7d 100644
--- a/Wrt3g/Config.php
+++ b/Wrt3g/Config.php
@@ -88,9 +88,20 @@ class Wrt3g_Config
         if ($file) {
             $this->loadFromFile($file);
         }
+        //environment variables: we use lowercase options, which should
+        // not collide with the uppercase env variables like USER
+        $this->loadOptions($_SERVER);
+        //commandline options
         $this->loadOptions($options);
 
-        //commandline options
+        $this->logger->log(
+            sprintf(
+                "Configuration: %s:%s@%s",
+                $this->user, $this->password, $this->host
+            ),
+            2
+        );
+
         if (isset($options['save'])) {
             $this->save($file);
         }
diff --git a/scripts/munin.php b/scripts/munin.php
index dc7d050..0722035 100755
--- a/scripts/munin.php
+++ b/scripts/munin.php
@@ -15,16 +15,9 @@
  */
 require_once 'Wrt3g.php';
 
-$options = array();
-foreach (array('host', 'user', 'password') as $variable) {
-    if (isset($_SERVER[$variable])) {
-        $options[$variable] = $_SERVER[$variable];
-    }
-}
-
 if (isset($argv[1]) && $argv[1] == 'autoconf') {
     $router = new Wrt3g();
-    $router->loadConfig($options);
+    $router->loadConfig();
     if ($router->config->host === null) {
         echo "no (no host configured)\n";
     } else {
@@ -81,7 +74,7 @@ TXT;
 
 try {
     $router = new Wrt3g();
-    $router->loadConfig($options);
+    $router->loadConfig();
 
     $arStatus = $router->getConnectionStatus();