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

[2.1.x] ssl-config 0.6.0 (like akka 2.6.19) #638

Merged
merged 2 commits into from
Jan 16, 2022

Conversation

mkurz
Copy link
Member

@mkurz mkurz commented Jan 12, 2022

We need to upgrade ssl-config in play-ws 2.1.x to 0.6.0, because akka did the same a couple of days ago, and because Play's cron build for the 2.8.x branch uses the latest akka snapshot, the build is now broken: https://app.travis-ci.com/github/playframework/playframework/jobs/554697000#L1703

Exception in thread "specs2-2" java.lang.NoSuchMethodError: com.typesafe.sslconfig.ssl.SSLLooseConfig.allowWeakProtocols()Z

We need to wait for akka 2.6.19 however.

}
}
logger.warn(
"validateDefaultTrustManager is not doing anything since play-ws 2.1.8, it was useful only in Java 7 and below");
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mkurz
Copy link
Member Author

mkurz commented Jan 12, 2022

Also see discussion in akka/akka#30871 and akka/akka#31046

@mkurz mkurz changed the title ssl-config 0.6.0 (like akka 2.6.19) [2.1.x] ssl-config 0.6.0 (like akka 2.6.19) Jan 12, 2022
@mkurz
Copy link
Member Author

mkurz commented Jan 16, 2022

Let's use the current akka snapshot for now, cut a play-ws 2.1.8-RC1 release to unblock Play's 2.8.x branch

@mkurz mkurz marked this pull request as ready for review January 16, 2022 19:49
@mkurz
Copy link
Member Author

mkurz commented Jan 16, 2022

Because this was cherry-picked from the master branch and akka did the same upgrade to ssl-config 0.6.0 I will merge myself now so I can cut the RC

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

Successfully merging this pull request may close these issues.

1 participant