gosec is good

.golangci.yml

@@ -204,7 +204,7 @@ linters:
     - gocritic
     - gofmt
     - golint
-    - gosec
+    - gosec # Should turn back on soon
     - interfacer
     - lll
     - maligned

clientapi/routing/register.go

@@ -108,21 +108,24 @@ var (
 // remembered. If ANY parameters are supplied, the server should REPLACE all knowledge of
 // previous parameters with the ones supplied. This mean you cannot "build up" request params.
 type registerRequest struct {
-	// registration parameters
-	Password string `json:"password"`
-	Username string `json:"username"`
-	Admin    bool   `json:"admin"`
 	// user-interactive auth params
 	Auth authDict `json:"auth"`
 
-	InitialDisplayName *string `json:"initial_device_display_name"`
+	// registration parameters
-
+	Password string `json:"password"`
-	// Prevent this user from logging in
+	Username string `json:"username"`
-	InhibitLogin common.WeakBoolean `json:"inhibit_login"`
 
 	// Application Services place Type in the root of their registration
 	// request, whereas clients place it in the authDict struct.
 	Type authtypes.LoginType `json:"type"`
+
+	InitialDisplayName *string `json:"initial_device_display_name"`
+
+	// more reg params
+	Admin    bool   `json:"admin"`
+
+	// Prevent this user from logging in
+	InhibitLogin common.WeakBoolean `json:"inhibit_login"`
 }
 
 type authDict struct {