Skip to content

Commit e195ab7

Browse files
nyamsprodnyamsprod
authored
Merge pull request #324 from jeremykendall/develop
Prepare 6.1.1 release
2 parents d8b196b + 1df0801 commit e195ab7

15 files changed

+2569
-954
lines changed

‎.github/workflows/build.yaml‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ jobs:
1010
runs-on: ubuntu-latest
1111
strategy:
1212
matrix:
13-
php: [ '7.4', '8.0' ]
13+
php: [ '7.4', '8.0', '8.1' ]
1414
composer-flags: [ '' ]
1515
phpunit-flags: [ '--coverage-text' ]
1616
steps:
@@ -23,8 +23,8 @@ jobs:
2323
- run: composer update --no-progress ${{ matrix.composer-flags }}
2424
- run: composer phpunit
2525
- run: composer phpstan
26-
if: ${{ matrix.php == '8.0' }}
26+
if: ${{ matrix.php == '8.1' }}
2727
- run: composer psalm
28-
if: ${{ matrix.php == '8.0' }}
28+
if: ${{ matrix.php == '8.1' }}
2929
- run: composer phpcs
30-
if: ${{ matrix.php == '8.0' }}
30+
if: ${{ matrix.php == '8.1' }}

‎.gitignore‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,5 +4,6 @@ docs
44
build
55
.idea
66
.php_cs.cache
7+
.php-cs-fixer.cache
78
.phpunit.result.cache
89
composer.lock

‎.php-cs-fixer.php‎

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -11,21 +11,31 @@
1111
'@PSR2' => true,
1212
'array_syntax' => ['syntax' => 'short'],
1313
'concat_space' => ['spacing' => 'none'],
14+
'global_namespace_import' => [
15+
'import_classes' => true,
16+
'import_constants' => true,
17+
'import_functions' => true,
18+
],
19+
'list_syntax' => ['syntax' => 'short'],
1420
'new_with_braces' => true,
1521
'no_blank_lines_after_phpdoc' => true,
1622
'no_empty_phpdoc' => true,
1723
'no_empty_comment' => true,
1824
'no_leading_import_slash' => true,
19-
'no_superfluous_phpdoc_tags' => true,
25+
'no_superfluous_phpdoc_tags' => [
26+
'allow_mixed' => true,
27+
'remove_inheritdoc' => true,
28+
'allow_unused_params' => false,
29+
],
2030
'no_trailing_comma_in_singleline_array' => true,
2131
'no_unused_imports' => true,
2232
'ordered_imports' => ['imports_order' => ['class', 'function', 'const'], 'sort_algorithm' => 'alpha'],
23-
'phpdoc_add_missing_param_annotation' => ['only_untyped' => false],
33+
'phpdoc_add_missing_param_annotation' => ['only_untyped' => true],
2434
'phpdoc_align' => true,
2535
'phpdoc_no_empty_return' => true,
2636
'phpdoc_order' => true,
2737
'phpdoc_scalar' => true,
28-
'phpdoc_to_comment' => false,
38+
'phpdoc_to_comment' => true,
2939
'phpdoc_summary' => true,
3040
'psr_autoloading' => true,
3141
'return_type_declaration' => ['space_before' => 'none'],
@@ -36,7 +46,5 @@
3646
'trailing_comma_in_multiline' => true,
3747
'trim_array_spaces' => true,
3848
'whitespace_after_comma_in_array' => true,
39-
'yoda_style' => true,
4049
])
41-
->setFinder($finder)
42-
;
50+
->setFinder($finder);

‎CHANGELOG.md‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,24 @@
22

33
All Notable changes to `PHP Domain Parser` starting from the **5.x** series will be documented in this file
44

