huge derp, forgot to actually test in browser

paulbellamy · Jan 25, 2013 · bfcdaa0 · bfcdaa0
1 parent 7cdc2fb
commit bfcdaa0
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/basic_auth.go b/basic_auth.go
@@ -7,7 +7,7 @@ import (
 )
 
 func defaultFailure() (Status, Headers, Body) {
-	return 403, Headers{"Content-Type": []string{"text/html"}}, Body("Access Denied.") // default failure page
+	return 401, Headers{"WWW-Authenticate": []string{"Basic realm=\"Basic\""}, "Content-Type": []string{"text/html"}}, Body("Access Denied.") // default failure page
 }
 
 func BasicAuth(auth func(string, string, Request, error) bool, failure func(Env) (Status, Headers, Body)) Middleware {

diff --git a/basic_auth_test.go b/basic_auth_test.go
@@ -80,7 +80,8 @@ func TestFailByDefault(t *testing.T) {
 		t.Error(err)
 	}
 
-	if status != 403 {
+	// TODO test header
+	if status != 401 {
 		t.Error("Request did not succeed, expected status 403, got:", status)
 	}
 }