fix: Remove debugging logs and update quota check documentation#844
Merged
Conversation
Contributor
There was a problem hiding this comment.
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.mdto downloadquota_check_params.shfrom the correctinfra/scriptspath. - Removed proxy request/response debug
console.logstatements insrc/App/server/server.jswhile 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
approved these changes
May 8, 2026
Contributor
|
🎉 This PR is included in version 2.5.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
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
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.
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:
curlcommand withindocs/QuotaCheck.mdto point to the correct script location.Code cleanup:
src/App/server/server.jsto reduce console noise.Does this introduce a breaking change?
Golden Path Validation
Deployment Validation