#5915 closed defect (bug) (wontfix)
Same problem as in #252: apop broken if zero occurs in banner in 2.3.3 class-pop3.php
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | Priority: | normal | |
Severity: | normal | Version: | 2.3.3 |
Component: | Keywords: | close | |
Focuses: | Cc: |
Description
While extracting the banner (function parse_banner) in preparation for apop, the empty method is called on $digit which when zero returns false and hence doesn't get put into the resultant $banner.
Example parse_banner reply with <14649.123343777@…> but the string was "+OK ready <14649.1203343777@…>"
or another example
"+OK ready <14649.1203343777@…>" and $pop3->parse_banner give back <14419.12334323@…>
then a md5($AuthString) must fail, because the 0 is significant by apop.
I put a workaround at http://www.babsi.de/silentapop/, that does it for me, but this touch also wp-main.php
Attachments (1)
Change History (13)
#3
follow-up:
↓ 5
@
15 years ago
- Component changed from Mail to Blog by Email
- Milestone changed from 2.9 to Future Release
- Owner set to westi
- Status changed from new to accepted
Moving all blog-by-email bugs to a seperate component and out of 2.9.
This will all be address by a new canonical blog-by-email plugin which is under development and will hopefully be available by the time 2.9 is released.
#5
in reply to:
↑ 3
;
follow-up:
↓ 6
@
10 years ago
Replying to westi:
This will all be address by a new canonical blog-by-email plugin which is under development and will hopefully be available by the time 2.9 is released.
Did this happen?
#6
in reply to:
↑ 5
;
follow-up:
↓ 9
@
10 years ago
Replying to ericlewis:
Replying to westi:
This will all be address by a new canonical blog-by-email plugin which is under development and will hopefully be available by the time 2.9 is released.
Did this happen?
Not before 2.9 I guess, but codebykat did a plugin for her OPW internship: https://github.com/codebykat/wp-post-by-email
If you want to figure out if/how we can move that out of core, that could be cool.
#8
@
10 years ago
- Keywords needs-patch removed
We can take a similar approach to the decommissioning of links. Leave the functionality and UI, hide UI if it isn't being used.
#10
@
9 years ago
- Keywords close added
This is a pretty old ticket, but looking at the class, it evidently still exists as a possible issue?
This particular class is one of those deeply buried, rarely touched kind of things. Maybe it's just me, but I can't even find where it is actually used anymore.
Is this still a relevant ticket? Should it be closed as wontfix?
Here is a patch.
-- t.baboon