-
Notifications
You must be signed in to change notification settings - Fork 769
Fix: Remove addition slash for check proxy health #500
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix: Remove addition slash for check proxy health #500
Conversation
Hi, could you grab the latest from main? We had to push a few bug fixes through including a security update, which looks like it caused some conflicts in your PR. Thanks! |
Hi @olaservo |
Hi @olaservo Finds out there is lint error, I have change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested locally, verified issue and fix.
LGTM! 👍
Remove the additional slash
/
at the end of the proxy base URLMotivation and Context
During an MCP course, I found out that if I type the inspector proxy address base URL as follows,

https://s172-29-9-71p6277.lab-aws-production.deeplearning.ai/
, it will cause an errorTurns out if the user types a slash
/
at the end of the address, which will cause a double slash//
, it makes the error occur.How Has This Been Tested?
Add a slash and without a slash
/
- https://inspector-proxy.ai[/]Breaking Changes
Will users need to update their code or configurations? No
Types of changes
Checklist
Additional context