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

Enable https:// for 'image' in the Authentication Hash #36

Closed
Matwolf08 opened this issue Mar 11, 2012 · 0 comments
Closed

Enable https:// for 'image' in the Authentication Hash #36

Matwolf08 opened this issue Mar 11, 2012 · 0 comments
Assignees

Comments

@Matwolf08
Copy link

I have modified the code of the Facebook Strategy class to use https:// or http:// for the URL of the Facebook user image inside the Authentication hash.
I use a new boolean parameter (eg. :secure) passed on provider declaration:

provider :facebook, ENV['FACEBOOK_KEY'], ENV['FACEBOOK_SECRET'], :secure => true

The code of the Facebook Strategy class that I've changed is in this gist: https://gist.github.com/2014580

I hope that you can implement something similar for the next version of the gem, it's useful (IMHO) for websites that use SSL to avoid browser complaints about inclusion of unsafe resources! ;)

@ghost ghost assigned mkdynamic Mar 11, 2012
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

No branches or pull requests

2 participants