Merge pull request #451 from influxdata/feature/ga-remove-pw-todos

Remove password in cleartext TODOs
pull/10616/head
Chris Goller 2016-11-10 12:26:53 -06:00 committed by GitHub
commit 4bd305b232
2 changed files with 5 additions and 5 deletions

View File

@ -72,7 +72,7 @@ type Source struct {
Name string `json:"name"` // Name is the user-defined name for the source
Type string `json:"type,omitempty"` // Type specifies which kinds of source (enterprise vs oss)
Username string `json:"username,omitempty"` // Username is the username to connect to the source
Password string `json:"password,omitempty"` // Password is in CLEARTEXT // TODO: fixme
Password string `json:"password,omitempty"` // Password is in CLEARTEXT
URL string `json:"url"` // URL are the connections to the source
Default bool `json:"default"` // Default specifies the default source for the application
}
@ -166,7 +166,7 @@ type Server struct {
SrcID int // SrcID of the data source
Name string // Name is the user-defined name for the server
Username string // Username is the username to connect to the server
Password string // Password is in CLEARTEXT // TODO: FIXME
Password string // Password is in CLEARTEXT
URL string // URL are the connections to the server
}

View File

@ -1385,11 +1385,11 @@
},
"username": {
"type": "string",
"description": "Username for authentication to kapacitor"
"description": "Username for authentication to kapacitor."
},
"password": {
"type": "string",
"description": "Password in cleartext!"
"description": "Password is in cleartext."
},
"url": {
"type": "string",
@ -1470,7 +1470,7 @@
},
"password": {
"type": "string",
"description": "Password in cleartext!"
"description": "Password is in cleartext."
},
"url": {
"type": "string",