feat: show security providers info screen (WPB-27916) by ohassine · Pull Request #5156 · wireapp/wire-android

ohassine · 2026-08-11T13:08:54Z

https://wearezeta.atlassian.net/browse/WPB-27916

PR Submission Checklist for internal contributors

The PR Title
- conforms to the style of semantic commits messages¹ supported in Wire's Github Workflow²
- contains a reference JIRA issue number like SQPIT-764
- answers the question: If merged, this PR will: ... ³
The PR Description
- is free of optional paragraphs and you have filled the relevant parts to the best of your ability

What's new in this PR?

Issues

Show security providers info screen in debug settings screen

Screen_recording_20260811_122754.mp4

Needs releases with:

GitHub link to other pull request

Testing

Test Coverage (Optional)

I have added automated test to this contribution

How to Test

Briefly describe how this change was tested and if applicable the exact steps taken to verify that it works as expected.

Notes (Optional)

Specify here any other facts that you think are important for this issue.

Attachments (Optional)

Attachments like images, videos, etc. (drag and drop in the text box)

PR Post Submission Checklist for internal contributors (Optional)

Wire's Github Workflow has automatically linked the PR to a JIRA issue

PR Post Merge Checklist for internal contributors

If any soft of configuration variable was introduced by this PR, it has been added to the relevant documents and the CI jobs have been updated.

References

https://sparkbox.com/foundry/semantic_commit_messages
https://github.com/wireapp/.github#usage
E.g. feat(conversation-list): Sort conversations by most emojis in the title #SQPIT-764.

…-info-screen # Conflicts: # app/src/main/kotlin/com/wire/android/di/metro/WireMetroViewModelBindings.kt # app/src/main/kotlin/com/wire/android/ui/debug/DebugInfoViewModelFactory.kt

codecov · 2026-08-11T13:47:04Z

Codecov Report

❌ Patch coverage is 0% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 52.41%. Comparing base (ed6a026) to head (c74c84b).

Files with missing lines	Patch %	Lines
...oid/ui/debug/securityproviders/AppPathsProvider.kt	0.00%	16 Missing ⚠️
...ug/securityproviders/SecurityProvidersViewModel.kt	0.00%	10 Missing ⚠️
...oid/ui/debug/securityproviders/SecurityProvider.kt	0.00%	8 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #5156      +/-   ##
===========================================
- Coverage    52.49%   52.41%   -0.08%     
===========================================
  Files          664      667       +3     
  Lines        23812    23846      +34     
  Branches      3929     3932       +3     
===========================================
  Hits         12499    12499              
- Misses       10152    10186      +34     
  Partials      1161     1161

Files with missing lines	Coverage Δ
...oid/ui/debug/securityproviders/SecurityProvider.kt	`0.00% <0.00%> (ø)`
...ug/securityproviders/SecurityProvidersViewModel.kt	`0.00% <0.00%> (ø)`
...oid/ui/debug/securityproviders/AppPathsProvider.kt	`0.00% <0.00%> (ø)`

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed6a026...c74c84b. Read the comment docs.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

github-actions · 2026-08-11T13:52:04Z

Test Results

2 265 tests +25 2 265 ✅ +25 2m 23s ⏱️ -3s
250 suites + 5 0 💤 ± 0
250 files + 5 0 ❌ ± 0

Results for commit f1c344e. ± Comparison against base commit 1a626ed.

♻️ This comment has been updated with latest results.

MohamadJaara · 2026-08-11T15:42:05Z

this will show all providers which is not the intendet behaivour
what is needed it to show the provider for each action
like when using SecureRandom() from kalium or the app waht is the provider at run time

ohassine · 2026-08-12T07:40:15Z

this will show all providers which is not the intendet behaivour what is needed it to show the provider for each action like when using SecureRandom() from kalium or the app waht is the provider at run time

The ticket asks for a section that "displays entries for security providers," which is what this implements. the installed provider list with their entries

What you're describing is different. Happy to build it, but could you update the AC since the scope changed?

yamilmedina · 2026-08-12T16:17:28Z

this will show all providers which is not the intendet behaivour what is needed it to show the provider for each action like when using SecureRandom() from kalium or the app waht is the provider at run time

The ticket asks for a section that "displays entries for security providers," which is what this implements. the installed provider list with their entries

What you're describing is different. Happy to build it, but could you update the AC since the scope changed?

Good point, my bad I oversaw some info, I've updated the ticket(s) with a couple of things to consider:

Renaming the Security Providers screen and section to Security Diagnostics since this was misleading in the direction of displaying security providers. https://wearezeta.atlassian.net/browse/WPB-27916
The actual list of Security providers is now described in https://wearezeta.atlassian.net/browse/WPB-27920 under section entropy providers

yamilmedina · 2026-08-13T12:23:28Z

    <string name="debug_settings_force_repair_faulty_keys" translatable="false">Reset</string>
    <string name="debug_settings_feature_flags" translatable="false">Feature Flags</string>
    <string name="debug_settings_conversation_crypto_stats" translatable="false">Conversation Crypto Stats</string>
