- 🇮🇳India rinku jacob 13 Kerala
Reviewed Merge request !3239 on drupal version 10.1.x-dev. After the merge request we can see trusted_host_patterns on line number 708 of default.settings.php file. Need RTBC+1
- 🇺🇸United States smustgrave
Tagging for release manager thought.
There is already documentation for trusted_host_patterns. The example being added could lead to bad practice of users not setting a trusted host I would be afraid of
- 🇺🇸United States xjm
We should manually test whether the status report still displays the expected error if it's set to an empty array instead of NULL. If the error is still displayed, I think this is OK.
- Status changed to RTBC
about 2 years ago 10:18pm 16 February 2023 - 🇺🇸United States smustgrave
So did add
$settings['trusted_host_patterns'] = [];
to my settings and cleared cached and I didn't get an error.
- 🇬🇧United Kingdom longwave UK
Committed and pushed 945c48ff03 to 10.1.x. Thanks!
- Status changed to Fixed
about 2 years ago 10:36pm 16 February 2023 -
longwave →
committed 945c48ff on 10.1.x
Issue #3333281 by MustangGB, smustgrave, xjm: Give trusted host...
-
longwave →
committed 945c48ff on 10.1.x
Automatically closed - issue fixed for 2 weeks with no activity.