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

Improve TLS identifier to include entire certificate chain #1163

Open
tsandall opened this issue Jan 15, 2019 · 2 comments
Open

Improve TLS identifier to include entire certificate chain #1163

tsandall opened this issue Jan 15, 2019 · 2 comments

Comments

@tsandall
Copy link
Member

tsandall commented Jan 15, 2019

Expected Behavior

Some users will want to have the entire certificate chain provided to the authz policy. One option would be to expose the entire certificate chain as the identity, however, this might be overly complex for most use cases. An alternative would be to let the identifier include multiple keys in addition to input.identity. In the x509 parsing built-ins we just JSON serialize the Go struct: https://github.com/open-policy-agent/opa/blob/master/topdown/crypto.go#L22. We could do the same here.

Actual Behavior

Today, the TLS identifier only extracts the subject.

@stale
Copy link

stale bot commented Nov 22, 2021

This issue has been automatically marked as inactive because it has not had any activity in the last 30 days.

@stale stale bot added the inactive label Nov 22, 2021
@stale stale bot removed the inactive label Dec 3, 2021
@stale
Copy link

stale bot commented Jan 2, 2022

This issue has been automatically marked as inactive because it has not had any activity in the last 30 days.

@stale stale bot added the inactive label Jan 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant