Merge pull request #370 from aaronjorbin/fix/null-cookie-value

handle no domain in a PHP7.4 compatable way

Author: jrfnl

library/Requests/Cookie.php

@@ -288,6 +288,11 @@ protected function normalize_attribute($name, $value) {
 				return $expiry_time;
 
 			case 'domain':
+				// Domains are not required as per RFC 6265 section 5.2.3
+				if (empty($value)) {
+					return null;
+				}
+
 				// Domain normalization, as per RFC 6265 section 5.2.3
 				if ($value[0] === '.') {
 					$value = substr($value, 1);

tests/Cookies.php

@@ -600,6 +600,13 @@ public function parseFromHeadersProvider() {
 				array( 'invalid' => false ),
 			),
 
+			# Empty Domain	
+			array(
+				'name=value; domain=',
+				'http://example.com/test/',
+				array(),
+			),
+
 			# Subdomain cookies
 			array(
 				'name=value; domain=test.example.com',
@@ -639,4 +646,4 @@ public function testParsingHeaderWithOrigin($header, $origin, $expected, $expect
 		$cookie = reset($parsed);
 		$this->check_parsed_cookie($cookie, $expected, $expected_attributes, $expected_flags);
 	}
-}
+}