Return-Path encoding

Resolved

(@xl32)

Thanks for the great plugin!

I faced another problem on my blog with Subscribe2. It does not correctly encodes Return-Path and Reply-To headers. That leads to something like this in mail client.

Patch I use on my blog is below:

--- class-s2-core.php   2013-02-19 00:07:50.000000000 -0800
+++ class-s2-core-fixed.php     2013-03-02 17:58:58.000000000 -0800
@@ -323,8 +323,8 @@
                        }
                }

-               $header['From'] = $this->myname . " <" . $this->myemail . ">";
-               $header['Reply-To'] = $this->myname . " <" . $this->myemail . ">";
+               $header['From'] = mb_encode_mimeheader($this->myname, 'UTF-8', 'Q') . " <" . $this->myemail . ">";
+               $header['Reply-To'] = mb_encode_mimeheader($this->myname, 'UTF-8', 'Q') . " <" . $this->myemail . ">";
                $header['Return-path'] = "<" . $this->myemail . ">";
                $header['Precedence'] = "list\nList-Id: " . html_entity_decode(get_option('blogname'), ENT_QUOTES) . "";
                if ( $type == 'html' ) {

http://wordpress.org/extend/plugins/subscribe2/

Viewing 3 replies - 1 through 3 (of 3 total)

mattyrob
(@mattyrob)

11 years, 1 month ago

@xl32,

Thank looks like a good fix, I’ll get that rolled into the next version of Subscribe2. There are a few bug reports in the comments of the function manual page with this function though, did you face any issues?

Thread Starter xl32
(@xl32)

11 years, 1 month ago

@mattyrob,

it works for me like a charm. However, I didn’t tested it with korean charsets. Please note the bugs those people refer to are closed in PHP 4.X in 2003 and 2005.

mattyrob
(@mattyrob)

11 years, 1 month ago

@xl32,

I’ve applied the patch on my sites and tested, no issues are apparent to me so I’ll roll this into the next version and we’ll see what happens. Thanks for your input. 🙂

Viewing 3 replies - 1 through 3 (of 3 total)

The topic ‘Return-Path encoding’ is closed to new replies.

Tags