aws/copilot-cli

Prefix Lists

Open

#4,272 opened on Dec 15, 2022

View on GitHub
 (5 comments) (2 reactions) (0 assignees)Go (441 forks)auto 404
good first issuesize/Stype/feature

Repository metrics

Stars
 (3,737 stars)
PR merge metrics
 (No merged PRs in 30d)

Description

Hi Team,

I've attempted to try to swap out an IP specification in a Load Balancer security group with a prefix list and I'm getting the following error.

» copilot env deploy --name test ✘ validate environment manifest for "test": validate "http config": validate "public": parse IPNet pl-xxxxxxxxxxxxxxxxx: invalid CIDR address: pl-xxxxxxxxxxxxxxxxx

Is there any way to achieve this in the current state or would this need to be a new feature?

Cheers,

Damon

Contributor guide

Research direction
Investigate how Copilot validates IP specifications in security groups (likely in pkg/deploy or pkg/manifest) and extend the parsing logic to accept prefix list IDs (e.g., pl xxxx) in addition to CIDR notation.
Tech stack
go
Domain
backendcloudinfrastructuredevops
Issue type
Feature
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
Half day
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitAWS
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
40