Skip Menu |

This queue is for tickets about the POE-Component-Client-NNTP CPAN distribution.

Report information
The Basics
Id: 39147
Status: resolved
Worked: 30 min
Priority: 0/
Queue: POE-Component-Client-NNTP

People
Owner: BINGOS [...] cpan.org
Requestors: dagolden [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: (no value)



Subject: Article headers break -- need a literal input separator for POE::Filter::Line
POE::Filter::Line needs to have a literal input separator for CRLF. The default regex is documented as having potential to break when CRLF is split between reads. This can inject blank lines, which in a news article header makes the header terminate prematurely. I edited line 188 of PoCo::Client::NNTP to this: Filter => POE::Filter::Line->new( InputLiteral => "\x0D\x0A" ), And the problem I was seeing appears to have gone away.
Applied change as suggested. New release has made its way on to CPAN. Many thanks, Chris 'BinGOs' Williams