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

Include root service output in debug logs #92

Closed
d4rken opened this issue Apr 3, 2023 · 1 comment · Fixed by #440
Closed

Include root service output in debug logs #92

d4rken opened this issue Apr 3, 2023 · 1 comment · Fixed by #440
Labels
c: IO SAF/Normal/Root access enhancement New feature, request, improvement or optimization Root

Comments

@d4rken
Copy link
Member

d4rken commented Apr 3, 2023

The current debug log only includes the log output from the main app process.
Root operations run in a different process and their log output is only available on ADB, when launched in debug mode.

While some of the root output is send back and logged, it could still make debugging some root related operations difficult.

Idea 1:
Write an extra log from the root process, set the permissions right and then include it in the final zip file.

Idea 2:
Two processes appending to the same logfile. This would probably cause corruption? 🤔

@d4rken d4rken added enhancement New feature, request, improvement or optimization c: IO SAF/Normal/Root access Root labels Apr 3, 2023
@dragonGod97677
Copy link

I prefer just including the root log in the final zip file rather appending it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: IO SAF/Normal/Root access enhancement New feature, request, improvement or optimization Root
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants