Fix bearer (#28)

Chris Loper · web-flow · commit fd40ce50d7f7 · 2022-12-07T15:20:19.000-06:00
* fix: make sure Bearer is first value in Authorization header

* chore: version bump
diff --git a/authentication.go b/authentication.go
@@ -23,7 +23,7 @@ func (a *App) AuthenticateRequest(r *http.Request) (string, error) {
 // returns the userID (string) on success, error on failure
 func (a *App) AuthenticateRequestWithHeader(r *http.Request) (string, error) {
 	authHeaderFields := strings.Fields(r.Header.Get("Authorization"))
-	if len(authHeaderFields) != 2 || authHeaderFields[0] == "Bearer" {
+	if len(authHeaderFields) != 2 || authHeaderFields[0] != "Bearer" {
 		return "", Error{Message: "missing authentication token: expected \"Bearer\" header"}
 	}
 
diff --git a/version.txt b/version.txt
@@ -1 +1 @@
-v1.3.1
+v1.3.2

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ func (a App) AuthenticateRequest(r http.Request) (string, error) {`
`23`	`23`	`// returns the userID (string) on success, error on failure`
`24`	`24`	`func (a App) AuthenticateRequestWithHeader(r http.Request) (string, error) {`
`25`	`25`	`authHeaderFields := strings.Fields(r.Header.Get("Authorization"))`
`26`		`- if len(authHeaderFields) != 2 \|\| authHeaderFields[0] == "Bearer" {`
	`26`	`+ if len(authHeaderFields) != 2 \|\| authHeaderFields[0] != "Bearer" {`
`27`	`27`	`return "", Error{Message: "missing authentication token: expected \"Bearer\" header"}`
`28`	`28`	`}`
`29`	`29`