Skip to content

parse_org_private_endpoint_remove can return an OrganizationPatchPrivateEndpoint with id: None (e.g. when the user passes only description=... / region=...). The CLI help text implies the endpoint id is required for removals, and sending a remove patch without an id is likely invalid. Consider validating endpoint.id.is_some() before returning, and erroring with a clear message if it’s missing. #32

@sdairs

Description

@sdairs

parse_org_private_endpoint_remove can return an OrganizationPatchPrivateEndpoint with id: None (e.g. when the user passes only description=... / region=...). The CLI help text implies the endpoint id is required for removals, and sending a remove patch without an id is likely invalid. Consider validating endpoint.id.is_some() before returning, and erroring with a clear message if it’s missing.


    if endpoint.id.is_none() {
        return Err(
            "missing required 'id' for remove-private-endpoint; specify an id positionally or as 'id=...'"
                .into(),
        );
    }

Originally posted by @Copilot in #25 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions