Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changing site settings creates an infinite request loop #23787

Open
dcalhoun opened this issue Nov 8, 2024 · 2 comments
Open

Changing site settings creates an infinite request loop #23787

dcalhoun opened this issue Nov 8, 2024 · 2 comments

Comments

@dcalhoun
Copy link
Member

dcalhoun commented Nov 8, 2024

Expected behavior

Updating a single setting should result in a single POST request.

Actual behavior

Updating a single setting results in a never-ending stream of POST requests.

Steps to reproduce the behavior

  1. Set up your device so that you can monitor network traffic—e.g., send device requests through Charles Proxy.
  2. Navigate to My Site.
  3. Tap More.
  4. Scroll down and tap Site Settings.
  5. Scroll down to and tap Discussion.
  6. Toggle Allow Comments.
Tested on iPhone SE, iOS 18.1, Jetpack iOS / WordPress iOS 25.4.2
@dangermattic
Copy link
Collaborator

dangermattic commented Nov 8, 2024

Thanks for reporting! 👍

@dcalhoun
Copy link
Member Author

dcalhoun commented Nov 8, 2024

From debugging this, it appears the persist settings guard clause is insufficient. Settings are persisted anytime the context changes, which then results in the context changing again. The guard clause needs to be more robust to ensure an infinite loop is not created when the settings object is changed.

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

No branches or pull requests

2 participants