Skip to content
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 scala 2.13? #265

Closed
louisjiang opened this issue Sep 20, 2019 · 21 comments
Closed

Support for scala 2.13? #265

louisjiang opened this issue Sep 20, 2019 · 21 comments
Milestone

Comments

@louisjiang
Copy link

Support for scala 2.13?

@leleuj
Copy link
Member

leleuj commented Sep 20, 2019

Not yet. Feel free to submit a contribution.

@louisjiang
Copy link
Author

Dependent on external project deadbolt-2-scala
mkurz/deadbolt-2-scala#50

@leleuj
Copy link
Member

leleuj commented Sep 25, 2019

OK. Let's wait for that then.

@aeismann
Copy link

aeismann commented Oct 8, 2019

FYI: @louisjiang That dependency has moved forward now:
mkurz/deadbolt-2-scala#50

@pbyrne84
Copy link

pbyrne84 commented Jan 2, 2020

Deadbolt scala does not look like it has been properly published for 2.11
https://repo1.maven.org/maven2/be/objectify/deadbolt-scala_2.11/2.7.1/
versus
https://repo1.maven.org/maven2/be/objectify/deadbolt-scala_2.13/2.7.1/

:(

Things seem to of been moved/renamed internally in deadbolt so it may be worth waiting for play 2.8 support as it looks like it has dropped 2.11 simplifying things.

@leleuj
Copy link
Member

leleuj commented Jan 6, 2020

@pbyrne84 Just to be sure: you suggest to first move to Play 2.8 and then remove Scala 2.11 and add Scala 2.13, right?

@pbyrne84
Copy link

Sorry for not getting back sooner. Well the move to play 2.8 will remove 2.11 as it is no longer supported. The deadbolt changes would now block you and would require incompatible versions of deadbolts to be running across different Scala versions in your build. That does not sound like fun to me.

There is an issue raised for the missing 2.11 build
mkurz/deadbolt-2-scala#51
I assume it is right place as it mentions 2.7.1

@mkurz
Copy link

mkurz commented Feb 10, 2020

I released deadbolt 2.7.1 for Scala 2.11 now:
https://repo1.maven.org/maven2/be/objectify/deadbolt-scala_2.11/2.7.1/
Seems like a problem occured when released last year.

@mkurz
Copy link

mkurz commented Feb 10, 2020

Oh yes and deadbolt 2.8.0 for Scala 2.12 and 2.13 is released now as well. Have fun!

@mkurz mkurz mentioned this issue Feb 10, 2020
1 task
@leleuj
Copy link
Member

leleuj commented Feb 10, 2020

Thanks

@leleuj
Copy link
Member

leleuj commented Feb 12, 2020

@mkurz Just to be sure: for Play 2.7, I must use Deadbolt 2.7 and for Play 2.8, I must use Deadbolt 2.8, right?

@mkurz
Copy link

mkurz commented Feb 12, 2020

@leleuj Yes, that's correct.

@mkurz
Copy link

mkurz commented Feb 12, 2020

Because from deadbolt 2.7 to deadbolt 2.8 nothing really changed, you may be able to use deadbolt 2.7 with Play 2.8. I think that should work as well given that deadbolt 2.8 is just compiled against Play 2.8 without any changes in its source code. However the safest way is probably to just use deadbolt 2.8.
deadbolt 2.7 and deadbolt 2.8 do not differ in features or anything else, it's just recompiled.

@leleuj
Copy link
Member

leleuj commented Feb 12, 2020

@mkurz Thanks for confirming this.

I was in v2.6.1 and I have compilation issues with v2.7.1. I guess things have changed: is there a migration guide somewhere to help me?

@mkurz
Copy link

mkurz commented Feb 12, 2020

Not really a migration guide, but maybe it helps: mkurz/deadbolt-2-java#89

@leleuj
Copy link
Member

leleuj commented Feb 12, 2020

I saw this doc, but it won't be enough for me.

I have taken a look at the documentation (https://deadbolt-java.readme.io/) and I haven't been able to find the guide for a release after the version 2.6.3: am I missing something?

@leleuj
Copy link
Member

leleuj commented Feb 12, 2020

OK. I see the problem: I'm relying on Http.Context which has been replaced by Http.RequestHeader. This is something I planned to do in play-pac4j even in Play 2.7, so I will start by that. The upgrade should be straightforward after that.

Thanks for your help.

@mkurz
Copy link

mkurz commented Feb 12, 2020

@leleuj If you need help along the way let me know.

@leleuj
Copy link
Member

leleuj commented Feb 13, 2020

Thanks for proposing. Almost all work was on my side. As soon as I used RequestHeader instead of Http.Context, the Deadbolt upgrade became easy.

@mkurz
Copy link

mkurz commented Feb 13, 2020

Great!

@leleuj
Copy link
Member

leleuj commented Feb 13, 2020

I just added the Scala 2.13 support as well.

@leleuj leleuj closed this as completed Feb 13, 2020
@leleuj leleuj added this to the 9.0.0 milestone Feb 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants