Bug 293298 - add rekonq version to user agent string
Summary: add rekonq version to user agent string
Status: RESOLVED FIXED
Alias: None
Product: rekonq
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Unlisted Binaries Linux
: NOR wishlist
Target Milestone: 0.10
Assignee: Andrea Diamantini
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-02-04 14:32 UTC by Anthon
Modified: 2012-06-07 23:24 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anthon 2012-02-04 14:32:48 UTC
Version:           unspecified
OS:                Linux

The User-Agent string contains 'rekonq'.  Please add the version number, eg
'rekonq/1.0'.

Reproducible: Didn't try


Actual Results:  
 

Expected Results:
Comment 1 Andrea Diamantini 2012-02-05 06:02:29 UTC
Why this will help you in any way? The important things in the UA string are just two:
The "Mozilla/5.0" part and the "AppleWebKit" one, checked by 99% of sites checking UAs. Some of them (usually, the ones providing CSS3 or HTML5 features) check also WebKit release number, that actually is 534.34.
Two different rekonq versions (let's say rekonq 0.8 and rekonq 0.9) provided against the same qtwebkit lib will do exactly the same rendering, so again: why do you need rekonq version number in the UA string?
Comment 2 Anthon 2012-02-05 13:01:41 UTC
It doesn't conform to convention of other browsers, i.e., appname/appversion.  See Mozilla's user agent string spec:

https://developer.mozilla.org/en/Gecko_user_agent_string_reference

Web analytics apps like Piwik that expect the UA to identify itself using this convention will instead detect rekonq as Safari.
Comment 3 Andrea Diamantini 2012-06-07 23:24:56 UTC
Git commit 1c3875353c2be223af4d78b57a774d61218742ee by Andrea Diamantini.
Committed on 08/06/2012 at 01:25.
Pushed by adjam into branch 'master'.

Add rekonq version to default user agent string

M  +4    -0    src/main.cpp

http://commits.kde.org/rekonq/1c3875353c2be223af4d78b57a774d61218742ee