Jeremy Mikola
Jeremy Mikola
I'm a bit confused how the remember-me integration relates (if at all), to the following TODO item we already have in [AutoLoginListener](https://github.com/jmikola/AutoLogin/blob/212cce9f04ddec5e67196f440c83daf0923adf4c/src/Jmikola/AutoLogin/Http/Firewall/AutoLoginListener.php#L100): > TODO: This authentication method should be considered...
> Let us know if any more work needs to be done before merging, or if we should split the PR to separate the remember_me feature, since this part looks...
Reminder to announce the completed feature here: https://www.facebook.com/topic.php?uid=2344061033&topic=18863
If #37 is implemented, this issue becomes irrelevant.
This is planned, likely in a 1.1 release. I recall seeing this in the spec, but it was after I had already released 1.0 -- and I never got around...
I can confirm that the JSON file above appears to be valid. In addition to running the script @martinburch shared, I inspected the data with a JS interpreter: ``` //...
I'm going to leave this open for a bit, as I want to ensure we're not overlooking any of the `json_encode()` options. Also, I think CRS support needs to be...
The library does not handle token generation. From the library's perspective, the token is just a string field that is used to fetch the user account. I would suggest you...
Holding off on any review here until we can sort out questions in the component's PR: https://github.com/jmikola/AutoLogin/pull/15.
I don't have any suggestion for this beyond the [existing README example](https://github.com/jmikola/JmikolaAutoLoginBundle#fosuserbundle-configuration-example). The InvalidArgumentException above is thrown when the user provider for the bundle does not implement the necessary interface...