-
Notifications
You must be signed in to change notification settings - Fork 263
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
[Ely 584] used methods, AuthenticationConfiguration.EMPTY is flaged as deprecated #819
Conversation
Can one of the admins verify this patch? |
We require that all contributions be made under the terms of the MIT License, http://www.opensource.org/licenses/mit-license.php. Do you agree to the these terms? |
Would you be able to update either your profile or reply here with your name, so we know who this is coming from? We need this for our records :) |
This is OK to test. |
I agree.
(Sure, I'm gonna go fill out the profile right now :) )
…On Thursday, May 18, 2017, Kabir Khan ***@***.***> wrote:
Would you be able to update either your profile or reply here with your
name, so we know who this is coming from? We need this for our records :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#819 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AZPTl-Vb1y2v4y8O6dEtLG1rHwvkgHfBks5r7BOpgaJpZM4Nd3gG>
.
|
Works now. |
This is OK to test |
Changes requested by @darranl : as a later API change there is a big risk that it could affect WildFly Core and WildFly and any clients we don't know about that may have used the field. It would be safer to leave the field public but flag it as deprecated. |
95c8ecf
to
2de0a77
Compare
this is ok to test |
Added method empty() for AuthenticationConfiguration to make things consistent.