Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable HTTP proxy support #89

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Enable HTTP proxy support #89

wants to merge 2 commits into from

Conversation

francoismichel
Copy link
Owner

@francoismichel francoismichel commented Jan 2, 2024

Right now, HTTP proxies cannot be used as privkey and OIDC authentication explicitly sign the conversation ID derived by TLS exporters, binding the SSH3 conversation to the TLS session.

This PR adds support for proxies (cf #67, and #44 to a lesser extent). If explicitly set by the user, the conversation ID is removed from the auth token claims and the server relies on the expiration (exp) claim to prevent replay attacks. When proxy support is enabled, the user authentication token is valid for 1 minute and can therefore be used by proxies during that time window.

Proxies must therefore be trusted during that minute, as they could reuse that token for themselve, as it is the case for any HTTP authentication process relying on Bearer tokens. 1 minute is considered reasonable as it is a smaller duration than most popular SSO platforms and should be long enough to traverse the Internet.

@francoismichel francoismichel mentioned this pull request Jan 3, 2024
@mpiraux mpiraux changed the title enable proxy support Enable HTTP proxy support Jan 4, 2024
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.

2 participants