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
ISPN-4306 Properly create the RealmUser when obtaining credentials over GSSAPI #2578
ISPN-4306 Properly create the RealmUser when obtaining credentials over GSSAPI #2578
Conversation
@tristantarrant Compilation error on CI:
|
This needs the WildFly pull to happen On Mon, Jun 2, 2014 at 10:21 AM, Galder Zamarreño notifications@github.com
|
Needs rebasing... |
Rebased |
Let's integrate #2579 first then and make sure the test is fixes... |
Rebase |
Rebased |
@@ -146,6 +146,9 @@ | |||
@Message(id = 10021, value = "Invalid Strength value: %s") | |||
IllegalStateException invalidStrength(String strengthValue); | |||
|
|||
@Message(id = 10022, value = "Endpoint '%s' requires Client Certificates, but no Trust Store is available in realm '%s'") |
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.
can you keep the old id in the old message and set the new id to the new message?
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.
That id was mistakenly introduced during the 7.0.0.Alpha cycle, so I'm aligning against the product so that we remain consistent.
just a small comment:
|
Updated, now it compiles. The test is HotRodKrbAuthIT.java but it won't work until #2664 has been merged too |
hmmm so it is better to wait for #2664 |
#2664 has been integrated now |
rebased |
Integrated |
https://issues.jboss.org/browse/ISPN-4306