Fix interface env var type

Author: maggie44

README.md

@@ -59,7 +59,7 @@ If you need to disable this feature to allow the LED to be used for other purpos
 
 ## Endpoints
 ### http://your-device:9090/v1/connect
-Connect to a nearby Wi-Fi access point.
+Connect to a nearby Wi-Fi access point. Once connected the device will automatically connect to the same network on next boot until you call the `/forget` endpoint. 
 
 #### POST
 ````

docker-compose.yml

@@ -17,7 +17,7 @@ services:
       #PWC_AC_PASSWORD: "your-password" # Optional. Must be 8 characters or more.
 
       ## Wi-Fi Interface ##
-      # PWC_INTERFACE: 'wlan0' # When disabled interface is auto detected
+      # PWC_INTERFACE: "wlan0" # When disabled interface is auto detected
 
       ## Enable/Disable LED interaction ##
       PWC_LED: "ON"

src/common/wifi.py

@@ -121,12 +121,9 @@ def connect(conn_type=config.type_hotspot,
         Pnm.Settings.AddConnection(conn_dict)
         logger.info(f"Adding connection of type {conn_type}")
 
-        # Save the wi-fi device object to a variable
-        dev = get_device()
-
         # Connect
         Pnm.NetworkManager.ActivateConnection(get_connection_id(),
-                                              dev,
+                                              get_device(),
                                               "/")
 
         # If not a hotspot, log the connection SSID being attempted
@@ -253,12 +250,9 @@ def list_access_points():
     logger.debug('Fetching Wi-Fi networks.')
 
     try:
-        # Fetch dictionary of devices
-        dev = get_device()
-
         # For each wi-fi connection in range, identify it's details
         compiled_ssids = [analyse_access_point(ap)
-                          for ap in dev.GetAccessPoints()]
+                          for ap in get_device().GetAccessPoints()]
     except Exception:
         logger.exception('Failed listing access points.')
         raise WifiNetworkManagerError

src/config.py

@@ -1,4 +1,3 @@
-import json
 import os
 
 # Set default Wi-Fi SSID.
@@ -39,7 +38,7 @@
 
 # Set default interface
 if "PWC_INTERFACE" in os.environ and \
-        json.loads(os.environ['PWC_INTERFACE']) is not False:
+        os.environ['PWC_INTERFACE'].lower() != 'false':
     interface = os.environ['PWC_INTERFACE']
 else:
     interface = False