anvilresearch / connect

A modern authorization server built to authenticate your users and protect your APIs
http://anvil.io
MIT License
361 stars 86 forks source link

OAuth2 `client_id` vs `clientId` #368

Open tomkersten opened 6 years ago

tomkersten commented 6 years ago

The recent refactor for tests resulted in the expected Client ID config param changing from client_id to clientId. Not a big deal once found, however it was undocumented so took a bit to track down. I'm not sure if we want it to be clientId or to restore it to client_id, as all other config params are underscore-based (redirect_uri, etc).

Thoughts?

camfou commented 6 years ago

Hi, this PR fix this issue : https://github.com/anvilresearch/connect/pull/374