Skip to content

Commit

Permalink
Fix 404 on SSO callback handler (#4924)
Browse files Browse the repository at this point in the history
Browsers would attempt to request `favicon.ico` which fail all request chains, leading to platform fallback to `BuiltInWebServer`/`DefaultWebServerPathHandler`.

This would be fine, except the platform webserver respects the browser's `Connection: keep-alive` flag and keeps the Netty channel open, and subsequent requests to `.../oauthCallback/...` get handled by `DefaultWebServerPathHandler` and always return 404.
  • Loading branch information
rli authored Oct 1, 2024
1 parent 7ff4f29 commit 21fac72
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -0,0 +1,4 @@
{
"type" : "bugfix",
"description" : "Fix issue where multiple SSO login attempts in a short time result in 404"
}
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
<title>AWS Authentication</title>
<meta name="viewport" content="width=device-width, initial-scale=1" />
<link rel="stylesheet" type="text/css" media="screen" href="auth.css" />
<link rel="icon" href="data:;base64,iVBORw0KGgo=">
</head>

<body>
Expand Down

0 comments on commit 21fac72

Please sign in to comment.