Implementation of encryption

[linocajo]

Hi,

I've struggling with a private tracker that recently moved to encryption.
I lost a lot of peers after the move. On the tracker forum, I was asked to sniff the announces of the  client to see how the crypto parameter was written.

So I tested the 3 possible Crypto QBT settings and sniffed the announces.

If setting Encryption Method =  Disable Encryption
then Announce has not parameter for Encryption. This is expected and is therefore correct.
Encryption Method = Prefer Encryption
then Announce has parameter: supportcrypto=1. This is expected and is therefore correct.
Encryption Method = Require Encryption
then Announce has parameter: supportcrypto=1. This is not expected. It should not be the same as Prefer Encryption, instead, it should be: requirecrypto=1

Setting "Prefer Encryption" solved my problem but I thought maybe I should share the unexpected outcome of "Require Encryption", maybe there is a bug somewhere.

cheers,