- πΊπ¦Ukraine ankondrat4 Lutsk
Hello.
Is this issue related to this issue too https://www.drupal.org/project/recaptcha/issues/3302403 β ? Whose check it with this patch?
- last update
almost 2 years ago 3 pass - Status changed to Needs work
over 1 year ago 7:50am 13 December 2023 - π©πͺGermany Anybody Porta Westfalica
Please turn this into a MR instead. And please provide documentation for this change, e.g. symfony docs or something like that showing it's correct.
Thank you
- Merge request !27Issue #3179035 Fix getting correct hostname by Drupal request β (Merged) created by ankondrat4
- last update
over 1 year ago Composer require failure - Status changed to Needs review
over 1 year ago 10:04am 13 December 2023 - πΊπ¦Ukraine ankondrat4 Lutsk
Hello.
I have created MR for these changes.
Documentation here:
https://github.com/symfony/http-foundation/blob/6.3/Request.php#L1126C38...
https://api.hostdog.eu/api/drupal-9/drupal%21core%21vendor%21symfony%21h... - π©πͺGermany Anybody Porta Westfalica
Thanks! LGTM and ->getHost() seems correct. Now we please need some people for community testing.
Alternative might be
$request->server->get('HTTP_HOST');
- Status changed to Needs work
over 1 year ago 8:49am 22 December 2023 - last update
over 1 year ago 4 pass - Status changed to Needs review
over 1 year ago 9:41am 22 December 2023 - π©πͺGermany Anybody Porta Westfalica
Rebased. If someone tests this and can confirm it works as expected, we can merge this. The tests are green now.
- π©πͺGermany Anybody Porta Westfalica
Tested and works as expected, thank you! :)
- last update
over 1 year ago 4 pass -
Anybody β
committed 112003cb on 8.x-3.x authored by
ankondrat4 β
Issue #3179035 by Anybody, interX: Support validation behind a reverse...
-
Anybody β
committed 112003cb on 8.x-3.x authored by
ankondrat4 β
- Status changed to Fixed
over 1 year ago 10:20am 22 December 2023 Automatically closed - issue fixed for 2 weeks with no activity.