lookyman / nette-oauth2-server-doctrine

Integration of The League of Extraordinary Packages' OAuth 2.0 Server into Nette Framework- Kdyby/Doctrine storage implementation
MIT License
13 stars 3 forks source link

Setup breaks with server version 5.1.4+ #13

Closed Fuco1 closed 7 years ago

Fuco1 commented 7 years ago

In versions 5.1.4+ and 6.0+ there were some changes to the setup: https://oauth2.thephpleague.com/v5-security-improvements/

specifically one has now to call

$server->setEncryptionKey('lxZFUEsBCJ2Yb14IF2ygAHI5N4+ZAUXXaSeeJm6+twsUmIen');

(where the string is some random data)

before adding grants or doing anything with the server.

On the 5 series this generates a deprecation warning only (which we treat as errors in our codebase). I can pin for the lower version for now but it would be nice to have this fixed :)

lookyman commented 7 years ago

Can you please test it with https://github.com/lookyman/nette-oauth2-server-doctrine/pull/14?

I have a huge refactoring in the works, which will be for 6.0+ only, but it needs more work. So this should help in the meantime.

Fuco1 commented 7 years ago

The fix in #14 works, it installs 5.1.3 for me.

lookyman commented 7 years ago

Thanks! https://github.com/lookyman/nette-oauth2-server-doctrine/releases/tag/v2.0.2