hetzner-cloud-php-sdk icon indicating copy to clipboard operation
hetzner-cloud-php-sdk copied to clipboard

Error on retrieving Firewall if there is a rule with "any" port

Open stefangeorgescu opened this issue 4 years ago • 3 comments

$firewall = $this->client->firewalls()->getByName('name'); I'm receiving the following error when retrieving a Firewall which has a rule with an "any" port:

PHP Fatal error: Uncaught TypeError: Argument 5 passed to LKDev\HetznerCloud\Models\Firewalls\FirewallRule::__construct() must be of the type string, null given, called in /vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/Firewalls/Firewall.php on line 112 and defined in /vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/Firewalls/FirewallRule.php:48 Stack trace: #0 /vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/Firewalls/Firewall.php(112): LKDev\HetznerCloud\Models\Firewalls\FirewallRule->__construct('in', 'icmp', Array, Array, NULL) #1 /vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/Firewalls/Firewalls.php(129): LKDev\HetznerCloud\Models\Firewalls\Firewall::parse(Object(stdClass)) #2 [internal function]: LKDev\HetznerCloud\Models\Firewalls\Firewalls->LKDev\HetznerCloud\Models\Firewalls\{closure}(Object(stdClass), 0) #3 /vendor/tightenco/collect/src/Collect/Support/Collection.ph in /vendor/lkdevelopment/hetzner-cloud-php-sdk/src/Models/Firewalls/FirewallRule.php on line 48

The issue should be easy to resolve adding null to the $port parameter type casting in FirewallRule.

stefangeorgescu avatar Apr 27 '21 23:04 stefangeorgescu

Hey @stefangeorgescu,

thank you for your report! I won't have the time to fix this in this week, would it be possible that you contribute the fix :)?

Thank you!

LKaemmerling avatar Apr 29 '21 04:04 LKaemmerling

Sure, here you go #72

stefangeorgescu avatar Apr 29 '21 07:04 stefangeorgescu

Hey @LKaemmerling

this issue seems to be resolved

MatusBoa avatar Dec 21 '21 14:12 MatusBoa