Skip to content

Buffer and propagate errors from transport - #205

Merged
djc merged 2 commits into
rustls:mainfrom
dtolnay-contrib:error
Sep 28, 2026
Merged

djc merged 2 commits into
rustls:mainfrom
dtolnay-contrib:error

Conversation

@dtolnay

@dtolnay dtolnay commented Sep 27, 2026

Copy link
Copy Markdown
Contributor

Fixes #204 using @trinity-1686a's proposed patch from #204 (comment).

@djc djc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the regression, and thanks for working on a fix.

Would be helpful to additionally include a commit with a version bump.

Comment thread src/client.rs
pub(crate) session: ClientConnection,
pub(crate) state: TlsState,
pub(crate) need_flush: bool,
pub(crate) buffered_err: Option<io::Error>,

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: let's rename this to just error, with a documentation comment explaining its usage?

Comment thread src/client.rs
@djc
djc merged commit bd0494d into rustls:main Sep 28, 2026
11 checks passed
@djc djc mentioned this pull request Sep 28, 2026
@djc

djc commented Sep 28, 2026

Copy link
Copy Markdown
Member

Preparing release in

@djc

djc commented Sep 28, 2026

Copy link
Copy Markdown
Member

Published 0.26.6.

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.

Regression in 0.26.5: turns ConnectionAborted transport error into clean EOF

3 participants