Include username in unauthorized message response

hluk · Jan 26, 2023 · 75ca274 · 75ca274
1 parent 51a76da
commit 75ca274
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/resultsdb/authorization.py b/resultsdb/authorization.py
@@ -74,4 +74,6 @@ def verify_authorization(user, testcase, permissions, ldap_host, ldap_searches):
     if not any_groups_found:
         raise Unauthorized(f"Failed to find user {user} in LDAP")
 
-    raise Unauthorized(f"You are not authorized to submit a result for the test case {testcase}")
+    raise Unauthorized(
+        f"User {user} is not authorized to submit a result for the test case {testcase}"
+    )
diff --git a/testing/test_api_v3.py b/testing/test_api_v3.py
@@ -235,7 +235,9 @@ def test_api_v3_permission_denied(client, permissions):
     data = brew_build_request_data()
     r = client.post("/api/v3/results/brew-builds", json=data)
     assert r.status_code == 401, r.text
-    assert "You are not authorized to submit a result for the test case testcase1" in r.text
+    assert (
+        "User testuser1 is not authorized to submit a result for the test case testcase1" in r.text
+    )
 
 
 def test_api_v3_permission_matches_username(client, permissions):