moltzaum commented on a change in pull request #2834: TO Golang: Rewrite
/api/*/users endpoints
URL: https://github.com/apache/trafficcontrol/pull/2834#discussion_r222782511
##########
File path: traffic_ops/testing/api/v14/todb.go
##########
@@ -136,12 +136,12 @@ func SetupTmusers(db *sql.DB) error {
// Creates users in different tenants
sqlStmt := `
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.Disallowed + `','` +
encryptedPassword + `','` + encryptedPassword + `', 1, 1);
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.ReadOnly + `','` +
encryptedPassword + `','` + encryptedPassword + `', 2, 1);
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.Operations + `','` +
encryptedPassword + `','` + encryptedPassword + `', 3, 1);
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.Admin + `','` +
encryptedPassword + `','` + encryptedPassword + `', 4, 1);
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.Portal + `','` +
encryptedPassword + `','` + encryptedPassword + `', 5, 1);
-INSERT INTO tm_user (username, local_passwd, confirm_local_passwd, role,
tenant_id) VALUES ('` + Config.TrafficOps.Users.Federation + `','` +
encryptedPassword + `','` + encryptedPassword + `', 6, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.Disallowed + `','` + encryptedPassword + `', 1, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.ReadOnly + `','` + encryptedPassword + `', 2, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.Operations + `','` + encryptedPassword + `', 3, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.Admin + `','` + encryptedPassword + `', 4, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.Portal + `','` + encryptedPassword + `', 5, 1);
+INSERT INTO tm_user (username, local_passwd, role, tenant_id) VALUES ('` +
Config.TrafficOps.Users.Federation + `','` + encryptedPassword + `', 6, 1);
`
Review comment:
Are you suggesting removing these lines of code entirely? (I didn't add them
in the first place)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
