search

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

Remove express as a dependency #13

Closed HappyZombies closed 7 months ago

HappyZombies commented 1 year ago

I must've just missed this, but there's no reason to include express as a dependeny. Sure people will use it for express, but by removing express we reduce our dependency map on our side

jankapunkt commented 1 year ago

Hey @HappyZombies I just updated the deps from the original and thought this is all good, keeping it stable etc.

If this dependency isn't required in the code then let's remove it or make it dev dependency if required for Tests.

jankapunkt commented 8 months ago

Implemented in v4.0.0-rc.0