From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Ted Zlatanov Newsgroups: gmane.emacs.bugs Subject: bug#16220: url-http.el: Not conforming to HTTP spec Date: Mon, 23 Dec 2013 08:08:42 -0500 Organization: =?UTF-8?Q?=D0=A2=D0=B5=D0=BE=D0=B4=D0=BE=D1=80_?= =?UTF-8?Q?=D0=97=D0=BB=D0=B0=D1=82=D0=B0=D0=BD=D0=BE=D0=B2?= @ Cienfuegos Message-ID: <87ha9zaf51.fsf@flea.lifelogs.com> References: <87wqiwbjne.fsf@flea.lifelogs.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable X-Trace: ger.gmane.org 1387804094 12177 80.91.229.3 (23 Dec 2013 13:08:14 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Mon, 23 Dec 2013 13:08:14 +0000 (UTC) Cc: 16220@debbugs.gnu.org To: =?UTF-8?Q?Jaros=C5=82aw_?= =?UTF-8?Q?Rzesz=C3=B3tko?= Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Mon Dec 23 14:08:20 2013 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1Vv5F3-0002RL-CC for geb-bug-gnu-emacs@m.gmane.org; Mon, 23 Dec 2013 14:08:17 +0100 Original-Received: from localhost ([::1]:33511 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vv5F2-0004HF-PL for geb-bug-gnu-emacs@m.gmane.org; Mon, 23 Dec 2013 08:08:16 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:36767) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vv5Eu-0004Gx-Fl for bug-gnu-emacs@gnu.org; Mon, 23 Dec 2013 08:08:14 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Vv5Eo-0007OD-M5 for bug-gnu-emacs@gnu.org; Mon, 23 Dec 2013 08:08:08 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:49680) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Vv5Eo-0007O9-H7 for bug-gnu-emacs@gnu.org; Mon, 23 Dec 2013 08:08:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1Vv5Eo-0007Ds-1V for bug-gnu-emacs@gnu.org; Mon, 23 Dec 2013 08:08:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Ted Zlatanov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Mon, 23 Dec 2013 13:08:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 16220 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 16220-submit@debbugs.gnu.org id=B16220.138780404427703 (code B ref 16220); Mon, 23 Dec 2013 13:08:01 +0000 Original-Received: (at 16220) by debbugs.gnu.org; 23 Dec 2013 13:07:24 +0000 Original-Received: from localhost ([127.0.0.1]:35466 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Vv5EB-0007Ck-Gb for submit@debbugs.gnu.org; Mon, 23 Dec 2013 08:07:23 -0500 Original-Received: from mail-qe0-f43.google.com ([209.85.128.43]:40792) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Vv5E9-0007Cc-FS for 16220@debbugs.gnu.org; Mon, 23 Dec 2013 08:07:22 -0500 Original-Received: by mail-qe0-f43.google.com with SMTP id jy17so5090314qeb.30 for <16220@debbugs.gnu.org>; Mon, 23 Dec 2013 05:07:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=lifelogs.com; s=google; h=from:to:cc:subject:organization:references:mail-copies-to :gmane-reply-to-list:date:in-reply-to:message-id:user-agent :mime-version:content-type:content-transfer-encoding; bh=kF3LOqUGu4cDR6hkC2m3Pvg8sGtyHRtRr68rP3HRRWk=; b=j4bOmGu3LDiRwBYOYQhNtGtVDjm6Xf5k1fcXd3RMABRmFdL/qGYREhqnIsjTq+4CQk 8iRkuOli29jeT+qWaIg2x0Ksx+jpy49gIsO+ldLhZFh0HbA1dg7KR3v5R6Ly3hxOWVm4 QO5szSymMthOhPA9cd/JBeAFUyoSGw7SObXs8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:from:to:cc:subject:organization:references :mail-copies-to:gmane-reply-to-list:date:in-reply-to:message-id :user-agent:mime-version:content-type:content-transfer-encoding; bh=kF3LOqUGu4cDR6hkC2m3Pvg8sGtyHRtRr68rP3HRRWk=; b=nHZs8+RLNcpuM35PPnp6VZWXf1vnzZERFcd+D2av13NDin/gUsapjvjqgCOOuGTmif SOEy7PgLsjoLhqJgtP8xMR07PlfLlM4RJ5Xw4bMdERbhkopilxqgVeMUtNyQ0zgs6pW/ 4o4A7JX9hIuKEUEUqfW+m/EySrijmpEj6u191hUelf1adGBK4FZ6ZFbKcmcZac+zQumQ 6oD5D6qJJGg7L5dsX7JqhL9AfeICtu24bd0LbHEuowMRhwnoDoblkjxTQMmwUeF9IZki h998TCnsovUn6fq5p8WC5b1b08amb8FCHheNFAF72nfmhk9BXnxTeKuVwWFxAthHaH2c bEQw== X-Gm-Message-State: ALoCoQkMZn9yQ4P3np+35VHKUiLIkc0afms92VWOpMZHV1Z00Xa0aFNXIjZfxKpduNReePhJUlGs X-Received: by 10.224.11.68 with SMTP id s4mr42135985qas.88.1387804040505; Mon, 23 Dec 2013 05:07:20 -0800 (PST) Original-Received: from flea.lifelogs.com (c-98-229-61-72.hsd1.ma.comcast.net. [98.229.61.72]) by mx.google.com with ESMTPSA id ni10sm21974861qeb.23.2013.12.23.05.07.19 for (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 23 Dec 2013 05:07:19 -0800 (PST) X-Face: bd.DQ~'29fIs`T_%O%C\g%6jW)yi[zuz6; d4V0`@y-~$#3P_Ng{@m+e4o<4P'#(_GJQ%TT= D}[Ep*b!\e,fBZ'j_+#"Ps?s2!4H2-Y"sx" Mail-Copies-To: never Gmane-Reply-To-List: yes In-Reply-To: ("=?UTF-8?Q?Jaros=C5=82aw_?= =?UTF-8?Q?Rzesz=C3=B3tko?="'s message of "Mon, 23 Dec 2013 07:51:57 +0100") User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/24.3.50 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 140.186.70.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:82433 Archived-At: On Mon, 23 Dec 2013 07:51:57 +0100 Jaros=C5=82aw Rzesz=C3=B3tko wrote:=20 JR> I hope this is enough of a proof that the extra newline is a bug. I was already convinced :) JR> 2013/12/22 Ted Zlatanov : >> On Sun, 22 Dec 2013 22:55:07 +0100 Jaros=C5=82aw Rzesz=C3=B3tko wrote: >>=20 JR> To turn this into a concrete proposal: I suggest this part in url-http.= el JR> (starting line 356 in trunk): >>=20 JR> ;; End request JR> "\r\n" JR> ;; Any data JR> url-http-data JR> ;; If `url-http-data' is nil, avoid two CRLFs (Bug#8931). JR> (if url-http-data "\r\n"))) >>=20 JR> Should read simply: >>=20 JR> ;; End request JR> "\r\n" JR> ;; Any data JR> url-http-data)) >> ... I am OK with this fix, if anyone else wants to look it over and commit. JR> Futhermore url-http-attempt-keepalives should be nil as default, or bet= ter JR> yet should be completely removed, as true keepalive connections are any= way JR> not currently supported on the Emacs side, are they? Not AFAIK. Ted