[prev in list] [next in list] [prev in thread] [next in thread] 

List:       horde-dev
Subject:    Re: [dev] small change in message.php
From:       Chuck Hagenbuch <chuck () horde ! org>
Date:       2000-12-31 4:13:33
[Download RAW message or body]

Quoting Atif Ghaffar <aghaffar@developer.ch>:

>  I have started to hack IMP again :) (Im back!) 

Welcome back!

>  I saw all the nice features in the CVS HEAD version. 
>  I like the filtering. I implemented it too, but on the server-side (Sieve,
>  but thats vendor-specific). 

I would be interested in offering a Sieve front-end if you'd be interested in 
contributing it.

>  Would it be possible to change it to 
>  $headers['From'] = get_barefrom();  # reuse code. 

Yup, makes sense. I've committed it.

>  Oh and a small remark for the function get_barefrom. 
>  line 730 in IMP.lib writes 
>  $_barefrom .= '@' . $imp['maildomain']; 
> 
>  which is not always the best thing. 
>  For example, I connect to mx23.4unet.net and it changes my From address to
> 
>  aghaffar@mx23.4unet.net for which there is no MX record and if someone
> replies 
>  to that address, some mail servers on the net gets confused for a while. 
> 
>  Possible solution: 
>  Since there is a $conf['server']['to_domain'] variable defined in 
>  imp/config/conf.php, it would be nice to check if the value exists there and
> 
>  append it instead of $imp['maildomain']. Also sometimes, $imp['maildomain']
> is 
>  just an internal address such as "localhost",
> "cyrus.internal.zurich.4unet.net" 
>  etc etc. 

... sounds like you're not using the $servers['<servername>']['from_host'] 
correctly. That will get put into $imp['maildomain'] if you set it.

-chuck

--
Charles Hagenbuch, <chuck@horde.org>
"If you can't stand the heat, get out of the chicken!" - Baby Blues

-- 
Horde Developers mailing list: http://horde.org/
Frequently Asked Questions: http://horde.org/faq/
To unsubscribe, mail: dev-unsubscribe@lists.horde.org

[prev in list] [next in list] [prev in thread] [next in thread]