Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(2008)

Unified Diff: environs/cloudinit/cloudinit.go

Issue 6659048: environs: various: make admin-secret mandatory
Patch Set: environs: various: make admin-secret mandatory Created 5 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « [revision details] ('k') | environs/cloudinit/cloudinit_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: environs/cloudinit/cloudinit.go
=== modified file 'environs/cloudinit/cloudinit.go'
--- environs/cloudinit/cloudinit.go 2012-10-11 12:36:30 +0000
+++ environs/cloudinit/cloudinit.go 2012-10-12 01:16:46 +0000
@@ -283,6 +283,9 @@
return fmt.Errorf("entity name must match started machine")
}
}
+ if len(cfg.StateInfo.Password) == 0 {
+ return fmt.Errorf("initial password is empty")
+ }
for _, r := range cfg.StateInfo.Password {
if r == '\'' || r == '\\' || r < 32 {
return fmt.Errorf("password has disallowed characters")
« no previous file with comments | « [revision details] ('k') | environs/cloudinit/cloudinit_test.go » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 204d58d