~AreEnn
~orignal
DUHOVKIN_
Guest7184
Most2
Nausicaa
Nikat
Ruskoye_911
Vort
Xeha
acetone
anon3
b3t4f4c3
fidoid
karamba_i2p
monkey
nemiga
not_bob_afk
plap
poriori
profetikla
qend
segfault
soos
teeth
tetrimer_
trust
uis
un
unlike
user
weko_
whothefuckami_
orignal
сделал AES
Vort
orignal: параметры из конфига ещё выкинуть надо: github.com/PurpleI2P/i2pd/blob/786da057f2ce74a1ebffcf4fb2951944e8fac618/libi2pd/api.cpp#L40-L42
Vort
и из документации. и из системы сборки
camellia
Blinded message
orignal
да ты прав и из мейфайла
orignal
и тесты добавить
orignal
то есть они есть просто надо закоомитить
camellia
Blinded message
onon
Я кажется понял, почему у меня pastebin не работал
onon
Срабатываля анти дудос защита
onon
Которую лось недавно сделал
onon
Когда мы на SYNACK не получаем ответа
onon
Я погонял с логированием, у меня так довольно часто получается почему-то
onon
Может где-то в логике косяк
onon
Понятно, если я не "новый" клиент, он берёт RTT из ShardRoutingPath и от него считает RTO, и если я не успеваю за этот RTO ответить он закрывает стрим.
onon
Надо подумать, и ,возможно, выключить RTT и RTO из ShardRoutingPath
onon
Я уже говорил, что мн не нравится этот SharedRoutingPath. Он полезен, если я клиент, и отправляю только акки, тогда выбирать один и тот же проверенный маршрут оправданно.
onon
Ну или если я сервер, и отправляю мало данных на каждый стрим.
onon
А когда много данных - тогда возникает проблема. Маршрут перегружается.
onon
Может сделать ограничение на одновременное использование одного маршрута для определённого количества стримов...
onon
Если превышает - уже выбирать другой
onon
В общем, пока предлагаю убрать пересчёт RTO из SharedRoutingPath. Это ни на что не повлияет, но хотя бы уберёт этот досадный баг с закрытием стрима.
onon
Streaming.cpp 1150 //m_RTO = std::max (MIN_RTO, (int)(m_RTT * 1.3 + m_Jitter)); // TODO: implement it better
orignal
заметь что я сделал как ты сказал
orignal
что надо сделать то?
onon
Ну так оно и работает для новых клиентов, котрыми он спамил
onon
А вот для уже существующих сессий получается вот так
orignal
ну так что поменять то?
onon
> Streaming.cpp 1150 //m_RTO = std::max (MIN_RTO, (int)(m_RTT * 1.3 + m_Jitter)); // TODO: implement it better
onon
Закомментировать или убрать
orignal
и как коммит назвать?
onon
Fixing a bug with an unexpected stream closing
onon
Что то вроде этого наверное
orignal
погодт
orignal
m_RTT = routingPath->rtt;
orignal
мы RTT меняем а RTO нет?
onon
Ну дальше там пересчитывается
onon
Или про что ты
onon
Нам вообще этот RTT не особо нужен
onon
Потому что он скорее всего не актуален
orignal
счас залью
onon
Но может иногда быть полезен если мы много мелких картинок например шлём
onon
Он тогда будет быстрее стартовать
onon
В общем думать нужно
orignal
смотри
orignal
правильно сделал?
onon
Угу
orignal
выпилил везде AESNI