Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix: add warning to fetch() calls that will change the requested port #992

Merged
merged 1 commit into from
Jun 28, 2022
Merged

fix: add warning to fetch() calls that will change the requested port #992

merged 1 commit into from
Jun 28, 2022

Commits on Jun 28, 2022

  1. fix: add warning to fetch() calls that will change the requested port 

    In Workers published to the Edge (rather than previews) there is a bug where a custom port on a downstream fetch request is ignored, defaulting to the standard port.
For example, `https://my.example.com:668` will actually send the request to `https://my.example.com:443`.

This does not happen when using `wrangler dev` (both in remote and local mode), but to ensure that developers are aware of it this change displays a runtime warning in the console when the bug is hit.

Closes cloudflare#1320
    @petebacondarwin
    petebacondarwin committed Jun 28, 2022
    Configuration menu
    Copy the full SHA
    5464941 View commit details
    Browse the repository at this point in the history