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

Updated ACL Template APIs #4

Merged
merged 3 commits into from
Dec 1, 2021
Merged

Updated ACL Template APIs #4

merged 3 commits into from
Dec 1, 2021

Conversation

yvikrant
Copy link
Collaborator

  • Updated ACL template API with latest payload
  • Created new endoint to check device ACL status

- Updated ACL template API with latest payload
- Created new endoint to check device ACL status
@displague
Copy link
Member

Hi @yvikrant, It's difficult to review these changes when they don't abide by the published spec: https://developer.equinix.com/catalog/network-edgev1#operation/createDeviceACLTemplateUsingPost (subnet vs subnets, for example).

It would help if there was a changelog available to compare the spec differences against the changes here.
It would help if the deprecation plan was available for the fields being removed in this PR. If those fields are persisting in the API, it would be best not to remove those features from the client.

Because this client is a v1 client, these breaking changes will need to be considered with v2 semantics:
https://go.dev/blog/module-compatibility
https://codeengineered.com/blog/2019/go-mod-major-versions/

@yvikrant
Copy link
Collaborator Author

yvikrant commented Nov 30, 2021

Hi @displague

Here's the old contract for creating ACL Template. I kept all the fields that are currently supported by API. Also, I have added comments against the properties which are deprecated.

{
    "name": "fqdn check",
    "description": "",
    "metroCode": "DC", // Deprecated as 
    "inboundRules": [
        {
            "srcType": "DOMAIN", // deprecated as FQDNs are not supported
            "protocol": "IP",
            "srcPort": "any",
            "dstPort": "any",
            "fqdn": "equinix.com", // Support Removed
            "subnets": [ // Deprecated and replaced with subnet
                "74.6.143.25/32",
            ],
            "seqNo": 1
        }
    ]
}

@displague displague self-assigned this Dec 1, 2021
@yvikrant
Copy link
Collaborator Author

yvikrant commented Dec 1, 2021

@displague . Added removed fields and marked them as deprecated. We'll plan to remove deprecated fields in the next major version.

@displague displague merged commit 1a7dd8f into equinix:master Dec 1, 2021
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