AzureAD / passport-azure-ad

The code for Passport Azure AD has been moved to the MSAL.js repo. Please open any issues or PRs at the link below.
https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/maintenance/passport-azure-ad
Other
422 stars 176 forks source link

README: recommend regenerating session by default #501

Closed jeffwilcox closed 4 years ago

jeffwilcox commented 4 years ago

The Passport library by default does not prevent against session fixation. Since the default experience for passport-azure-ad is to use cookies, I thought it would be helpful for the README example to clearly regenerate the session.

This is not a vulnerability with the library itself, but rather just extra caution common to the passport ecosystem per https://github.com/jaredhanson/passport/issues/192.