Skip to content

Conversation

@fabiocav
Copy link
Contributor

Removing the custom handler request forwarding configuration to avoid confusion.

Purpose

  • ...
    This pull request makes a minor configuration update to the host.json file, changing the HTTP request handling setting. The enableForwardingHttpRequest property has been removed and replaced with enableHttpProxyingRequest.

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Other Information

Removing the custom handler request forwarding configuration to avoid confusion.
@justinyoo
Copy link

Without this property, enableForwardingHttpRequest, this MCP server is not captured by GHCP agent mode. It should not be removed.

@lilyjma
Copy link
Contributor

lilyjma commented Aug 18, 2025

@fabiocav - is this a change that would require the new version of the host?

@fabiocav
Copy link
Contributor Author

This change requires the latest version of the host and Core Tools. Those are already fully deployed, so you should be able to merge this.

Without this property, enableForwardingHttpRequest, this MCP server is not captured by GHCP agent mode. It should not be removed.

This is likely because your validation was done with an older version of the tool and proxying was not working (as it wasn't a feature in the old versions), so it would fallback to forwarding, which is not something we want to do.

Copy link
Contributor

@lilyjma lilyjma left a comment

Choose a reason for hiding this comment

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

Tested. Needs latests Azure Functions Core Tools (v.4.2.1) to work without the setting.

@lilyjma lilyjma merged commit 35282c3 into Azure-Samples:main Sep 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants