LMTP fix: 7.7 or 7.9?

View: New views
4 Messages — Rating Filter:   Alert me  

LMTP fix: 7.7 or 7.9?

by Bill Welliver :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

I found a semantic error in the LMTP module: the various domain checks
assume one or more "handled domains" that consist of exactly 2 host
components. Therefore, if you try to create an LMTP server object with a
domain of "foo.bar.com", checks will fail due to the code ignoring the
"foo" component.

Is a fix for this something that is allowable for 7.7, or shall it be held
for 7.9?

Bill

LMTP fix: 7.7 or 7.9?

by Marcus Comstedt (ACROSS) (Hail Ilpalazzo!) @ Pike (-) developers forum :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Fix it.

Re: LMTP fix: 7.7 or 7.9?

by Stephen R. van den Berg :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Bill Welliver wrote:
>I found a semantic error in the LMTP module: the various domain checks
>assume one or more "handled domains" that consist of exactly 2 host
>components. Therefore, if you try to create an LMTP server object with a
>domain of "foo.bar.com", checks will fail due to the code ignoring the
>"foo" component.

>Is a fix for this something that is allowable for 7.7, or shall it be held
>for 7.9?

That qualifies as a bug, and should go into 7.7, IMHO.
--
Sincerely,
           Stephen R. van den Berg.

"Always remember that you are unique.  Just like everyone else."

Re: LMTP fix: 7.7 or 7.9?

by Bill Welliver :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

OK, I've committed the fix to 7.7. Not sure whether it qualifies for a  
changelog entry, as it's a bugfix rather than a new feature.

Bill

On Jul 22, 2008, at 7:09 AM, Stephen R. van den Berg wrote:

> That qualifies as a bug, and should go into 7.7, IMHO.
LightInTheBox - Buy quality products at wholesale price