Skip to content

Update docs with curl cmdline which follows redirects.#8

Merged
robmadole merged 1 commit intorobmadole:developfrom
mithro:develop
Jul 23, 2014
Merged

Update docs with curl cmdline which follows redirects.#8
robmadole merged 1 commit intorobmadole:developfrom
mithro:develop

Conversation

@mithro
Copy link
Contributor

@mithro mithro commented Jul 23, 2014

To improve security github has moved all user content from the github.com domain to githubuserconent.com domain, see https://developer.github.com/changes/2014-04-25-user-content-security/ for more information.

They offer redirects from the original locations, but curl won't follow redirects by default. Annoying curl doesn't error even give you an error when it hits a redirect and does nothing! Thus the command
curl https://raw.github.com/robmadole/jig-plugins/lists/common.txt > .jigplugins.txt looks like it succeeds but just ends up creating an empty .jigplugins.txt file. This confused me for a while.

To improve security github has moved all user content from the github.com
domain to githubuserconent.com domain, see
https://developer.github.com/changes/2014-04-25-user-content-security/ for more
information.

They offer redirects from the original locations, but curl won't follow
redirects without the -L option.

An alternative would be to update the `https://raw.github.com/` URLs to be
`https://raw.githubusercontent.com/` instead but if they ever change the domain
again you'll need to update the documentation again.
@robmadole
Copy link
Owner

Wow I didn't realize it was creating empty .jigplugins.txt files! Thanks for fixing this. I'll merge it now and cut a release here in a few minutes.

robmadole added a commit that referenced this pull request Jul 23, 2014
Update docs with curl cmdline which follows redirects.
@robmadole robmadole merged commit c6fa9e9 into robmadole:develop Jul 23, 2014
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