[Feature Request] Require Encryption

I would like to see the following feature in 2.0:

new ClientEngine(new EngineSettingsBuilder
{
    RequireEncryption = true
}.ToSettings());

This should behave like other torrent clients do. Don’t connect to a peer that doesn’t support encryption. The added value is that we can be sure that nobody can see the transfer except the receiving peer. Other torrent clients often offer the following options:

• No Encryption
• Prefer Encryption
• Require Encrytion

So perhaps it would be nice to deprecate PreferEncryption in favor of an enum:

new ClientEngine(new EngineSettingsBuilder
{
    EncryptionStyle = EncryptionStyle.Require
}.ToSettings());

enum EncryptionStyle {
  None,
  Prefer,
  Require
}

What do you think?

[Deathspike]

Oh! This wasn’t clear to me. I’m now using this as per your suggestion:

new ClientEngine(new EngineSettingsBuilder
{
    AllowedEncryption = EncryptionTypes.RC4Full | EncryptionTypes.RC4Header,
    PreferEncryption = true
}.ToSettings()

I think that dropping PreferEncryption would be better since AllowedEncryption already includes PlainText.

Your suggestion for AllowedEncryption with an order would be ideal for me! I really like that one. 😃

[Deathspike]

Thanks! Looks good to me!