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

Document turning off validation for other busses. #387

Merged
merged 1 commit into from Nov 10, 2016

Conversation

Projects
None yet
2 participants
@ralphbean
Contributor

ralphbean commented Nov 10, 2016

There's usually no point in cryptographically validating messages from other
busses. They likely won't bear message certificates and signatures. Other
busses usually use TLS on the connection itself for authentication and
authorization.

Document turning off validation for other busses.
There's usually no point in cryptographically validating messages from other
busses.  They likely won't bear message certificates and signatures. Other
busses usually use TLS on the connection itself for authentication and
authorization.
@ralphbean

This comment has been minimized.

Contributor

ralphbean commented Nov 10, 2016

@mprahl, can you review?

@mprahl

mprahl approved these changes Nov 10, 2016

LGTM

@ralphbean ralphbean merged commit c176579 into fedora-infra:develop Nov 10, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ralphbean ralphbean deleted the ralphbean:feature/no-validation-for-other-busses branch Nov 10, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment