-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Remove application.secret and play.crypto.secret support #9061
Conversation
Fixed playframework#8981 playframework#8981 Removed `play.crypto.secret` & `application.secret` leaving `play.http.secret.key` as the only way to set secret key
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.
Thanks a lot, @chipz.
See my comment. After that, I think this will be good to merge. :-)
} | ||
|
||
} | ||
|
||
trait SecretConfigurationParserSpec extends Specification { |
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.
We can change this trait to be a class and use play.http.secret.key
as the secretKey
value. And then remove ActualKeySecretConfigurationParserSpec
as well. :-)
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.
thanks for the review. done.
@@ -285,7 +285,7 @@ object HttpConfiguration { | |||
val Blank = """\s*""".r | |||
|
|||
val secret = | |||
config.getDeprecated[Option[String]]("play.http.secret.key", "play.crypto.secret", "application.secret") match { | |||
config.getDeprecated[Option[String]]("play.http.secret.key") match { |
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.
You can just use get
here now instead of getDeprecated
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.
good point
…nfigurationParserSpec from trait to class - Changed getDeprecated to get as we removed deprecated code
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.
LGTM.
Thank you, @chipz.
Fixes
Fixes #8981
Purpose
Removed
play.crypto.secret
&application.secret
leavingplay.http.secret.key
as the only way to set secret key