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

Kennedy is not sending close_notify #1

Open
dimkr opened this issue May 8, 2024 · 0 comments
Open

Kennedy is not sending close_notify #1

dimkr opened this issue May 8, 2024 · 0 comments

Comments

@dimkr
Copy link

dimkr commented May 8, 2024

Hello!

My client distinguishes between close_notify (as the Gemini spec requires) and a FIN: it warns the user when it doesn't know whether or not the page is truncated. Only few capsules (mostly old ones, before this requirement) don't send close_notify so it's mostly OK. Kennedy is also affected (😢) and that's the only reason why I don't bookmark it in the default configuration file of my client.

If I understand correctly, SslStream.ShutdownAsync() does this by calling SSL_shutdown(). However, the SslStream.Dispose() call of the using block in ProcessRequest() closes the connection without calling ShutdownAsync() first.

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

No branches or pull requests

1 participant