Skip to content

Feature/1376/logging and reporting#6373

Open
mahibi wants to merge 1 commit into
masterfrom
feature/1376/loggingAndReporting
Open

Feature/1376/logging and reporting#6373
mahibi wants to merge 1 commit into
masterfrom
feature/1376/loggingAndReporting

Conversation

@mahibi

@mahibi mahibi commented Jun 19, 2026

Copy link
Copy Markdown
Collaborator

This PR will add:

  • file logging
  • view logfiles in settings
  • option to enable advanced logging (Level.DEBUG) or disable advanced logging (Level.INFO)
  • crash screen to catch unhandled exceptions
  • Share logfile / crash report via email or download as ZIP file

This PR does not yet replace all Log command with the new logger. This should be done in separate PRs while also cleaning up unnecessary Log commands.

🖼️ Screenshots

Settings screen

grafik grafik

Logs screen

grafik grafik

Crash reporting screen

grafik

Testing

To test the crash screen, just add

throw RuntimeException("Test crash")

somewhere in the code and trigger it.

🚧 TODO

  • set predefined mail address by configs
  • create own email address for talk android
    • set it as predefined mail

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@mahibi mahibi self-assigned this Jun 19, 2026
@mahibi mahibi added the 2. developing Work in progress label Jun 19, 2026
@github-actions

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/27824399768/artifacts/7749295330
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@github-actions

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/27842498569/artifacts/7756417285
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

@mahibi mahibi force-pushed the feature/1376/loggingAndReporting branch 3 times, most recently from 2bff679 to 141de3f Compare June 22, 2026 12:20
@mahibi mahibi force-pushed the feature/1376/loggingAndReporting branch 2 times, most recently from f9fa3b0 to bef07f0 Compare June 23, 2026 17:34
@mahibi mahibi added 3. to review Waiting for reviews and removed 2. developing Work in progress labels Jun 23, 2026
@mahibi mahibi marked this pull request as ready for review June 23, 2026 17:57
@mahibi mahibi added this to the 24.1.0 milestone Jun 23, 2026
  Introduces a structured local file logger with configurable log levels
  (INFO/DEBUG), advanced HTTP request logging, and tagged loggers that
  auto-capture the calling class name.

  Adds crash reporting via a custom UncaughtExceptionHandler that captures
  a diagnosis snapshot and log files when the app crashes, and a
  ShowErrorActivity to surface the crash report.

  Adds a "Diagnosis & Logs" section to Settings with:
  - A log viewer screen (LogsActivity) with an inline advanced logging toggle
    and privacy notice when advanced logging is enabled
  - A "Share diagnosis report" action that bundles logs and diagnosis data
    for email, ZIP download, or GitHub issue creation
  - A red warning subline under "Logs" when advanced logging is active
  - The report email extracted to setup.xml as `nc_report_email` for branding

  Extracts diagnosis data to DiagnosisReport, removes sharing actions from
  individual screen menus (DiagnosisActivity, LogsActivity), and consolidates
  all export flows through SettingsActivity.

AI-assistant: 2.1.183 (Claude Code) (Claude Sonnet 4.6)

Signed-off-by: Marcel Hibbe <dev@mhibbe.de>
@mahibi mahibi force-pushed the feature/1376/loggingAndReporting branch from bef07f0 to 32d6b30 Compare June 23, 2026 18:19
@github-actions

Copy link
Copy Markdown
Contributor

APK file: https://github.com/nextcloud/talk-android/actions/runs/28047427695/artifacts/7829811461
To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.
qrcode (please click on link to get QR code displayed)

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

Labels

3. to review Waiting for reviews AI assisted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant