Repository navigation
Fix large workspace upload reliability and status reporting - #1387
Merged
Paul Lizer (paullizer) merged 29 commits intoSep 3, 2026
Merged
Paul Lizer (paullizer) merged 29 commits into
Paul Lizer (paullizer) merged 29 commits into
Conversation
Resolves the merge conflicts with Development (VERSION and release notes) and replaces the IS_DEVELOPMENT gate with a correct fix. Logout redirected to /.auth/logout and returned 404 whenever WEBSITE_AUTH_AAD_ALLOWED_TENANTS was set without Easy Auth actually running. That variable is configured by hand per the advanced environment variable guidance, so it was never evidence Easy Auth was intercepting requests. This affected production deployments, not just development ones. Detection now relies only on the X-MS-CLIENT-PRINCIPAL headers App Service injects into requests it intercepts, so Easy Auth logout is preserved wherever Easy Auth is genuinely active, including in development. Adds DISABLE_APP_SERVICE_EASY_AUTH_LOGOUT for deployments where Easy Auth is active but /.auth/* is not routed to the origin, plus debug logging of the logout path decision and a Logout Behavior Across Environments section in the local development guide. Also removes a customer hostname from the test fixture and replaces an exact VERSION assertion in the idle logout test with the shared version helper. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Promote v0.260.028 from Development to Staging
This prevents 404 error after logout.
…y prevent confusion.
Version and test updates.
…load overloaded cosmos gate.
Fixed goofy github conflict
Updating with latest fixes to deploy to dev for testing
… workspace document uploads are fixed.
…smos gate is not overwhelmed when many documents are uploaded.
Promote Development to Staging for v0.261.002
Promote Staging to main for v0.261.002
…ng that throws orderddict error.
Documentation Updates
Documentation Updates
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Contributor
Author
|
Paul Lizer (@paullizer) Bionic711 This new PR has both the document upload fix and the Easy Auth route issue as per your comment on the other PR. I will go ahead and close that PR. Please let me know if you need anything else or have any questions. Thanks! |
…t return to pass github security check.
Paul Lizer (paullizer)
merged commit Sep 3, 2026
f18c667
into
microsoft:Development
11 of 12 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR improves reliability and user feedback for large workspace document uploads.
OrderedDict mutated during iterationMarkdown parser failure before marking a document failed./.auth/logoutis not publicly reachable.Validation
git diff --checkpython -m py_compile application/single_app/config.py application/single_app/functions_data_management_search_write_fence.py application/single_app/functions_documents.py application/single_app/gunicorn.conf.py application/single_app/route_frontend_authentication.py functional_tests/test_app_service_easy_auth_logout.py functional_tests/test_data_management_search_write_fence.py functional_tests/test_idle_logout_timeout.py functional_tests/test_markdown_processing_batches_search_writes.py ui_tests/test_workspace_upload_request_status_copy.pypython functional_tests/test_app_service_easy_auth_logout.pypython functional_tests/test_idle_logout_timeout.pypython functional_tests/test_markdown_processing_batches_search_writes.pypython ui_tests/test_workspace_upload_request_status_copy.pypython -m pytest functional_tests/test_data_management_search_write_fence.pypython scripts/check_swagger_routes.py application/single_app/route_frontend_authentication.pypython functional_tests/route_tests/test_route_blueprint_policy_inventory.pypython functional_tests/route_tests/test_route_unauthenticated_policy_contract.pypython functional_tests/route_tests/test_route_policy_test_coverage.pynode --check application/single_app/static/js/workspace/workspace-documents.jsnode --check application/single_app/static/js/public/public_workspace.jspython scripts/check_xss_sinks.py --base-sha upstream/Development --head-sha HEAD ...python scripts/check_broken_access_control.py --base-sha upstream/Development --head-sha HEAD ...python functional_tests/test_docs_app_surface_coverage.pypython functional_tests/test_docs_site_quality.pyNotes
example.envchanges are commented sample configuration only and contain no secrets.