Skip to content

Fix to issues/572#573

Merged
hannesdejager merged 1 commit into
bolcom:masterfrom
martin-cowie:fix/multiline-empty-line-panic
Jul 19, 2026
Merged

Fix to issues/572#573
hannesdejager merged 1 commit into
bolcom:masterfrom
martin-cowie:fix/multiline-empty-line-panic

Conversation

@martin-cowie

Copy link
Copy Markdown
Contributor

Summary

  • Fix a panic in FtpCodec::encode when a multiline reply contains an empty line, by replacing .unwrap() with .is_some_and(...) when checking whether a line starts with a digit.
  • Add a regression test covering encoding of a multiline reply with a blank line.

Fixes #572

@hannesdejager

Copy link
Copy Markdown
Collaborator

Thank you!

@hannesdejager
hannesdejager merged commit 212fdd4 into bolcom:master Jul 19, 2026
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.

FtpCodec::encode panics on multi-line Reply holding a non-final empty line.

2 participants