Sebas - 2017-08-28 15:47:05
Hello,
Please consider to make State parameter configurable. The current logic of the class is great indeed and I believe it should stay as it is further but with little more flexibility using condition IF for the state parameter. For example if to use own encode/decode algorithm instead of php session. Therefore would be really good to use state value just with own parameters (where I can wrap the timestamp as well for checking that later to prevent repetition attacks).
Would be really good to add to your class empty state option $client->state = '' (similar to $client->scope = '') -> if $client->state = '' (empty) then class will be using build-in random session ($_SESSION['OAUTH_STATE']) otherwise use the defined value.
Thanks