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

Misc. patches to be cherry-picked if need be #143

Merged
merged 3 commits into from Jun 7, 2013

Conversation

Projects
None yet
2 participants
@laarmen
Contributor

laarmen commented Jun 7, 2013

These patches fix several little things unrelated to one another, feel free to cherry-pick what is of interest.

laarmen added some commits Jun 7, 2013

Listify the endpoints in tail_messages
In some places of the code, single enpoints are transformed into lists,
it seemed only fair to do this here as well.
Stub out the x509 backend when m2crypto isn't available
Since m2rypto and m2ext dependencies are not enforced, it is a likely
scenario that someone install fedmsg without them, which leads to crash,
cries and unhappiness. With this commit, fedmsg-logger can still run
while still complaining that it cannot check the signatures.
@ralphbean

This comment has been minimized.

Show comment
Hide comment
@ralphbean

ralphbean Jun 7, 2013

Contributor

👍

Good stuff. Thank you!

Contributor

ralphbean commented Jun 7, 2013

👍

Good stuff. Thank you!

ralphbean added a commit that referenced this pull request Jun 7, 2013

Merge pull request #143 from laarmen/bugfixes
Misc. patches to be cherry-picked if need be

@ralphbean ralphbean merged commit 34d8ecb into fedora-infra:develop Jun 7, 2013

1 check passed

default The Travis CI build passed
Details

@laarmen laarmen deleted the laarmen:bugfixes branch Jun 7, 2013

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