Skip to content

[DRAFT] fix(portshide): fix push-notifications#122

Draft
AndreyGubin wants to merge 1 commit into
okhsunrog:mainfrom
AndreyGubin:fix/restore-push-notifications
Draft

[DRAFT] fix(portshide): fix push-notifications#122
AndreyGubin wants to merge 1 commit into
okhsunrog:mainfrom
AndreyGubin:fix/restore-push-notifications

Conversation

@AndreyGubin

Copy link
Copy Markdown

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

@okhsunrog

Copy link
Copy Markdown
Owner

@AndreyGubin это поведение действительно чинит Ozon?

@AndreyGubin

Copy link
Copy Markdown
Author

@okhsunrog Да. Чинит не только Озон, но и другие приложения. Но у меня нет уверенности что это не раскроет порты для сканирующих приложений. Могу снять метку draft.

@okhsunrog

Copy link
Copy Markdown
Owner

@AndreyGubin просьба поребейзить этот PR. И можете описать, как мне воспроизвести проблему? В каких приложениях наблюдается? У меня с Озон и portshide приходят уведомления - не могу воспроизвести. Если смогу воспроизвести проблему и точно исследовать причину - я начну работать над точным фиксом, чтобы не пальцем в небо. Если честно, вот этот подход мне пока не нравится

@AndreyGubin

Copy link
Copy Markdown
Author

Мне тоже не нравится подход, поэтому и поставил метку черновика. Воспроизводится со всеми приложениями, которым выставлен флаг P. Ради интереса проверял на телеграме, поставил флаг P, пуши перестали поступать (до фикса).
Xiaomi 15, Android 15, kmod, в magisk приложения с проблемными пушами не скрыты, прошивка xiaomi.eu. С дебагом и перехватом трафика на таком уровне (для проверки куда приложение стучится и почему не видит пуши) я раньше не работал, но было бы интересно понять причину и научиться.

Воспроизвести довольно просто: Установил portshide из releases, поставил телеграму флаги L, K, O и P, перезапустил телеграм, отправил себе сообщение с другого аккаунта, пуш не отобразился. Если установить мой фикс, то пуши приходят стабильно.

@okhsunrog

Copy link
Copy Markdown
Owner

@AndreyGubin окей. Завтра сделаю релиз новый большой - потестируешь там. Если воспроизводится так же - буду копать глубже

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: не работают push-уведомления

2 participants