Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace http with https in twitter plugin. #195

Merged
merged 1 commit into from
Feb 13, 2014

Conversation

kleinschmidt
Copy link
Contributor

http requests produce a 403 error during OAuth.

http requests produce a 403 error during OAuth.
@eliotfowler
Copy link

Did you perhaps for from an unstable branch? I can't see why your changes would have failed the CI build.

@kleinschmidt
Copy link
Contributor Author

I was wondering about that. I cloned straight from master; is that not stable? If not I can make a pull request from a different branch.

@mroth
Copy link
Member

mroth commented Feb 12, 2014

It looks like it only failed on 1.9.3, on a completely unrelated test to this change. TravisCI has been pretty unreliable lately for some reason. I've restarted that job to see if it passes this time.

https://travis-ci.org/mroth/lolcommits/jobs/18663933

@eliotfowler
Copy link

Okay, great thanks!

@mroth
Copy link
Member

mroth commented Feb 13, 2014

Looks good.

mroth added a commit that referenced this pull request Feb 13, 2014
Replace http with https in twitter plugin.
@mroth mroth merged commit c8b2e01 into lolcommits:master Feb 13, 2014
@matthutchinson
Copy link
Member

@kleinschmidt 👍 thanks for fixing this

@matthutchinson
Copy link
Member

released just now, in v0.5.3

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.

None yet

4 participants