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

WIP: feat: add bYOIPv6 + endpoint + Destination IP support at locations(don;t merge yet; need cloudflare-go next version) #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.

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