Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Обновление BrightSide #81

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

Stampoo
Copy link

@Stampoo Stampoo commented Jul 20, 2021

Добавил новые директории для проверки на Jailbreak в BrightSide

@Stampoo Stampoo self-assigned this Jul 20, 2021
Copy link
Contributor

@chausovSurfStudio chausovSurfStudio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну и подтянуть бы последние изменения

Utils/Utils/BrightSide/BrightSide.swift Outdated Show resolved Hide resolved
@chausovSurfStudio
Copy link
Contributor

ну и Trailing Whitespace Violation кажись там есть, тоже бы поправить

@Stampoo Stampoo force-pushed the Update-BrightSide-utility branch 2 times, most recently from a44aa3c to ed480c6 Compare March 19, 2022 08:56
@Stampoo Stampoo requested review from chausovSurfStudio and removed request for LastSprint March 19, 2022 08:59
+ suspiciousAppsDir
+ suspiciousSystemDir
return jailbreakRelativelyFilesAndPaths
.allSatisfy(FileManager.default.fileExists(atPath:))
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

а тут логика точно не поменялась? раньше, получается, true возвращалось если хотя бы по одному из путей файл был, ну и в комменте так и указано к методу) а теперь - true ведь будет, если allSatisfy? -_-

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

как помню из переписки - Илья сказал что да, косяк есть, но он хотел покрыть тестами, чтобы получить ошибку на них, и исправить

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поправил

@KASAFF KASAFF self-assigned this Sep 10, 2024
@KASAFF
Copy link

KASAFF commented Sep 11, 2024

Добавлено еще больше директорий файлов, проверок.

Протестил логику на Зените, все отрабатывает корректно

По поводу Unit тестов, я не совсем понимаю нужны ли они тут? По сути вариант теста это замокать Filemanager и его ответы, не совсем понятно что нам дадут эти тестыы

Передвинул проверку на симулятор в конец, тк при джейлбрейке проверки можно обойти эти проверки, и тесты остальные будут бесполезны.

  • Некоторые директории доступны на симе, поэтому закрыл их проверкой на !isSimulator

Важно: Для открытия URLScheme необходимо добавить схемы в Plist, иначе 1 проверка с UIApplication.shared.canOpenURL(URL) будет всегда возвращать false

<key>LSApplicationQueriesSchemes</key>
<array>
        <string>cydia</string>
        <string>filza</string>
        <string>undecimus</string>
        <string>zbra</string>
        <string>sileo</string>
</array>

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.

3 participants