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

Document isAuthenticated usage #630

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blzaugg
Copy link

@blzaugg blzaugg commented Dec 3, 2017

No description provided.

@coveralls
Copy link

coveralls commented Dec 3, 2017

Coverage Status

Coverage remained the same at 98.87% when pulling da1b293 on blzaugg:patch-1 into 13c550c on jaredhanson:master.

1 similar comment
@coveralls
Copy link

coveralls commented Dec 3, 2017

Coverage Status

Coverage remained the same at 98.87% when pulling da1b293 on blzaugg:patch-1 into 13c550c on jaredhanson:master.

rwky added a commit to passport-next/passport that referenced this pull request Jul 7, 2018
@rwky
Copy link

rwky commented Jul 7, 2018

Merged in https://github.com/passport-next/passport

@godmar
Copy link

godmar commented Feb 10, 2019

FWIW, req.isAuthenticated does not require sessions.

@I3uckwheat
Copy link

I3uckwheat commented Mar 11, 2020

LGTM, can we get this merged? This has been an incredible pain point for me.

@uzair004
Copy link

please accept this pull request, very useful and will save people tons of hours.

@I3uckwheat
Copy link

@jaredhanson Can you comment on getting this merged?

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

6 participants