bol-van / zapret

DPI bypass multi platform
8.49k stars 639 forks source link

[uninstall_easy.sh] Не удаляет /opt/zapret/ #713

Closed Yoti closed 2 weeks ago

Yoti commented 2 weeks ago

Собственно, файл для удаления выполняет функцию отключения, не удаляя по факту сами файлы. Тестовая система OpenWrt 23.05.5 (r24106-10cc5fcd00).

HentayHentay commented 2 weeks ago

Судя по тому что там написано, происходит удаление службы и очистка ip/nf таблиц. Далее ручками уберись

Скорее сделано для того, что бы ты мог свои конфиги и листы куда-то сохранить

PS и почему все считают, что удаление - это помимо отключения функционала, ещё и уборка файлов следом....

PS2 Вот что делает удаление

    clear_ipset
service_remove_sysv
remove_openwrt_firewall
remove_openwrt_iface_hook
nft_del_table
restart_openwrt_firewall
crontab_del
konstantisu commented 2 weeks ago

Возможно, многие накушались Windows, где реестр загаживается не удаляющим за собой мусор софтом до переустановки системы :)? Или не хочет разбираться, проще переставить owrt с нуля, или искать мешающие "хвосты". Или не у всех роутеры с гиговыми флешками? Вариантов немало. Но да, главное чтобы конфиги удавалось возвращать в состояние "как было". Про пакеты и зависимости лучше не упоминать...

Yoti commented 2 weeks ago

PS и почему все считают, что удаление - это помимо отключения функционала, ещё и уборка файлов следом....

Install_easy разворачивает файлы в /opt. Обратное действие (uninstall_easy) должно эти файлы убирать по определению.

PS2 Вот что делает удаление

Я прочитал, что оно делает. И отписался о том, что оно не делает. Почему я считаю, что это баг? Смотри ответ на цитату выше.

eastone11 commented 2 weeks ago

PS и почему все считают, что удаление - это помимо отключения функционала, ещё и уборка файлов следом....

Install_easy разворачивает файлы в /opt. Обратное действие (uninstall_easy) должно эти файлы убирать по определению.

PS2 Вот что делает удаление

Я прочитал, что оно делает. И отписался о том, что оно не делает. Почему я считаю, что это баг? Смотри ответ на цитату выше.

разработчик видимо не согласен ) image

HentayHentay commented 2 weeks ago

@eastone11 А я уж думал, что никто в ридми не заглянет ....

eastone11 commented 2 weeks ago

@eastone11 А я уж думал, что никто в ридми не заглянет ....

с пылу с жара кстати)) image

Yoti commented 2 weeks ago

с пылу с жара кстати))

Видимо, это реакция на мой предыдущий репорт. Но да, последний коммит я не смотрел. Думаю, этого достаточно, чтобы закрыть issue.

bol-van commented 2 weeks ago

Сделано предложение удалить зависимости. По умолчанию - нет. Про снос /opt/zapret пишется, но не делается

Yoti commented 2 weeks ago

В контексте моей просьбы достаточно этого https://github.com/bol-van/zapret/commit/868d115c5d3e5f049d2588e905b2cfbdd105e33f#diff-2853695723668172fcde2eee778eaded1781661d07078a9700cae82dcba8bce8R77