Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
ethomson merged 1 commit intomasterlibgit2/libgit2:masterfrom
cmn/bump-pretend-gitlibgit2/libgit2:cmn/bump-pretend-gitCopy head branch name to clipboard
Dec 20, 2016
Merged

http: bump the pretend git version in the User-Agent#4041
ethomson merged 1 commit intomasterlibgit2/libgit2:masterfrom
cmn/bump-pretend-gitlibgit2/libgit2:cmn/bump-pretend-gitCopy head branch name to clipboard

Conversation

@carlosmn
Copy link
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 carlosmn added this to the libgit2 v0.25 - Rutschgefahr milestone Dec 20, 2016
@ethomson
Copy link
Member

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

@ethomson ethomson merged commit 42864e5 into master Dec 20, 2016
@ethomson ethomson deleted the cmn/bump-pretend-git branch January 9, 2019 10:29
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.