Merge pull request #306 from fuller-inc/sanitize-user-input

sanitize user inputs
fuller-inc · Apr 15, 2022 · c25218f · c25218f
2 parents 08f16f1 + df01202
commit c25218f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/provider/assume-role/assume-role.go b/provider/assume-role/assume-role.go
@@ -176,7 +176,7 @@ func (h *Handler) handle(ctx context.Context, req *requestBody) (*responseBody,
 }
 
 func (h *Handler) handleError(w http.ResponseWriter, r *http.Request, err error) {
-	log.Println(err)
+	log.Printf("error: %v", err)
 	status := http.StatusInternalServerError
 	var body *errorResponseBody
 

diff --git a/provider/assume-role/github/github.go b/provider/assume-role/github/github.go
@@ -118,7 +118,7 @@ func canonicalURL(rawurl string) (string, error) {
 	case "":
 		u.Scheme = "http"
 	default:
-		return "", fmt.Errorf("unknown scheme: %s", u.Scheme)
+		return "", fmt.Errorf("unknown scheme: %q", u.Scheme)
 	}
 	if port == defaultPort {
 		port = ""