Skip to content

Commit d82140c

Browse files
author
Jiangtao Li
authored
alts: change errr status to permission deny if alts context not found (#6562)
1 parent f07b79b commit d82140c

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

alts/src/main/java/io/grpc/alts/AuthorizationUtil.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public static Status clientAuthorizationCheck(
3737
AltsAuthContext altsContext =
3838
(AltsAuthContext) call.getAttributes().get(AltsProtocolNegotiator.AUTH_CONTEXT_KEY);
3939
if (altsContext == null) {
40-
return Status.NOT_FOUND.withDescription("Peer ALTS AuthContext not found");
40+
return Status.PERMISSION_DENIED.withDescription("Peer ALTS AuthContext not found");
4141
}
4242
if (expectedServiceAccounts.contains(altsContext.getPeerServiceAccount())) {
4343
return Status.OK;

alts/src/test/java/io/grpc/alts/AuthorizationUtilTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public void altsAuthorizationCheck() throws Exception {
4242
Status status =
4343
AuthorizationUtil.clientAuthorizationCheck(
4444
new FakeServerCall(null), Lists.newArrayList("Alice"));
45-
assertThat(status.getCode()).isEqualTo(Status.Code.NOT_FOUND);
45+
assertThat(status.getCode()).isEqualTo(Status.Code.PERMISSION_DENIED);
4646
assertThat(status.getDescription()).startsWith("Peer ALTS AuthContext not found");
4747
status =
4848
AuthorizationUtil.clientAuthorizationCheck(

0 commit comments

Comments
 (0)