Skip to content

fix: App cannot be started by user - WPB-24848#4777

Open
netbe wants to merge 3 commits into
release/cycle-4.16from
fix/WPB-24848-skip-blacklist-loading-session
Open

fix: App cannot be started by user - WPB-24848#4777
netbe wants to merge 3 commits into
release/cycle-4.16from
fix/WPB-24848-skip-blacklist-loading-session

Conversation

@netbe
Copy link
Copy Markdown
Collaborator

@netbe netbe commented May 28, 2026

BugWPB-24848 [iOS] [App Launch] App cannot be started by user

Issue

On C build, on special environment, the biometric prompt for EAR seemed not to happen especially on cold start. After more testing, it did happen but after 60 seconds after the blacklist check is done.

For some reason, the resolving of the url (which points to a 404, blacklist disabled on this environment) times out.

Solution:

Port fix of 4.18 to differ checking the blacklist from the userSession loading. See #4485

Testing

Run app on the special environment


Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@netbe netbe requested a review from samwyndham May 28, 2026 14:05
@netbe netbe changed the title fix: App cannot be started by user - WPB-24848 fix: App cannot be started by user - WPB-24848 May 28, 2026
@netbe netbe requested a review from David-Henner May 28, 2026 14:09
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 28, 2026

Test Results

   17 files  1 193 suites   10m 49s ⏱️
8 823 tests 8 794 ✅ 28 💤 1 ❌
8 835 runs  8 807 ✅ 28 💤 0 ❌

For more details on these failures, see this check.

Results for commit 8cf3756.

♻️ This comment has been updated with latest results.

Summary: workflow run #26971611510
Allure report (download zip): html-report-30532-fix_WPB-24848-skip-blacklist-loading-session

Comment on lines +111 to +115
let pathStream = AsyncStream<NWPath> { pathContinuation in
monitor.pathUpdateHandler = { pathContinuation.yield($0) }
pathContinuation.onTermination = { _ in monitor.cancel() }
monitor.start(queue: DispatchQueue(label: "CheckBlacklistWorker.NWPathMonitor"))
}
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

this code is new, since the original code was not supporting iOS 16

Copy link
Copy Markdown
Contributor

@samwyndham samwyndham left a comment

Choose a reason for hiding this comment

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

LGTM

@netbe netbe enabled auto-merge (squash) June 4, 2026 18:29
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Jun 4, 2026

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.

2 participants