5+
## 6.1.1 - 2022-02-18
6+
7+
### Added
8+
9+
- None
10+
11+
### Fixed
12+
13+
- [#321](https://github.com/jeremykendall/php-domain-parser/issues/321) improve resolving private domain suffix. `Rules::getPrivateDomain` will now throw if the domain name does not contain a "private" TLD.
14+
15+
### Deprecated
16+
17+
- None
18+
19+
### Removed
20+
21+
- None
22+
523
## 6.1.0 - 2021-06-19
624

725
### Added

‎composer.json‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -46,18 +46,18 @@
4646
"ext-json": "*"
4747
},
4848
"require-dev": {
49-
"friendsofphp/php-cs-fixer": "^2.16",
49+
"friendsofphp/php-cs-fixer": "^v3.5.0",
5050
"guzzlehttp/guzzle": "^7.0",
51-
"guzzlehttp/psr7": "^1.6",
52-
"phpstan/phpstan": "^0.12",
53-
"phpstan/phpstan-phpunit": "^0.12",
54-
"phpstan/phpstan-strict-rules": "^0.12",
55-
"phpunit/phpunit": "^9.3",
56-
"psalm/plugin-phpunit": "^0.15.0",
51+
"guzzlehttp/psr7": "^1.6||^2.0",
52+
"phpstan/phpstan": "^1.4.6.",
53+
"phpstan/phpstan-phpunit": "^1.0.0",
54+
"phpstan/phpstan-strict-rules": "^1.1.0",
55+
"phpunit/phpunit": "^9.5.13",
56+
"psalm/plugin-phpunit": "^0.15.2",
5757
"psr/http-factory": "^1.0",
5858
"psr/simple-cache": "^1.0",
5959
"symfony/cache": "^v5.0",
60-
"vimeo/psalm": "^4.3"
60+
"vimeo/psalm": "^4.20"
6161
},
6262
"suggest": {
6363
"psr/http-client-implementation": "To use the storage functionnality which depends on PSR-18",

‎phpstan.neon‎

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,4 +4,12 @@ includes:
44
- vendor/phpstan/phpstan-phpunit/rules.neon
55
parameters:
66
ignoreErrors:
7+
- message: '#has no value type specified in iterable type array.#'
8+
path: src/Rules.php
9+
- message: '#has parameter \$(properties|rules|list|tmpList) with no value type specified in iterable type array.#'
10+
path: src/Rules.php
11+
- message: "#Cannot access offset '!' on mixed.#"
12+
path: src/Rules.php
13+
- message: '#Variable \$line on left side of \?\? always exists and is not nullable.#'
14+
path: src/Rules.php
715
reportUnmatchedIgnoredErrors: true

‎src/DomainName.php‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,6 @@ public function withLabel(int $key, $label): self;
101101
* If $key is non-negative, the removed label will be the label at $key position from the start.
102102
* If $key is negative, the removed label will be the label at $key position from the end.
103103
*
104-
* @param int ...$keys
105104
*
106105
* @throws CannotProcessHost If the key is out of bounds
107106
*/

‎src/DomainTest.php‎

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
namespace Pdp;
66

77
use PHPUnit\Framework\TestCase;
8+
use stdClass;
89
use TypeError;
910

1011
/**
@@ -314,7 +315,7 @@ public function withLabelWorksProvider(): iterable
314315
public function testWithLabelFailsWithTypeError(): void
315316
{
316317
$this->expectException(TypeError::class);
317-
Domain::fromIDNA2008('example.com')->withLabel(1, new \stdClass());
318+
Domain::fromIDNA2008('example.com')->withLabel(1, new stdClass());
318319
}
319320

320321
public function testWithLabelFailsWithInvalidKey(): void

‎src/Idna.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ public static function toAscii(string $domain, int $options): IdnaInfo
9999

100100
self::supportsIdna();
101101

102-
/** @param-out array{errors: int, isTransitionalDifferent: bool, result: string} $idnaInfo */
102+
/* @param-out array{errors: int, isTransitionalDifferent: bool, result: string} $idnaInfo */
103103
idn_to_ascii($domain, $options, INTL_IDNA_VARIANT_UTS46, $idnaInfo);
104104

105105
/** @var array{result:string, isTransitionalDifferent:bool, errors:int} $idnaInfo */
@@ -124,7 +124,7 @@ public static function toUnicode(string $domain, int $options): IdnaInfo
124124

125125
self::supportsIdna();
126126

127-
/** @param-out array{errors: int, isTransitionalDifferent: bool, result: string} $idnaInfo */
127+
/* @param-out array{errors: int, isTransitionalDifferent: bool, result: string} $idnaInfo */
128128
idn_to_utf8($domain, $options, INTL_IDNA_VARIANT_UTS46, $idnaInfo);
129129

130130
/** @var array{result:string, isTransitionalDifferent:bool, errors:int} $idnaInfo */

‎src/Rules.php‎

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,12 +36,12 @@ final class Rules implements PublicSuffixList
3636
/**
3737
* PSL rules as a multidimensional associative array.
3838
*
39-
* @var array<string, array<array>>
39+
* @var array{ICANN_DOMAINS: array<array>, PRIVATE_DOMAINS: array<array>}
4040
*/
4141
private array $rules;
4242

4343
/**
44-
* @param array<string, array<array>> $rules
44+
* @param array{ICANN_DOMAINS: array<array>, PRIVATE_DOMAINS: array<array>} $rules
4545
*/
4646
private function __construct(array $rules)
4747
{
@@ -84,7 +84,7 @@ public static function fromString($content): self
8484
/**
8585
* Convert the Public Suffix List into an associative, multidimensional array.
8686
*
87-
* @return array<string, array<array>>
87+
* @return array{ICANN_DOMAINS: array<array>, PRIVATE_DOMAINS: array<array>}
8888
*/
8989
private static function parse(string $content): array
9090
{
@@ -165,7 +165,7 @@ private static function addRule(array $list, array $ruleParts): array
165165
}
166166

167167
/**
168-
* @param array{rules:array<string, array<array>>} $properties
168+
* @param array{rules:array{ICANN_DOMAINS: array<array>, PRIVATE_DOMAINS: array<array>}} $properties
169169
*/
170170
public static function __set_state(array $properties): self
171171
{
@@ -290,18 +290,23 @@ private function getPublicSuffixLengthFromSection(DomainName $domain, string $se
290290
$labelCount = 0;
291291
foreach ($domain->toAscii() as $label) {
292292
//match exception rule
293-
if (isset($rules[$label], $rules[$label]['!'])) {
293+
if (isset($rules[$label]['!'])) {
294294
break;
295295
}
296296

297297
//match wildcard rule
298-
if (isset($rules['*'])) {
298+
if (array_key_exists('*', $rules)) {
299299
++$labelCount;
300300
break;
301301
}
302302

303303
//no match found
304-
if (!isset($rules[$label])) {
304+
if (!array_key_exists($label, $rules)) {
305+
// for private domain suffix MUST be fully matched else no suffix is found
306+
// https://github.com/jeremykendall/php-domain-parser/issues/321
307+
if (self::PRIVATE_DOMAINS === $section && [] !== $rules) {
308+
$labelCount = 0;
309+
}
305310
break;
306311
}
307312

0 commit comments

Comments
 (0)