[lgwebos] Console command to show the access key (openhab#7801)

* [lgwebos] Console command to show the access key Access key no more logged * Fix in README Signed-off-by: Laurent Garnier <[email protected]>
markus7017 · Sep 18, 2020 · e2da78e · e2da78e
1 parent 91f0d33
commit e2da78e
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 3 deletions.
diff --git a/bundles/org.openhab.binding.lgwebos/README.md b/bundles/org.openhab.binding.lgwebos/README.md
@@ -97,17 +97,19 @@ A sample HABPanel remote control widget can be found [in this github repository.
 
 ## Console Commands
 
-The binding provides two specific commands you can use in the console.
+The binding provides a few commands you can use in the console.
 Enter the command `lgwebos` to get the usage.
 
 ```
 openhab> lgwebos
 Usage: smarthome:lgwebos <thingUID> applications - list applications
 Usage: smarthome:lgwebos <thingUID> channels - list channels
+Usage: smarthome:lgwebos <thingUID> accesskey - show the access key
 ```
 
 The command `applications` reports in the console the list of all applications with their id and name.
 The command `channels` reports in the console the list of all channels with their id, number and name.
+The command `accesskey` reports in the console the access key used to connect to your TV.
 
 ## Example
 

diff --git a/...s/src/main/java/org/openhab/binding/lgwebos/internal/console/LGWebOSCommandExtension.java b/...s/src/main/java/org/openhab/binding/lgwebos/internal/console/LGWebOSCommandExtension.java
@@ -39,6 +39,7 @@ public class LGWebOSCommandExtension extends AbstractConsoleCommandExtension {
 
     private static final String APPLICATIONS = "applications";
     private static final String CHANNELS = "channels";
+    private static final String ACCESS_KEY = "accesskey";
 
     private final ThingRegistry thingRegistry;
 
@@ -72,6 +73,9 @@ public void execute(String[] args, Console console) {
                     case CHANNELS:
                         handler.reportChannels().forEach(console::println);
                         break;
+                    case ACCESS_KEY:
+                        console.println("Your access key is " + handler.getKey());
+                        break;
                     default:
                         printUsage(console);
                         break;
@@ -85,6 +89,7 @@ public void execute(String[] args, Console console) {
     @Override
     public List<String> getUsages() {
         return Arrays.asList(new String[] { buildCommandUsage("<thingUID> " + APPLICATIONS, "list applications"),
-                buildCommandUsage("<thingUID> " + CHANNELS, "list channels") });
+                buildCommandUsage("<thingUID> " + CHANNELS, "list channels"),
+                buildCommandUsage("<thingUID> " + ACCESS_KEY, "show the access key") });
     }
 }
diff --git a/...ng.lgwebos/src/main/java/org/openhab/binding/lgwebos/internal/handler/LGWebOSHandler.java b/...ng.lgwebos/src/main/java/org/openhab/binding/lgwebos/internal/handler/LGWebOSHandler.java
@@ -256,7 +256,7 @@ public String getKey() {
     @Override
     public void storeKey(@Nullable String key) {
         if (!getKey().equals(key)) {
-            logger.info("Store {} as access Key in the thing configuration", key);
+            logger.debug("Store new access Key in the thing configuration");
             // store it current configuration and avoiding complete re-initialization via handleConfigurationUpdate
             getLGWebOSConfig().key = key;