Skip to content

Commit d7d2d77

Browse files
Return 403 instead of 401 when required permissions are missing (#9)
1 parent c26a8dd commit d7d2d77

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

router/src/main/kotlin/io/moia/router/RequestHandler.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ abstract class RequestHandler : RequestHandler<APIGatewayProxyRequestEvent, APIG
3333
val matchedAcceptType = routerFunction.requestPredicate.matchedAcceptType(input.acceptedMediaTypes())
3434
?: MediaType.parse(router.defaultContentType)
3535
if (!permissionHandlerSupplier()(input).hasAnyRequiredPermission(routerFunction.requestPredicate.requiredPermissions))
36-
return createApiExceptionErrorResponse(matchedAcceptType, input, ApiException("unauthorized", "UNAUTHORIZED", 401))
36+
return createApiExceptionErrorResponse(matchedAcceptType, input, ApiException("missing permissions", "MISSING_PERMISSIONS", 403))
3737

3838
val handler: HandlerFunction<Any, Any> = routerFunction.handler
3939
return try {

router/src/test/kotlin/io/moia/router/RequestHandlerTest.kt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,7 @@ class RequestHandlerTest {
244244
)), mockk()
245245
)
246246

247-
assert(response.statusCode).isEqualTo(401)
247+
assert(response.statusCode).isEqualTo(403)
248248
}
249249

250250
@Test

0 commit comments

Comments
 (0)