fix: falsely omitted fields in firewall schema (#396)

In the [API Docs](https://docs.hetzner.cloud/#firewalls-get-a-firewall)
the `description`, `port`, `destination_ips` and `source_ips` properties
are not marked as required (thus being optional) although in practice
they are always set. This leads to inconsistencies between marshaled
schemas and API responses. This PR modifies the schema so that it is
consistent with the actual API output.

Author: phm07

hcloud/schema/firewall.go

@@ -15,11 +15,11 @@ type Firewall struct {
 // FirewallRule defines the schema of a Firewall rule.
 type FirewallRule struct {
 	Direction      string   `json:"direction"`
-	SourceIPs      []string `json:"source_ips,omitempty"`
-	DestinationIPs []string `json:"destination_ips,omitempty"`
+	SourceIPs      []string `json:"source_ips"`
+	DestinationIPs []string `json:"destination_ips"`
 	Protocol       string   `json:"protocol"`
-	Port           *string  `json:"port,omitempty"`
-	Description    *string  `json:"description,omitempty"`
+	Port           *string  `json:"port"`
+	Description    *string  `json:"description"`
 }
 
 // FirewallListResponse defines the schema of the response when listing Firewalls.