Skip to content

Conversation

@itsmeichigo
Copy link
Contributor

@itsmeichigo itsmeichigo commented Oct 8, 2025

Closes WOOMOB-1450

Description

There are crash reports regarding the removal of retried requests when flagging sites as unsupported for application passwords. These are caused by the race condition when the list of retried requests is altered simultaneously from different threads.

This PR fixes the crash by ensuring that the retried request list is updated atomically with a barrier block.

Testing steps

It's not straightforward to reproduce race conditions so it's tricky to test the crash. Simply smoke testing the flow should be sufficient.

Use the test plan in pe5sF9-4Am-p2 - run TC4. Confirm that the app doesn't crash.

Testing information

  • Added a unit test to confirm the race condition fix. The test fails before the fix.
  • Ran TC4 with simulator iPhone 17 and confirmed the app doesn't crash.

Screenshots

N/A


  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

@itsmeichigo itsmeichigo added this to the 23.5 milestone Oct 8, 2025
@itsmeichigo itsmeichigo added the type: crash The worst kind of bug. label Oct 8, 2025
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Oct 8, 2025

App Icon📲 You can test the changes from this Pull Request in WooCommerce iOS Prototype by scanning the QR code below to install the corresponding build.

App NameWooCommerce iOS Prototype
Build Numberpr16223-54b98ac
Version23.4
Bundle IDcom.automattic.alpha.woocommerce
Commit54b98ac
Installation URL5tmrktlo6q458
Automatticians: You can use our internal self-serve MC tool to give yourself access to those builds if needed.

@itsmeichigo
Copy link
Contributor Author

Closing to target 23.4 instead.

@itsmeichigo itsmeichigo deleted the woomob-1450-crash-in-alamofirenetworkerrorhandler branch October 13, 2025 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: crash The worst kind of bug.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants