New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Road to version 2 - Refactor and rewrite most of the library #33
Conversation
45750bf
to
7b12b2b
Compare
c4d9799
to
3cd0f0f
Compare
$request
to each CAS methods
af697b4
to
0aa843b
Compare
0aa843b
to
f884d92
Compare
d3a1b21
to
1d22bd4
Compare
$request
to each CAS methods1d22bd4
to
469f720
Compare
6e6462f
to
bc9a8dc
Compare
0539fae
to
5a3fe7d
Compare
…ests accordingly.
a1496b4
to
13e929d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks promising. I won't pretend that I got all the complexity of the lib and that I understood the reasons behind all the choices you made, but I can say that I like the way you merged the handlers, replaced the introspection by responses, simplified the configuration, and finally removed the ServerRequestInterface
to inject the Request
object in relevant methods instead.
Reviewing the Cas
class was not made easy by all the GitHub Actions checks but I tried my best ^^
Anyway I find this version much more straight forward and easy to understand. Good job here!
31d7644
to
619cc7e
Compare
a05c55e
to
5b90e2f
Compare
This is the kind of Content-Type header that EU Login is sending.
BREAKING CHANGE: yes
… make it throws when the PGT is not available.
081d2d7
to
c548388
Compare
@J-Ben87 This branch has been merged into In the upcoming days, I will update After a while testing the master branch of those packages, I'll tag'n'cut a release! |
Thank you so much! Amazing work 💪 |
This PR
$request
in the CAS constructor, also make sure that theRequest
is passed to each relevant method instead of being injected in the constructor.allowed_parameters
property.handlers
,services
andredirect
classes.monolog/monolog
and throw exceptions when needed.Introspector
, useResponse
decorator and factoriesloophp/psr17
Related to ecphp/cas-bundle#63 and ecphp/cas-bundle#59
Related to ecphp/ecas#27
Related to ecphp/eu-login-bundle#40