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

filter: New virtual wrapper for access control #26

Merged
merged 1 commit into from
Feb 8, 2017

Conversation

ueno
Copy link
Member

@ueno ueno commented Dec 14, 2016

This is a dependency of #15. When we forward the Unix domain socket to another machine, it would be good to limit the access to a specific token, rather than exposing the entire module and all the tokens managed by that module.

Maybe this could be used together with the PKCS#11 URIs, something like:

p11-kit server <token URI> ...

instead of:

p11-kit server <module filename> ...

@ueno
Copy link
Member Author

ueno commented Dec 14, 2016

Other use-case could be to blacklist certain driver in a PKCS#11 module. That could be done through a module configuration, something like "allow-token" or "deny-token" with a PKCS#11 URI as the argument.

@ueno ueno added this to the future milestone Dec 14, 2016
@ueno
Copy link
Member Author

ueno commented Dec 19, 2016

Rewritten using the new P11KitIter API (#28) for enumerating tokens.

@ueno ueno force-pushed the wip/dueno/filter branch 2 times, most recently from 30257bc to 401ae0b Compare December 25, 2016 01:50
@ueno ueno force-pushed the wip/dueno/filter branch 2 times, most recently from d376a95 to c4aec1d Compare January 9, 2017 09:16
@ueno ueno force-pushed the wip/dueno/filter branch 2 times, most recently from 3416a2f to 5134eb2 Compare January 24, 2017 13:56
@ueno ueno merged commit fd9b5c1 into p11-glue:master Feb 8, 2017
@ueno
Copy link
Member Author

ueno commented Feb 8, 2017

Merged this, as it is merely an internal change.

@ueno ueno deleted the wip/dueno/filter branch February 16, 2017 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant