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=-2.5 required=3.0 tests=ALL_TRUSTED,BAYES_00, BODY_8BITS 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 18DC01F66F; Wed, 4 Nov 2020 19:47:07 +0000 (UTC) Date: Wed, 4 Nov 2020 19:47:06 +0000 From: Eric Wong To: Andrey Melnikov Cc: meta@public-inbox.org Subject: Re: [PATCH] nntp: delimit Newsgroup: header with commas Message-ID: <20201104194706.GA24387@dcvr> References: <20201103225559.GA5247@dcvr> <20201103235942.GA6163@dcvr> <20201104104448.GA6053@dcvr> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: List-Id: 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). > 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. Though I suppose the public-inbox code could warn or extract the hostname out of it. (NNTPS support would be nice, but maybe OpenSSL or O::Socket::SSL on RHEL/CentOS 8 is too old)