GNU bug report logs - #13598
24.3.50; url-http.el doesn't correctly parse headers when they are sent line-by-line

Previous Next

Package: emacs;

Reported by: coroa <at> online.de (Jonas Hoersch)

Date: Thu, 31 Jan 2013 18:11:01 UTC

Severity: normal

Merged with 14372

Found in versions 24.3, 24.3.50

Fixed in version 24.4

Done: Glenn Morris <rgm <at> gnu.org>

Bug is archived. No further changes may be made.

Full log

Message #23 received at 13598 <at> debbugs.gnu.org (full text, mbox):

From: Bastien <bzg <at> altern.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 13598 <at> debbugs.gnu.org, Jonas Hörsch <coroa <at> online.de>
Subject: Re: bug#13598: 24.3.50;
	url-http.el doesn't correctly parse headers when they are sent
	line-by-line
Date: Thu, 14 Feb 2013 07:08:40 +0100

Hi Glenn,

Glenn Morris <rgm <at> gnu.org> writes:

>>>> i'm positive now, that changing the regex to "^\r+$" is the way to go.
>
> Would "^\r?\n" be better?

The quote of the OP is misleading -- he proposed to change the regexp
in (re-search-forward "^\r*$" nil t) to "^\r*\n", which is the fix I'm
talking about.

But yes, "^\r?\n" is slightly better than "^\r*\n" because AFAIK there
can be only one CR in the line separating the headers from the body.

Let me know if you want to fix this yourself or if I should do it.

Thanks,

-- 
 Bastien
This bug report was last modified 11 years and 78 days ago.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.