Israel Carberry

Results 16 comments of Israel Carberry

> @elchris any support for 5.3 with the new new Authenticator-based Security ? @MatthieuMan see #685 - it's based off of main, rather than this branch.

Still working on this. Adding an authenticator that manages authentication via the new `Passport` with user and credential badges.

Not sure about the user provider dependency injection yet, will be testing in the context of another app.

Token storage is already taken care of in `AuthenticatorManager::handleAuthenticationSuccess()` when `::executeAuthenticator()` is absolutely sure everything is kosher, so the token storage dependency in the `OAuthAuthenticator` was not needed.

@deguif I noticed when testing this in an app that an unauthenticated request results in an `AccessDeniedException` rather than an `AuthenticationCredentialsNotFoundException`. Is this a result of other recent changes, or...

@victormacko yes, at least for this PR, or until changes are requested. Feel free to contribute by forking my fork, and I'll pull any changes into here. Some other Symfony...

Rebased to include commits "[Remove support for Propel](https://github.com/FriendsOfSymfony/FOSOAuthServerBundle/commit/bf5080b5c8f3a2035dcefee56b9b9e933ebb3ef4)" and "[Move Travis to GH Actions and fix CI builds](https://github.com/FriendsOfSymfony/FOSOAuthServerBundle/commit/dc8ff343363cf794d30eb1a123610d186a43f162)".

Removing support of Symfony 5.1 and 5.2 as a result of ProxyManager dependency change for ClassGenerator.

Working on it [here](https://github.com/iisisrael/FOSOAuthServerBundle/tree/symfony-6-clean). Builds on #685. Trying to maintain BC support for Symfony 5.3, but that might be a lost cause. Branch / version management on this package could...

Forgot to mention, I've only worked on this in a linux environment. I'll boot into windows tomorrow and see what I see.