fix: scope parameter from medusa-config.js not passed through to passport

adrien2p / medusa-plugins

A collection of awesome plugins for medusa :rocket:

https://medusa-plugins.vercel.app

MIT License

144 stars 42 forks source link

fix: scope parameter from medusa-config.js not passed through to passport #168

Closed F-Wilke closed 3 weeks ago

F-Wilke commented 3 weeks ago

I had to extend the scope of the token request to Microsoft Entra in order to receive the claims that I needed.

For that, setting the scope parameter in the passportAuthenticateMiddlewareOptions was not enough, since passport (at least for version 0.6.0) determines the scope from the argument "options" passed to the constructor of the strategy.

I hope you are fine with me bumping the version and writing to the changelog. I'm sorry if you're not. It's my first pull request.

vercel[bot] commented 3 weeks ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
medusa-plugins	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jun 10, 2024 11:14pm