Skip to content

update-app-polling - Fix anonymous notification polling 401 spam.#1002

Merged
Bionic711 merged 1 commit into
microsoft:Developmentfrom
Xeelee33:update-app-polling
Jul 7, 2026
Merged

update-app-polling - Fix anonymous notification polling 401 spam.#1002
Bionic711 merged 1 commit into
microsoft:Developmentfrom
Xeelee33:update-app-polling

Conversation

@chadpalmer

Copy link
Copy Markdown
Contributor

Summary

  • Prevents unauthenticated pages from loading the notification polling script.
  • Stops the login screen from repeatedly calling /api/notifications/count after idle timeout or before sign-in.
  • Updates version, release notes, and notification polling regression coverage.

Validation

  • git diff --check
  • python -m py_compile application/single_app/config.py functional_tests/test_notification_polling_redirect_guard.py ui_tests/test_notifications_polling_redirect_guard.py
  • python -m compileall -q application/single_app
  • python scripts/check_xss_sinks.py --full-file application/single_app/templates/base.html
  • pytest functional_tests/test_notification_polling_redirect_guard.py

Notes

  • Local UI test execution was blocked because the environment did not provide the playwright pytest fixture.

@chadpalmer

chadpalmer commented Jul 7, 2026

Copy link
Copy Markdown
Contributor Author

@Bionic711 @paullizer This PR turns off polling for unauthenticated users to prevent 401 error log spamming when users get logged out due to idle timeout or load the app page without signing in. Please let me know if you need any updates from me.

@Bionic711

Copy link
Copy Markdown
Collaborator

This be like some kind of crazy talk. Why on Earth would you not want you app insights and console logs filled with 401 spam? That is what separates the chaff from the real admins!

Will review tomorrow.

@Bionic711 Bionic711 merged commit 3450af6 into microsoft:Development Jul 7, 2026
8 checks passed
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