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

InternetGateway with igw-id fails #83

Closed
rmenn opened this issue Feb 3, 2015 · 2 comments
Closed

InternetGateway with igw-id fails #83

rmenn opened this issue Feb 3, 2015 · 2 comments
Labels
bug This issue is a bug.

Comments

@rmenn
Copy link

rmenn commented Feb 3, 2015

Greetings,

I found that if we add a gw-id to the DescribeInternetGateway method you get an error from AWS.

Below is a sample code

    id := "igw-XXXXX"
    DescribeIGWOpts := &ec2.DescribeInternetGatewaysRequest{
        InternetGatewayIDs: []string{id},
    }
    resp, err := cli.DescribeInternetGateways(DescribeIGWOpts)
    if err != nil {
        panic(err)
    }
    res := resp.InternetGateways[0]
    ID := res.InternetGatewayID
    fmt.Println(*ID)

Error:
panic: The parameter InternetGatewayIds is not recognised

@lsegal lsegal added the bug This issue is a bug. label Feb 3, 2015
@lsegal
Copy link
Contributor

lsegal commented Feb 10, 2015

Closing in favor of #100. This is a limitation of the current EC2 implementation, which does not yet fully support the full protocol. Follow along in the related ticket for progress updates on EC2 support.

@lsegal lsegal closed this as completed Feb 10, 2015
@catsby
Copy link

catsby commented Mar 3, 2015

@rmenn I found the problem here to be this line in gen/ec2/ec2.go:

InternetGatewayIDs []string         `ec2:"InternetGatewayIds" xml:"internetGatewayId>item"`

It should be ec2:"InternetGatewayId", matching VPNGatewayIDs in DescribeVPNGatewaysRequest, et. al.

Looking at the API json, I'm not sure why gen/ec2/ec2.go got generated that way, though I'm probably not understanding something about the code generation process.

For what it's worth, changning that to ec2:"InternetGatewayId" resolves this issue

Update: fixed typos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
Development

No branches or pull requests

3 participants