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

GSSAPI tests and fixes #34

Merged
merged 1 commit into from
Oct 7, 2014
Merged

GSSAPI tests and fixes #34

merged 1 commit into from
Oct 7, 2014

Conversation

thaarok
Copy link

@thaarok thaarok commented Oct 5, 2014

Changes in production code:

  • Fixed AbstractGssapiMechanism.DEFAULT_MAX_BUFFER_SIZE (3 octets)
  • Fixed bugs in networkOrderBytesToInt(), intToNetworkOrderBytes() and parsePreferredQop()
  • ChallengeResponseState should return byte[0] instead of null
  • SecurityLayerAdvertiser: gssContext.getWrapSizeLimit() moved on the
    opposite side of communication
  • getNegotiatedProperty now returns Strings
  • Allowed disabling of mutual authentication by client props (if not auth-int/auth-conf)

* Fixed AbstractGssapiMechanism.DEFAULT_MAX_BUFFER_SIZE (3 octets)
* Fixed bugs in networkOrderBytesToInt() and intToNetworkOrderBytes()
* ChallengeResponseState should return byte[0] instead of null
* SecurityLayerAdvertiser: gssContext.getWrapSizeLimit() moved on the
   opposite side of communication
* getNegotiatedProperty now returns Strings
* Allowed disabling of mutual authentication by client props
gssContext.requestMutualAuth(true);
}
boolean requestMutualAuth = serverAuth || mayRequireSecurityLayer;
log.trace("Setting requering mutual authentication to "+Boolean.toString(requestMutualAuth));
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry missed this one, this needs to be a tracef call otherwise we have a redundant String concatenation.

@darranl
Copy link
Contributor

darranl commented Oct 7, 2014

I will go ahead and merge, I will add a commit myself for the two comments I just added.

@darranl darranl merged commit 41becb8 into wildfly-security:master Oct 7, 2014
darranl added a commit that referenced this pull request Oct 7, 2014
@darranl
Copy link
Contributor

darranl commented Oct 7, 2014

Merged

@thaarok thaarok deleted the gssapi-test-fix branch October 9, 2014 08:25
fjuma added a commit to fjuma/wildfly-elytron that referenced this pull request Dec 14, 2018
[ELY-1711] Create a new 'wildfly-elytron-auth-util' module.
darranl pushed a commit to darranl-archive/2020-wildfly-elytron that referenced this pull request Jan 7, 2019
[ELY-1043] PicketBox Vault 2.0 conversion
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants