Invert if to the more consistent style

cristaloleg · nhooyr · commit 0e007607e7d5 · 2019-09-26T11:19:45.000-05:00
diff --git a/handshake.go b/handshake.go
@@ -221,10 +221,10 @@ func authenticateOrigin(r *http.Request) error {
 	if err != nil {
 		return fmt.Errorf("failed to parse Origin header %q: %w", origin, err)
 	}
-	if strings.EqualFold(u.Host, r.Host) {
-		return nil
+	if !strings.EqualFold(u.Host, r.Host) {
+		return fmt.Errorf("request Origin %q is not authorized for Host %q", origin, r.Host)
 	}
-	return fmt.Errorf("request Origin %q is not authorized for Host %q", origin, r.Host)
+	return nil
 }
 
 // DialOptions represents the options available to pass to Dial.

Original file line number	Diff line number	Diff line change
`@@ -221,10 +221,10 @@ func authenticateOrigin(r *http.Request) error {`
`221`	`221`	`if err != nil {`
`222`	`222`	`return fmt.Errorf("failed to parse Origin header %q: %w", origin, err)`
`223`	`223`	`}`
`224`		`- if strings.EqualFold(u.Host, r.Host) {`
`225`		`- return nil`
	`224`	`+ if !strings.EqualFold(u.Host, r.Host) {`
	`225`	`+ return fmt.Errorf("request Origin %q is not authorized for Host %q", origin, r.Host)`
`226`	`226`	`}`
`227`		`- return fmt.Errorf("request Origin %q is not authorized for Host %q", origin, r.Host)`
	`227`	`+ return nil`
`228`	`228`	`}`
`229`	`229`
`230`	`230`	`// DialOptions represents the options available to pass to Dial.`