CLI Relfection

- forgot to add the parsing of Duration and unit64 types (capturing the time.Duration really)
louketo · Jan 9, 2017 · 0f59394 · 0f59394
1 parent 260d2c8
commit 0f59394
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,7 +2,11 @@
 #### **2.0.2**
 
 FEATURES:
- * adding the --enable-cors-global to switch on CORs header injects into every response
+ * adding the --enable-cors-global to switch on CORs header injects into every response [#PR174](https://github.com/gambol99/keycloak-proxy/pull/174)
+
+BUGS:
+ * Fixed the time.Duration flags in the reflection code [#PR173](https://github.com/gambol99/keycloak-proxy/pull/173)
+ * Fixed the environment variable type [#PR176](https://github.com/gambol99/keycloak-proxy/pull/176)
 
 #### **2.0.1**
 

diff --git a/cli.go b/cli.go
@@ -172,9 +172,17 @@ func parseCLIOptions(cx *cli.Context, config *Config) (err error) {
 				for _, x := range cx.StringSlice(name) {
 					reflect.Append(reflect.ValueOf(config).Elem().FieldByName(field.Name), reflect.ValueOf(x))
 				}
+			case reflect.Int64:
+				switch field.Type.String() {
+				case "time.Duration":
+					reflect.ValueOf(config).Elem().FieldByName(field.Name).SetInt(int64(cx.Duration(name)))
+				default:
+					reflect.ValueOf(config).Elem().FieldByName(field.Name).SetInt(cx.Int64(name))
+				}
 			}
 		}
 	}
+	fmt.Printf("cors %s", config.CorsMaxAge.String())
 	if cx.IsSet("tag") {
 		tags, err := decodeKeyPairs(cx.StringSlice("tag"))
 		if err != nil {