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

npm publish 0.1.7 #20

Closed
pierreinglebert opened this issue Mar 20, 2014 · 10 comments
Closed

npm publish 0.1.7 #20

pierreinglebert opened this issue Mar 20, 2014 · 10 comments

Comments

@pierreinglebert
Copy link

The last version available on npmjs is the 0.1.5 and this repo is 0.1.7, could you please publish it ?

@orweinberger
Copy link

+1

@burtonjc
Copy link

👍 @jaredhanson, we have some security concerns with 0.1.5 because we cannot specify state: true. That looks like it is fixed in master though. Any idea on when we could get the next version of this?

@theogravity
Copy link

Try @cfsghost 's https://github.com/cfsghost/passport-github

Verified working. You'll need to use the user:email scope when doing your initial auth or it will die on you.

app.get('/auth/git', passport.authenticate('github', { scope: ['user:email'] }))

@burtonjc
Copy link

@jaredhanson if you are looking for help maintaining this, I would be happy to help. I would prefer to still use this over a fork.

@lauweijie
Copy link

Any plans to publish 0.1.7 on npm?

0.1.5 lists passport-oauth 0.1.x as a dependency and that gives a deprecated warning:
createCredentials() is deprecated, use tls.createSecureContext instead

@ianjuma
Copy link

ianjuma commented Apr 25, 2015

+1

@knownasilya
Copy link

@jaredhanson ping

@manland
Copy link

manland commented Jul 11, 2015

👍

@jaredhanson
Copy link
Owner

I've published this to npm as passport-github@1.0.0.

@burtonjc
Copy link

Thank you @jaredhanson!

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

9 participants