-
Notifications
You must be signed in to change notification settings - Fork 510
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
Support for play 2.5 #582
Comments
+1 |
I had following error with Play 2.5, Guice and SecureSocial 3.0-M4. Fixed with adding
|
#587 I think my pull request will fix most Play 2.5 migration problems/ |
I am using securesocial 3.0-M4 with play 2.5.3 (although this also happens downgrading to 2.5.0). I am trying to customize the login controller as follows: class CustomLoginController @Inject() (override implicit val env: MyRuntimeEnvironment) extends BaseLoginPage When hitting the login page, I get is the following error:
Digging a bit I found that I tried to work around the problem by redefining the |
@fedeoasi Have you found a solution to the problem? I'm running into the same errors now and would really appreciate a pointer in the right direction Will this be fixed anytime soon? |
@markusz: I have reverted the play version used by my project to 2.4.6, the latest play version that still has I haven't tried it out, but after a quick look at #587 I think it will fix the problem because it upgrades the play dependency to 2.5.1 (in project/plugins.sbt). |
To make Securesocial java compatible with play 2.5, the BaseUserService methods would need to migrate from F.Promise to java.util.concurrent.CompletableStage (see https://www.playframework.com/documentation/2.5.x/JavaMigration25). This allows java users to implement an override of BaseUserService. |
+1, would love to see this happening! |
I just pushed 3.0-M6 and master with support for 2.5. Sample apps are working fine. Please open new issues if problems arise. Thanks! |
As far as I can see, and if you ignore deprecation, securesocial is compatible with Play 2.5 in scala. Java needs a couple of things rewritten. Is a 2.5 version planned?
The text was updated successfully, but these errors were encountered: