Merge pull request #55 from seomoz/nested_policy_fix

Nested policy fix
nemosupremo · May 3, 2018 · 7b7ead6 · 7b7ead6
2 parents ec6e513 + 920121b
commit 7b7ead6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/README.md b/README.md
@@ -63,6 +63,8 @@ VGM also supports the client environment variables used by vault such as, `VAULT
 
 `SEAL_HTTP_STATUS` | `-seal-http-status` - *Default: `200`* - Configures HTTP Status Code to be returned when querying /status.json.  By default uses 200 in both cases, but can be configured to return 429, for example, if the status is sealed.
 
+`DEFAULT_RENEWABLE` | `-default-renewable-tokens` - *Default: `true`* - The default value for renewable on tokens created.
+
 ### Vault Startup Authorization Methods
 
 `VAULT_TOKEN` - Vault authorization token to make requests with.

diff --git a/policy_nested.go b/policy_nested.go
@@ -56,6 +56,9 @@ func (p policies) loadNestedPolicies(authToken string) error {
 
 	/* Copy tempPolicies to policies */
 	for k,v := range tempPolicies{
+		if v.Renewable == nil {
+			v.Renewable = &config.DefaultRenewable
+		}
 		p[k] = v
 	}