Skip to content

fix: Remove debugging logs and update quota check documentation#844

Merged
Roopan-Microsoft merged 2 commits into
devfrom
rc-bug-fix0605
May 8, 2026
Merged

fix: Remove debugging logs and update quota check documentation#844
Roopan-Microsoft merged 2 commits into
devfrom
rc-bug-fix0605

Conversation

@Ragini-Microsoft

Copy link
Copy Markdown
Collaborator

Purpose

This pull request includes minor improvements to documentation and code cleanliness. It updates a script URL in the quota check documentation and removes debugging log statements from the server proxy code.

  • Documentation update:

    • Fixed the URL in the curl command within docs/QuotaCheck.md to point to the correct script location.
  • Code cleanup:

    • Removed unnecessary proxy request and response logging from src/App/server/server.js to reduce console noise.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

Copilot AI review requested due to automatic review settings May 7, 2026 18:29
@Ragini-Microsoft Ragini-Microsoft changed the title Remove debugging logs and update quota check documentation fix: Remove debugging logs and update quota check documentation May 7, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR makes small documentation and cleanliness updates: it fixes the quota-check script download URL in the docs and removes noisy proxy request/response debug logging from the frontend server proxy.

Changes:

  • Updated docs/QuotaCheck.md to download quota_check_params.sh from the correct infra/scripts path.
  • Removed proxy request/response debug console.log statements in src/App/server/server.js while keeping error logging and SSE-related proxy behavior intact.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/App/server/server.js Removes proxy debug logs; keeps SSE streaming headers/flush and error handling unchanged.
docs/QuotaCheck.md Fixes the raw GitHub URL for downloading quota_check_params.sh to the correct repo path.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Roopan-Microsoft Roopan-Microsoft merged commit d105440 into dev May 8, 2026
21 of 22 checks passed
@Roopan-Microsoft Roopan-Microsoft deleted the rc-bug-fix0605 branch May 8, 2026 04:10
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 2.5.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants