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

[JBEAP-10709] [ELY-1130] keystore entry alias is not necessary if the… #806

Merged
merged 1 commit into from May 11, 2017

Conversation

soul2zimate
Copy link
Contributor

…re is only one entry in keystore.
JBEAP issue: https://issues.jboss.org/browse/JBEAP-10709
ELY issue: https://issues.jboss.org/browse/ELY-1130

@darranl
Copy link
Contributor

darranl commented May 8, 2017

I would actually like a bigger review of the alias handling - it should be possible for a KeyManager to have multiple entries available to it and then based on the selected cipher suite a suitable entry is chosen. Server side we support filtering that allows for multiple aliases but we don't support this in the client.

@hkalina hkalina added the +1 HK label May 9, 2017
@soul2zimate
Copy link
Contributor Author

IIUC, the bigger review of alias handling means to add keys and signatures compatibility check with configured cipher suite into ConfigurationKeyManager?
About this case, this is in the keystore level when it parses alias and retrieves entry, the ConfigurationKeyManager builder also requires X509CertificateChainPrivateCredential instance (which is already parsed by the supplier) to build out the KeyManage.

@darranl
Copy link
Contributor

darranl commented May 10, 2017

Most of the cross referencing of ciphers and key types is actually handled by the KeyManager implementation.

Historically we have followed the pattern where we hard code one alias only from a KeyStore, that selected alias will be usable with a subset of all possible cipher suites.

However another possibility is the desire to use new cipher suites, in that case you may want two entries in the keystore one with keys compatible with the older cipher suites and one compatible with new.

You still only want a subset of entires from the KeyStore to be usable but we need a filter. We do have a filtering key store implementation that covers this server side but client side conversion to the credential would make this different.

@dmlloyd dmlloyd added the +1 DML label May 11, 2017
@dmlloyd dmlloyd merged commit 596f25e into wildfly-security:master May 11, 2017
@soul2zimate soul2zimate deleted the JBEAP-10709 branch May 12, 2017 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants