Skip to content

解析機能におけるヘルスチェックをOAuthで連携する場合にのみ実施するように修正#177

Open
chiku-samugari wants to merge 1 commit intoRCOSDP:developfrom
chiku-samugari:hotfix/gate-healthcheck-on-authorize-url
Open

解析機能におけるヘルスチェックをOAuthで連携する場合にのみ実施するように修正#177
chiku-samugari wants to merge 1 commit intoRCOSDP:developfrom
chiku-samugari:hotfix/gate-healthcheck-on-authorize-url

Conversation

@chiku-samugari
Copy link
Copy Markdown

  • Ticket: 58940
  • Feature flag: n/a

Purpose

解析機能においてTLJHを利用したときに「解析環境を起動する」ボタンが無効化され、解析環境を起動できないバグを修正する。

PR #159 で実装されたBinderHub/JupyterHubのヘルスチェックは、TLJHのように、OAuth2による連携を利用しないHubにおいては不要である。本PRではこのヘルスチェックをOAuth2による認証URL (authorize_url) が null でない場合にのみ行なうように修正することで、バグを修正する。

Summary of Changes

BinderHub/JupyterHubのヘルスチェックを、OAuth2による認証URLが null でない場合にのみ行なうように修正

QA Notes

Gate eager BinderHub/JupyterHub health checks on `authorize_url` being
present. When `authorize_url` is `null`, as in TLJH, no health check is
performed, so the UI remains fully functional. This avoids the false
failure caused by TLJH lacking /health and /hub/health endpoints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant