Skip to content

Disable file log writer on per-oreo Android versions#1264

Merged
luhmirin-s merged 1 commit into
release/2025.2.0from
hotfix/log-writer-fix
Jul 15, 2025
Merged

Disable file log writer on per-oreo Android versions#1264
luhmirin-s merged 1 commit into
release/2025.2.0from
hotfix/log-writer-fix

Conversation

@luhmirin-s
Copy link
Copy Markdown
Contributor

@luhmirin-s luhmirin-s commented Jul 15, 2025

JIRA ticket
Will be released in: 2025.2.0

Root cause analysis (for bugfixes only)

First known affected version: 2025.1.0

  • Log writer internally references Java 8 time API which is only available starting from Android 8. This causes NoClassDefFoundError exception on app start.

Notable changes

  • Only initialise the log writer starting from Android 8 (API 26)

Testing guidance

  • Install the release build of the app on Android 7 device - it should work, file logs should not be available.
  • Do the same on Android 8 - file logs should be available.

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@cla-bot cla-bot Bot added the ... label Jul 15, 2025
@luhmirin-s luhmirin-s marked this pull request as ready for review July 15, 2025 06:42
@luhmirin-s luhmirin-s requested review from a team, BurningAXE, TristramN, alex-vt, alexandr-simprints, meladRaouf and ybourgery and removed request for a team July 15, 2025 06:42
@sonarqubecloud
Copy link
Copy Markdown

@luhmirin-s luhmirin-s merged commit 844168c into release/2025.2.0 Jul 15, 2025
20 of 21 checks passed
@luhmirin-s luhmirin-s deleted the hotfix/log-writer-fix branch July 15, 2025 12:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants