FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

http: bump the pretend git version in the User-Agent by carlosmn · Pull Request #4041 · libgit2/libgit2 · GitHub

Repository navigation

http: bump the pretend git version in the User-Agent - #4041

Merged
ethomson merged 1 commit into
masterfrom
cmn/bump-pretend-git
Dec 20, 2016
Merged

ethomson merged 1 commit into
masterfrom
cmn/bump-pretend-git

Conversation

Copy link
Copy Markdown
Member

We want to keep the git UA in order for services to recognise that we're
a Git client and not a browser. But in order to stop dumb HTTP some
services have blocked UAs that claim to be pre-1.6.6 git.

Thread these needles by using the "git/2.0" prefix which is still close
enough to git's yet distinct enough that you can tell it's us.


It was reported that at least the GNOME host blocks us due to the UA claiming to be pre-1.6.6 git. While a bit ham-fisted it sure would block clients which don't speak the smart protocol. Chances are they're not the only ones who have taken these steps so let's bump it up to 2.0 since it's made up anyway.

We want to keep the git UA in order for services to recognise that we're
a Git client and not a browser. But in order to stop dumb HTTP some
services have blocked UAs that claim to be pre-1.6.6 git.

Thread these needles by using the "git/2.0" prefix which is still close
enough to git's yet distinct enough that you can tell it's us.
carlosmn added this to the libgit2 v0.25 - Rutschgefahr milestone Dec 20, 2016

Copy link
Copy Markdown
Member

I'm sure this will break something, I'm just curious to find out what.

ethomson merged commit 42864e5 into master Dec 20, 2016
ethomson deleted the cmn/bump-pretend-git branch January 9, 2019 10:29
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL