Return-Path: X-Original-To: apmail-hc-dev-archive@www.apache.org Delivered-To: apmail-hc-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9267ADAA7 for ; Sat, 17 Nov 2012 01:42:56 +0000 (UTC) Received: (qmail 65988 invoked by uid 500); 17 Nov 2012 01:42:56 -0000 Delivered-To: apmail-hc-dev-archive@hc.apache.org Received: (qmail 65951 invoked by uid 500); 17 Nov 2012 01:42:56 -0000 Mailing-List: contact dev-help@hc.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: "HttpComponents Project" Delivered-To: mailing list dev@hc.apache.org Received: (qmail 65942 invoked by uid 99); 17 Nov 2012 01:42:56 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 17 Nov 2012 01:42:56 +0000 X-ASF-Spam-Status: No, hits=1.5 required=5.0 tests=HTML_MESSAGE,RCVD_IN_DNSWL_LOW,SPF_PASS,URIBL_DBL_REDIR X-Spam-Check-By: apache.org Received-SPF: pass (athena.apache.org: domain of garydgregory@gmail.com designates 209.85.210.51 as permitted sender) Received: from [209.85.210.51] (HELO mail-da0-f51.google.com) (209.85.210.51) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 17 Nov 2012 01:42:47 +0000 Received: by mail-da0-f51.google.com with SMTP id t11so1250203daj.10 for ; Fri, 16 Nov 2012 17:42:27 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type; bh=sFdqk4RIIvdDkBgsfNRUQW0OLTGENIErl5wArEbEs9M=; b=UBqb+WxJ9vzGd/y2qZkYPWVcY5OFmP8yrGsFDbF+4JQ6CaSYeBdgLpPtuH+oDsxPil zzaMb+JrY0P6wCZV8jKrPo88kgnYQ/qkyJhmnmuw8XUnMgNbBs/PuPkwnnftefXi6+jp dlZWlKQZt56BFQ4JThn4Er8QSQswNPa+Z8DqLtVsg9d0bP1um3+mvMBHx8D7P/WrpM4H NJRqKu4UqvSppsaNvTtnuxXv/erqFfJzHWLTtg6zoVo+h3TtULfn7HDqGzAvFWKWWKyw /GDRHcgrq63S+G+WdWzgOmbZmR0O6gpNEpVCGoQ02W+zijtMLtqwOGJqrlSKFFaJxQuU LtUw== MIME-Version: 1.0 Received: by 10.68.238.72 with SMTP id vi8mr14476576pbc.55.1353116546932; Fri, 16 Nov 2012 17:42:26 -0800 (PST) Received: by 10.68.134.36 with HTTP; Fri, 16 Nov 2012 17:42:26 -0800 (PST) In-Reply-To: <1353097416.20127.2.camel@ubuntu> References: <1353081301.20463.23.camel@ubuntu> <1353097416.20127.2.camel@ubuntu> Date: Fri, 16 Nov 2012 20:42:26 -0500 Message-ID: Subject: Re: Log misleading for User-Agent From: Gary Gregory To: HttpComponents Project Content-Type: multipart/alternative; boundary=047d7b339bb71ea0cf04cea6ff83 X-Virus-Checked: Checked by ClamAV on apache.org --047d7b339bb71ea0cf04cea6ff83 Content-Type: text/plain; charset=UTF-8 On Fri, Nov 16, 2012 at 3:23 PM, Oleg Kalnichevski wrote: > On Fri, 2012-11-16 at 17:33 +0000, sebb wrote: > > On 16 November 2012 15:55, Oleg Kalnichevski wrote: > > > On Fri, 2012-11-16 at 06:38 -0500, Jon Moore wrote: > > >> Yes, I'd be in favor of either one of these changes (removing or > > >> reporting dynamically the right version). I'd probably lean towards > > >> removing it if pressed to express an opinion. > > >> > > >> As I recall (although I don't have the code in front of me), the > > >> CachingHttpClient sends a slightly different User-Agent, mentioning > > >> the fact that the cache is present in the stack in the comment field. > > >> I think this made sense originally, when it was really an add-on > > >> module. However, since Oleg is re-wiring the "stack" for the default > > >> client in 4.3, that may not be needed any more. > > >> > > >> It could just be that I'm remembering the comment in the Via header > > >> the caching layer adds, though; in which case the caching client > > >> wouldn't modify the User-Agent. > > >> > > >> Jon > > >> > > >> On Thu, Nov 15, 2012 at 9:00 PM, Gary Gregory > wrote: > > >> > Hi All: > > >> > > > >> > At the debug logging level I see: > > >> > > > >> > User-Agent: Apache-HttpClient/4.2.1 (java 1.5) > > >> > > > >> > But I am running on Java 6... so, Why is 1.5 hard coded? > > >> > > > >> > Could/Should we remove "(java 1.5)" or have it report the correct > version? > > >> > > > >> > Thoughts? > > >> > > > >> > Gary > > >> > > > > > > > If my memory serves me well the original intention was to include a > > > minimal JRE version a particular release is compatible with. > > > > > > I am fine with either dropping it or replacing with a dynamically > > > generated JRE version requests are generated with. > > > > Or replace with: > > > > User-Agent: Apache-HttpClient/4.2.1 (Java 1.5+) > > > > if we want to keep the original meaning (but clarify it). > > > > Sounds very reasonable. > > > Do other user agents include OS/Java details in their UA strings? > > > > I believe JRE's internal HTTP client does. > It is reasonable indeed, I'll give you that ;) but I find it slightly confusing and of dubious value. So if we are doing to change it, I'd like to discuss an alternative. It is confusing because we have two versions in the string, but they represent different concepts, one is a version stamp and the other a platform requirement. I find it of dubious value because I find it more interesting to know the Java version of the user agent rather than what the minimum Java version user agent requires to run. But hey, that's me. In my mind, the options are: 1) do nothing 2) User-Agent: Apache-HttpClient/4.2.1 (Java 1.5+ min) 3) User-Agent: Apache-HttpClient/4.2.1 (Java/1.6.0_35) 3 is in the spirit of the JRE's HttpURLConnection FWIW. Gary > > Oleg > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org > For additional commands, e-mail: dev-help@hc.apache.org > > -- E-Mail: garydgregory@gmail.com | ggregory@apache.org JUnit in Action, 2nd Ed: http://bit.ly/ECvg0 Spring Batch in Action: http://bit.ly/bqpbCK Blog: http://garygregory.wordpress.com Home: http://garygregory.com/ Tweet! http://twitter.com/GaryGregory --047d7b339bb71ea0cf04cea6ff83--