From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on dcvr.yhbt.net X-Spam-Level: X-Spam-Status: No, score=-3.2 required=3.0 tests=ALL_TRUSTED,AWL,BAYES_00 shortcircuit=no autolearn=ham autolearn_force=no version=3.4.2 Received: from localhost (dcvr.yhbt.net [127.0.0.1]) by dcvr.yhbt.net (Postfix) with ESMTP id 1B58E1F66F; Thu, 5 Nov 2020 07:50:23 +0000 (UTC) Date: Thu, 5 Nov 2020 07:50:21 +0000 From: Eric Wong To: Andrey Melnikov Cc: meta@public-inbox.org Subject: Re: [PATCH] nntp: delimit Newsgroup: header with commas Message-ID: <20201105075021.GA83277@dcvr> References: <20201103225559.GA5247@dcvr> <20201103235942.GA6163@dcvr> <20201104104448.GA6053@dcvr> <20201104194706.GA24387@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: List-Id: Andrey Melnikov wrote: > ????, 4 ????????. 2020 ??. ?? 22:47, Eric Wong : > > > > Andrey Melnikov wrote: > > > ????, 4 ????????. 2020 ??. ?? 13:44, Eric Wong : > > > > I suppose a few extra bytes won't hurt, too much :x > > > > > > Current: > > > Path: y > > > org.linuxdriverproject.driverdev-devel:35214 > > > org.kernel.vger.linux-devicetree:433117 > > > org.kernel.vger.linux-kernel:3765410 > > > org.kernel.vger.linux-media:192196 > > > > Huh? From what I can see, the last 4 lines are part of the > > Xref: header, which is unchanged atm (I plan on speeding it up > > with extindex, though). > > Yeah, leafnode rewrite Xref: header.. > -- raw -- > Sender: "devel" > Path: y > Xref: nntp://nntp.lore.kernel.org > org.linuxdriverproject.driverdev-devel:35265 > org.kernel.vger.linux-devicetree:433316 > org.kernel.vger.linux-kernel:3766059 > org.kernel.vger.linux-mmc:59564 > org.kernel.vger.linux-wireless:204883 > org.kernel.vger.netdev:733193 > Newsgroups: org.linuxdriverproject.driverdev-devel > org.kernel.vger.linux-devicetree > org.kernel.vger.linux-kernel org.kernel.vger.linux-mmc > org.kernel.vger.linux-wireless org.kernel.vger.netdev > Archived-At: > List-Archive: > -- on disk -- > Sender: "devel" > Path: y > org.linuxdriverproject.driverdev-devel:35265 > org.kernel.vger.linux-devicetree:433316 > org.kernel.vger.linux-kernel:3766059 > org.kernel.vger.linux-mmc:59564 > org.kernel.vger.linux-wireless:204883 > org.kernel.vger.netdev:733193 > Newsgroups: org.linuxdriverproject.driverdev-devel > org.kernel.vger.linux-devicetree > org.kernel.vger.linux-kernel org.kernel.vger.linux-mmc > org.kernel.vger.linux-wireless org.kernel.vger.netdev > Archived-At: > List-Archive: > Xref: news.localnet org.linuxdriverproject.driverdev-devel:4323 > --- > Haha. It simply does not expect folded multiline Xref: header. And > generate by itself a new (unfolded, separated by space) header when > storing article on disk. > Is really Newsgroups: & Xref: headers need folding? Nope, but currently PublicInbox::Eml folds by default (and Email::MIME/Email::Simple did in ->as_string, too). I didn't expect clients to have problems with folded lines since Received: and Cc: are often folded, anyways. Does leafnode struggle with those headers? Or just Newsgroups/Xref? We may need a separate API method for setting NNTP-only headers to appease leafnode... > > > > and new: > > > Path: nntp.lore.kernel.org!not.for.mail > > > save more extra bytes... > > > > > > Hehe... yet another cosmetic thing: > > > $ telnet nntp.lore.kernel.org 119 > > > Trying 54.189.247.149... > > > Connected to korg-lkml-1-news-lb-839eef9f3a4cef4e.elb.us-west-2.amazonaws.com. > > > Escape character is '^]'. > > > 201 nntp://nntp.lore.kernel.org ready - post via email > > > quit > > > Connection closed by foreign host. > > > > > > 'nntp://nntp.lore.kernel.org' is not a node name per RFC... But this > > > is a configuration problem on nntp.lore.kernel.org side. > > > > You can notify helpdesk@kernel.org to get it fixed. > > This is cosmetics. nntp readers do not use this hostname. It may be nice to extract hostname from URI, anyways, since it should be possible to advertise NNTPS endpoint in the WWW UI.