-
Notifications
You must be signed in to change notification settings - Fork 81
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
add GetFirewall method #138
Conversation
84ab503
to
bfee9f3
Compare
bfee9f3
to
3c80824
Compare
@@ -18,7 +18,7 @@ type NetworkAddresses struct { | |||
|
|||
// A FirewallRule is a whitelist of ports, protocols, and addresses for which traffic should be allowed. | |||
type FirewallRule struct { | |||
Ports string `json:"ports"` | |||
Ports string `json:"ports,omitempty"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is important, as with ICMP, ports
is invalid if present in the request.
@@ -19,7 +20,7 @@ var ( | |||
func TestListFirewalls(t *testing.T) { | |||
client, _, teardown, err := setupFirewall(t, []firewallModifier{ | |||
func(createOpts *linodego.FirewallCreateOptions) { | |||
createOpts.Label = randString(12, lowerBytes, digits) + "-linodego-testing" | |||
createOpts.Label = randString(12, lowerBytes, upperBytes) + "-linodego-testing" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A Firewall's label cannot begin with a number, rerunning this threw an error because of it.
add GetFirewall method
add GetFirewall method
add GetFirewall method
No description provided.