[DRAFT] fix(portshide): fix push-notifications#122
Conversation
|
@AndreyGubin это поведение действительно чинит Ozon? |
|
@okhsunrog Да. Чинит не только Озон, но и другие приложения. Но у меня нет уверенности что это не раскроет порты для сканирующих приложений. Могу снять метку draft. |
|
@AndreyGubin просьба поребейзить этот PR. И можете описать, как мне воспроизвести проблему? В каких приложениях наблюдается? У меня с Озон и portshide приходят уведомления - не могу воспроизвести. Если смогу воспроизвести проблему и точно исследовать причину - я начну работать над точным фиксом, чтобы не пальцем в небо. Если честно, вот этот подход мне пока не нравится |
|
Мне тоже не нравится подход, поэтому и поставил метку черновика. Воспроизводится со всеми приложениями, которым выставлен флаг P. Ради интереса проверял на телеграме, поставил флаг P, пуши перестали поступать (до фикса). Воспроизвести довольно просто: Установил portshide из releases, поставил телеграму флаги L, K, O и P, перезапустил телеграм, отправил себе сообщение с другого аккаунта, пуш не отобразился. Если установить мой фикс, то пуши приходят стабильно. |
|
@AndreyGubin окей. Завтра сделаю релиз новый большой - потестируешь там. Если воспроизводится так же - буду копать глубже |
DO NOT MERGE. NEEDS TESTING AND REVIEW.
WHY:
Current app breaks push-notifications with flag "P" is enabled.
Android using localhost to receive notifications. As a result apps can't sync data in background.
SUMMARY:
Changed behaviour from "block all localhost connections" to "block rapid/burst localhost connections (bruteforce ports)"
Closes #121