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

Use HTTPS for Vimeo OAuth and endpoint requests. #49

Merged
merged 1 commit into from
Sep 13, 2013

Conversation

lukeasrodgers
Copy link
Contributor

Using HTTPS ensures, among other advantages, that URLs to assets returned by (e.g.) the videos endpoint will point to the version of the resource served over HTTPS, so that we get https://secure-b.vimeocdn... instead of http://b.vimeocdn....

I was unable to check whether this breaks any specs, as I couldn't get the test suite to run -- #33.

@lukeasrodgers
Copy link
Contributor Author

Rebased a commit into this PR so that all specs now pass.

@paulkoegel
Copy link

+1
need this to serve teaser images from vimeo on my HTTPS page and not run into mixed content warnings.
thanks for the PR!

@jhilden
Copy link

jhilden commented Sep 11, 2013

+1

@mrcljx mrcljx merged commit 4aa3ce0 into matthooks:master Sep 13, 2013
@jhilden
Copy link

jhilden commented Sep 17, 2013

Thanks for merging this.

@jhilden
Copy link

jhilden commented Sep 17, 2013

Are you planning to cut a new release anytime soon?

@jhilden
Copy link

jhilden commented Dec 4, 2013

Any new release in sight?

@jhilden
Copy link

jhilden commented Jan 28, 2014

here is your friendly bi-monthly release reminder ;)

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.

4 participants