node-oauth / express-oauth-server

🔒 Complete, compliant and well tested module for implementing an OAuth2 Server/Provider with express in node.js
https://www.npmjs.com/package/@node-oauth/express-oauth-server
MIT License
27 stars 8 forks source link

Update deps and use @node-oauth/oauth2-server #7

Closed jankapunkt closed 1 year ago

jankapunkt commented 1 year ago

Test this with your local setup by cloning this repo and checkout the branch 3.0.0

Other notes:

shrihari-prakash commented 1 year ago

Might also be a good idea to remove bluebird dependency since promises are native in a good amount of node versions. But I don't know if there is already a plan for this.

HappyZombies commented 1 year ago

Might also be a good idea to remove bluebird dependency since promises are native in a good amount of node versions. But I don't know if there is already a plan for this.

@shrihari-prakash I agree but let's keep that for another version/MR, this whole module should/will probably be refactored entirely at some point. The focus on this MR is to just update the node-oauth dependency and other deps.

jankapunkt commented 1 year ago

Thanks @HappyZombies @shrihari-prakash I will merge and then create a release under our org