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.

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

feat: add bYOIPv6 + endpoint + Destination IP support at locations #4805

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rexscaria
Copy link
Contributor

No description provided.

@rexscaria rexscaria force-pushed the rex/endpoint+ipv6_support_at_location branch from 67f285f to 3e14762 Compare December 19, 2024 17:30
Copy link
Contributor

changelog detected ✅

@jacobbednarz
Copy link
Member

jacobbednarz commented Dec 24, 2024

what is the plan with policy_ids here? we can't just remove it without a migration path. most likely, you'll want to continue supporting policy_ids in the new structs in a backwards compatible way.

@rexscaria
Copy link
Contributor Author

rexscaria commented Dec 25, 2024 via email

@jacobbednarz
Copy link
Member

if it's no longer usable via the API or the product, we can remove it here but need to be sure no one is using it internally.

if it's still usable (although deprecated), we'll need to continue to have it here but marked as deprecated.

@rexscaria
Copy link
Contributor Author

rexscaria commented Dec 25, 2024 via email

@jacobbednarz
Copy link
Member

cool. i'll PR the removal before i cut the next release.

@rexscaria rexscaria force-pushed the rex/endpoint+ipv6_support_at_location branch from 3e14762 to 9752125 Compare January 3, 2025 20:15
@rexscaria rexscaria force-pushed the rex/endpoint+ipv6_support_at_location branch from 9752125 to 132d89b Compare January 3, 2025 20:26
@rexscaria rexscaria changed the title WIP: feat: add bYOIPv6 + endpoint + Destination IP support at locations(don;t merge yet; need cloudflare-go next version) feat: add bYOIPv6 + endpoint + Destination IP support at locations(don;t merge yet; need cloudflare-go next version) Jan 3, 2025
@rexscaria rexscaria changed the title feat: add bYOIPv6 + endpoint + Destination IP support at locations(don;t merge yet; need cloudflare-go next version) feat: add bYOIPv6 + endpoint + Destination IP support at locations(don't merge yet; need cloudflare-go next version) Jan 6, 2025
@rexscaria
Copy link
Contributor Author

I have updated this PR to reflect the current gateway API. This is ready to merge. Pls let me know if any addition is needed here.

@rexscaria rexscaria changed the title feat: add bYOIPv6 + endpoint + Destination IP support at locations(don't merge yet; need cloudflare-go next version) feat: add bYOIPv6 + endpoint + Destination IP support at locations Jan 6, 2025
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.

2 participants