search

pavel-pimenov / flylinkdc-r5xx

flylinkdc-r5xx
GNU General Public License v2.0
54 stars 26 forks source link

Установить приоритет наборов шифров хотя бы для TLS 1.3 #1737

Closed Tantrix closed 4 years ago

Tantrix commented 4 years ago

Как здесь: https://github.com/airdcpp/airgit/blob/master/airdcpp/airdcpp/CryptoManager.cpp#L113-L138

TLS_AES_128_GCM_SHA256 требует меньше ресурсов нежели дефолтный на данный момент TLS_AES_256_GCM_SHA384 при приемлемом уровне стойкости.

pavel-pimenov commented 4 years ago

У флая еще есть ALPN

https://github.com/pavel-pimenov/flylinkdc-r5xx/blob/158881e55cb3495f0f534ecdb6e749d3f246c4b4/client/CryptoManager.cpp#L103-L106

там тоже нужно звать SSL_CTX_set_ciphersuites по идее? ты сам проверял как работает после патча?

Tantrix commented 4 years ago

Смотря что за идея (: ALPN-то нету больше нигде в клиентах, насколько я знаю. И ты учти, что шифронаборы для TLS 1.2 и TLS 1.3 несовместимы.

Так-то когда клиенты договорились о версии шифрования (будет использоваться максимальная доступная, понятно), они будут обмениваться шифрами согласно списку. Выше = приоритетней.

AirDC++по дефолту выбирает 128/256, но соглашается на 256/384. Все довольны.

pavel-pimenov commented 4 years ago

Ок. заберу тогда кусок от Air - бетку сделаю сегодня вечером.

pavel-pimenov commented 4 years ago

Сделал так чтобы в окне передаче показывался шифр при котором прошел SSL_Accept изображение