+    <string name="debug_settings_security_providers" translatable="false">Security Providers</string>


Suggested change

<string name="debug_settings_security_providers" translatable="false">Security Providers</string>

<string name="debug_settings_security_providers" translatable="false">Security Diagnostics</string>

yamilmedina

lgtm ! just pls let's change the name of the Security providers item to Security Diagnostics before merging as indicated and be aligned with the docs/ticket :D

github-actions · 2026-08-14T16:41:05Z

APKs built during tests are available here. Scroll down to Artifacts!

github-actions · 2026-08-18T08:12:17Z

APKs built during tests are available here. Scroll down to Artifacts!

feat: show security providers info screen

08f3bd7

pull-request-size Bot added the size/M label Aug 11, 2026

github-actions Bot assigned ohassine Aug 11, 2026

ohassine requested review from a team, MohamadJaara and yamilmedina and removed request for a team August 11, 2026 13:09

ohassine added 2 commits August 11, 2026 15:11

Merge remote-tracking branch 'origin/develop' into security-providers…

e1a4088

…-info-screen # Conflicts: # app/src/main/kotlin/com/wire/android/di/metro/WireMetroViewModelBindings.kt # app/src/main/kotlin/com/wire/android/ui/debug/DebugInfoViewModelFactory.kt

chore: conflicts

ab16e8f

yamilmedina reviewed Aug 12, 2026

View reviewed changes

Comment thread app/src/main/kotlin/com/wire/android/ui/debug/securityproviders/SecurityProvidersViewModel.kt Outdated

ohassine requested a review from yamilmedina August 12, 2026 07:40

feat: remove security providers section

98f2255

pull-request-size Bot added size/S and removed size/M labels Aug 13, 2026

yamilmedina reviewed Aug 13, 2026

View reviewed changes

yamilmedina approved these changes Aug 13, 2026

View reviewed changes

ohassine added 2 commits August 13, 2026 14:54

feat: update strings.xml

44cfbdb

chore: compose stablity

f1c344e

pull-request-size Bot added size/M and removed size/S labels Aug 14, 2026

MohamadJaara approved these changes Aug 17, 2026

View reviewed changes

ohassine added this pull request to the merge queue Aug 17, 2026

github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 17, 2026

Merge branch 'develop' into security-providers-info-screen

c74c84b

pull-request-size Bot added size/S and removed size/M labels Aug 18, 2026

ohassine enabled auto-merge August 18, 2026 07:40

ohassine added this pull request to the merge queue Aug 18, 2026

Merged via the queue into develop with commit 22c0b5c Aug 18, 2026
18 checks passed

ohassine deleted the security-providers-info-screen branch August 18, 2026 08:10

Navigation Menu

Uh oh!

feat: show security providers info screen (WPB-27916) - #5156

feat: show security providers info screen (WPB-27916)#5156
ohassine merged 7 commits into
developfrom
security-providers-info-screen

ohassine commented Aug 11, 2026 •

edited by github-actions Bot

Loading

Uh oh!

codecov Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Aug 11, 2026 •

edited

Loading

Uh oh!

MohamadJaara commented Aug 11, 2026

Uh oh!

Uh oh!

ohassine commented Aug 12, 2026

Uh oh!

yamilmedina commented Aug 12, 2026 •

edited

Loading

Uh oh!

yamilmedina Aug 13, 2026

Uh oh!

yamilmedina left a comment

Uh oh!

github-actions Bot commented Aug 14, 2026

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Aug 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

	<string name="debug_settings_security_providers" translatable="false">Security Providers</string>
	<string name="debug_settings_security_providers" translatable="false">Security Diagnostics</string>

Uh oh!

Conversation

ohassine commented Aug 11, 2026 • edited by github-actions Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Submission Checklist for internal contributors

What's new in this PR?

Issues

Testing

Test Coverage (Optional)

How to Test

Notes (Optional)

Attachments (Optional)

PR Post Submission Checklist for internal contributors (Optional)

PR Post Merge Checklist for internal contributors

References

Uh oh!

codecov Bot commented Aug 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

github-actions Bot commented Aug 11, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

MohamadJaara commented Aug 11, 2026

Uh oh!

Uh oh!

ohassine commented Aug 12, 2026

Uh oh!

yamilmedina commented Aug 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

yamilmedina Aug 13, 2026

Choose a reason for hiding this comment

Uh oh!

yamilmedina left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Aug 14, 2026

Uh oh!

Uh oh!

Uh oh!

github-actions Bot commented Aug 18, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ohassine commented Aug 11, 2026 •

edited by github-actions Bot

Loading

codecov Bot commented Aug 11, 2026 •

edited

Loading

github-actions Bot commented Aug 11, 2026 •

edited

Loading

yamilmedina commented Aug 12, 2026 •

edited

Loading