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

PKCS#11 URI and system configuration fixes #9

Merged
merged 5 commits into from
Apr 29, 2015
Merged

Conversation

dwmw2
Copy link
Contributor

@dwmw2 dwmw2 commented Dec 14, 2014

These patches make engine_pkcs11 accept key/cert specifications in the form of a PKCS#11 URI as described at https://tools.ietf.org/html/draft-pechanec-pkcs11uri-16

There is also a patch to make it automatically load p11-kit-proxy.so as its PKCS#11 provider instead of just failing if none is specified. This has the effect of making it Just Work™ with the system configuration for which modules should be present.

These are both important standardisation efforts to make PKCS#11 more usable on the desktop. We really need provider modules to be able to automatically install themselves with the system, and a consistent way of specifying objects within PKCS#11 tokens. Which is what we get from p11-kit, and the PKCS#11 URI, respectively.

Note that the first patch doesn't add a dependency on p11-kit libraries; it only uses pkg-config to determine the location of the p11-kit-proxy.so provider module — which loads all the providers which are registered in the system configuration, into slots of itself.

David Woodhouse added 4 commits January 8, 2015 09:46
In the general case, PKCS#11-aware applications should automatically be
using the tokens listed in the system's p11-kit configuration, rather than
needing to be explicitly pointed at a provider module.

We currently just *fail* if no provider is given. Fix that to use the
p11-kit-proxy.so module from p11-kit if it's detected.

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
@nmav
Copy link
Contributor

nmav commented Apr 2, 2015

+1

Signed-off-by: David Woodhouse <David.Woodhouse@intel.com>
@dwmw2 dwmw2 merged commit 0cc9d88 into OpenSC:master Apr 29, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants