Skip to content

fix: rate limit error message change - #356

Merged
Roopan-Microsoft merged 1 commit into
devfrom
bug-fixes
Aug 3, 2025
Merged

fix: rate limit error message change#356
Roopan-Microsoft merged 1 commit into
devfrom
bug-fixes

Conversation

@Ravikirana-Microsoft

@Ravikirana-Microsoft Ravikirana-Microsoft commented Jul 31, 2025

Copy link
Copy Markdown
Contributor

Purpose

This pull request makes updates to the src/backend/app_kernel.py file, focusing on improving error messaging and adding clarity to middleware configuration. The most significant changes include refining error messages for rate limit scenarios, simplifying exception details, and clarifying a comment in the CORS middleware setup.

Error messaging improvements:

  • Updated the error message for rate limit scenarios to a more user-friendly and generic message: "Application temporarily unavailable due to quota limits. Please try again later." ([src/backend/app_kernel.pyL272-R272](https://github.com/microsoft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/pull/356/files#diff-c839d97b372f2799977f9f197d6c18667f795b2566b1b2c42e27b1479be26173L272-R272))
  • Simplified the HTTPException detail message by removing redundant text and directly using the error_msg variable. ([src/backend/app_kernel.pyL282-R282](https://github.com/microsoft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/pull/356/files#diff-c839d97b372f2799977f9f197d6c18667f795b2566b1b2c42e27b1479be26173L282-R282))

Middleware configuration:

  • Added a clarifying comment to the allow_origins parameter in the CORS middleware, specifying that all origins are allowed for development but should be restricted in production. ([src/backend/app_kernel.pyL77-R77](https://github.com/microsoft/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator/pull/356/files#diff-c839d97b372f2799977f9f197d6c18667f795b2566b1b2c42e27b1479be26173L77-R77))

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install
  • Test the code

What to Check

Verify that the following are valid

  • ...

Other Information

@Roopan-Microsoft
Roopan-Microsoft merged commit 760c42c into dev Aug 3, 2025
8 checks passed
@Roopan-Microsoft
Roopan-Microsoft deleted the bug-fixes branch August 3, 2025 12:18
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 2.2.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

deepammi pushed a commit to deepammi/Multi-Agent-Custom-Automation-Engine-Solution-Accelerator that referenced this pull request Dec 7, 2025
fix: rate limit error message change
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.

2 participants