functions-framework-dart icon indicating copy to clipboard operation
functions-framework-dart copied to clipboard

feat: added support for IPv6

Open CosmicPangolin opened this issue 1 year ago • 6 comments

@kevmoo Tagging you here 'cause I figure this repo doesn't get a lot of attention. This is a trivial patch that I needed to unlock some functionalities on the Fly.io subnet, which relies heavily on IPv6 for internal resolution and load balancing. Doubt it conflicts with any GCP needs?

CosmicPangolin avatar Mar 22 '24 07:03 CosmicPangolin

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

google-cla[bot] avatar Mar 22 '24 07:03 google-cla[bot]

I think the convention on cloud run is to use IPv4 still.

I wonder if we'll have to just add a configuration option or something...

kevmoo avatar Mar 22 '24 23:03 kevmoo

My understanding is that the IPv6 option covers IPv4 addresses as well (this could be documented better in the dart:io api, methinks).

CosmicPangolin avatar Mar 24 '24 01:03 CosmicPangolin

My understanding is that the IPv6 option covers IPv4 addresses as well (this could be documented better in the dart:io api, methinks).

It's the opposite. All ipV4 addresses are v6, but not the other way around!

kevmoo avatar Mar 24 '24 21:03 kevmoo

Agreed :) I just mean InternetAddress.anyIPv6 resolves both, wrt cloud run requiring v4.

CosmicPangolin avatar Mar 25 '24 04:03 CosmicPangolin

it seems like tests still pass!

Would you add a changelog entry here!

kevmoo avatar Apr 17 '24 20:04 kevmoo