BAD HEADER i notifieringsmejl

[tumba25]

Eftersom vi ändå är OT. Det innebär väl att ni stänger av viruskollen i mail. Är det inte bättre med lite fula headers?

[Slacker]

Jag har just kollat upp med en som uppgraderat till 2.0.23 och i den sker ingen kodning av subject. Däremot är subject kodat till UTF-8 i phpbb3. I e-postklienter kodas också ämnesraden, men t.ex på inlägg kodas hela på inlägg, medan patchen jag prövat kodar på och inlägg separat och det skapar problem med ihopskrivning. Patchen skulle behöva modifieras så att två närliggande ord med åäö kodas gemensamt.

Tipset jag gav om AVG8 stänger inte av viruskollen av e-mail utan bara att det i varje mail skrivs att mailet är viruskollat med AVG.

[Marcus]

Är inte det sista du skrev där ett belägg för att säga att AVG8 kodar om ditt meddelande, och därmed orsakar BAD HEADER?

Börjar klia lite i "flytta tråden"-fingret...

/marcus

[Slacker]

Nej, inte alls. BAD HEADER orsakas av phpbb2, men den upptäcks nästan enbart om man har AVG8 installerat, eftersom den raderar bokstäver med BAD HEADER från ämnesraden. BAD HEADER rapporteras också i headern av Amavis, som används som antivirus för att scanna e-mail av webbhotellet One.com. Det finns alltså en bugg egentligen i phpbb2 vad gäller subjectradens kodning.

[Slacker]

Jag tror att problemet är löst nu. Jag hittade nämligen en patch på en koreansk hemsida visserligen för UTF-8, men den var så allmänt skriven så den fungerade också för ISO-8859-1. Förändringen är enkel och det är bara att lägga till en extra rad i emailer.php samt göra en ändring:

Sök efter:

Kod: Markera allt

// Build header
		$this->extra_headers = (($this->reply_to != '') ? "Reply-to: $this->reply_to\n" : '') . (($this->from != '') ? "From: $this->from\n" : "From: " . $board_config['board_email'] . "\n") . "Return-Path: " . $board_config['board_email'] . "\nMessage-ID: <" . md5(uniqid(time())) . "@" . $board_config['server_name'] . ">\nMIME-Version: 1.0\nContent-type: text/plain; charset=" . $this->encoding . "\nContent-transfer-encoding: 8bit\nDate: " . date('r', time()) . "\nX-Priority: 3\nX-MSMail-Priority: Normal\nX-Mailer: PHP\nX-MimeOLE: Produced By phpBB2\n" . $this->extra_headers . (($cc != '') ? "Cc: $cc\n" : '')  . (($bcc != '') ? "Bcc: $bcc\n" : '');

Efter denna ett mellanrum och lägg till sedan raden:

Kod: Markera allt

		$this->subject = $this->encode($this->subject); //

Sök raden:

Kod: Markera allt

// determine length of encoded text within chunks and ensure length is even
                $length = 75 - strlen($start) - strlen($end);
                $length = floor($length / 2) * 2;

Ändra:

Kod: Markera allt

 // determine length of encoded text within chunks and ensure length is even
                $length = 140 - strlen($start) - strlen($end);
                $length = floor($length / 2) * 2;

Om sedan långa subjectrader inte blir fullständigt kodade, så höj värdet på $lenght ytterligare.
Hela subjectraden blir kodad i ISO-8859-1 med denna patch,

[Marcus]

Tack för ditt bidrag!

Vi kommer inte att "utvärdera" din patch, utan räknar med att den stämmer. Skulle den inte göra det, hoppas vi att feedback hamnar här också.

Hur som helst, så tackar vi för bidraget att förlänga livet för phpBB2. Det finns ju de som sitter på "tok-moddade" forum och phpBB3 är en omöjlig väg att gå utan att tappa funktioner (just nu).

/marcus