search

node-oauth / node-oauth2-server

🚀 The successor to oauthjs/oauth2-server. 🔒 Complete, compliant, maintained and well tested OAuth2 Server for node.js. Includes native async await and PKCE.
https://www.npmjs.com/package/@node-oauth/oauth2-server
MIT License
317 stars 46 forks source link

fix: type of "scope" in AuthenticateOptions is now string #323

Closed robey closed 1 month ago

jankapunkt commented 1 month ago

THank you @robey for your contribution. Please provide a description by the template we provided. Please also refer to the issue that this PR is about to close.

Note, that development is not longer the target branch. If that's still in the guidelines, then we need to change that.

robey commented 1 month ago

Take or leave this patch. I decided the repo was abandoned and forked it for myself.

jankapunkt commented 1 month ago

We all have our obligations and do this in our free time. It's not a company backed repo.

At the same time you could have helped by simply reading the contribution guidelines and prepare you pr accordingly, saving us the time of research whether this PR is even relevant.

Instead, you provide an empty description. Finally, you passive-agressively declare this repo as abandoned because things don't play out in the way you want them.