fedora-infra / fedora-tagger

Small tagger app for Fedora Community
https://apps.fedoraproject.org/tagger/
15 stars 23 forks source link

Floor the rating for fedmsg messages. #185

Closed ralphbean closed 9 years ago

ralphbean commented 9 years ago

This, like in fedora-infra/fedmsg#201, is just to reduce erroneous fedmsg validation failures. Without this, the rating gets encoded on one machines as 0.9111111111112 on one machine and as 0.91111111111 on another. Those are not the same strings, and so the signature computation produces different signatures -- recipients think that the message is "invalid".

pypingou commented 9 years ago

:+1: :)

ralphbean commented 9 years ago

Thanks @pypingou!