The Needs Review Queue Bot โ tested this issue. It either no longer applies to Drupal core, or fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
Apart from a re-roll or rebase, this issue may need more work to address feedback in the issue or MR comments. To progress an issue, incorporate this feedback as part of the process of updating the issue. This helps other contributors to know what is outstanding.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.
- ๐บ๐ธUnited States SocialNicheGuru
No longer applies to Drupal 9.5.9.
- ๐บ๐ธUnited States SocialNicheGuru
I rerolled the patch for Drupal 9.5.9
I have to learn to how to do a proper interdiff.
The changes below are just text changes in commentspatching file 'core/core.services.yml'
patching file 'core/lib/Drupal/Core/Access/CsrfAccessCheck.php'
patching file 'core/lib/Drupal/Core/Access/RouteProcessorCsrf.php'
patching file 'core/modules/menu_link_content/tests/src/Kernel/MenuLinkContentCacheabilityBubblingTest.php'
1 out of 4 hunks failed--saving rejects to 'core/modules/menu_link_content/tests/src/Kernel/MenuLinkContentCacheabilityBubblingTest.php.rej'
patching file 'core/modules/system/tests/src/Kernel/Common/UrlTest.php'
1 out of 7 hunks failed--saving rejects to 'core/modules/system/tests/src/Kernel/Common/UrlTest.php.rej'
patching file 'core/tests/Drupal/Tests/Core/Access/CsrfAccessCheckTest.php'
patching file 'core/tests/Drupal/Tests/Core/Access/RouteProcessorCsrfTest.php'more core/modules/menu_link_content/tests/src/Kernel/MenuLinkContentCacheabilityBubblingTest.php.rej
@@ -15,6 +15,8 @@
use Symfony\Component\Routing\Route;/**
+ * The menu link content cache bubbling kernel tests.
+ *
* Ensures that rendered menu links bubble the necessary bubbleable metadata
* for outbound path/route processing.
*more core/modules/system/tests/src/Kernel/Common/UrlTest.php.rej
@@ -12,6 +12,8 @@
use Drupal\KernelTests\KernelTestBase;/**
+ * Url kernel tests.
+ *
* Confirm that \Drupal\Core\Url,
* \Drupal\Component\Utility\UrlHelper::filterQueryParameters(),
* \Drupal\Component\Utility\UrlHelper::buildQuery(), and - last update
about 1 year ago Patch Failed to Apply 58:46 55:34 Running- First commit to issue fork.
- last update
12 months ago Custom Commands Failed - @rpayanm opened merge request.
- last update
12 months ago Custom Commands Failed - last update
12 months ago 29,803 pass - Status changed to Needs review
12 months ago 4:28pm 4 July 2023 - Status changed to Needs work
12 months ago 1:28am 5 July 2023 - Status changed to Needs review
12 months ago 1:34pm 7 July 2023 - ๐บ๐พUruguay rpayanm
Sorry, I can see the all tests passed. Which tests failed?
- Status changed to Needs work
12 months ago 2:19pm 10 July 2023 - last update
8 months ago 29,680 pass - last update
8 months ago 29,680 pass - ๐ต๐นPortugal joum Porto
Can confirm #128 applies cleanly and works on Drupal 10.1.6
- ๐บ๐ธUnited States bradjones1 Digital Nomad Life
This needs an MR against 11.x at this point.
- last update
5 months ago Build Successful - ๐จ๐ฆCanada joseph.olstad
It's possible to remove the CSRF check by using a directive set to the modulename.routing.yml file
see how this dropzonejs patch is doing it:
#3197207-02: Anonymous users cannot upload caused by invalid csrf-token โ - ๐บ๐ธUnited States SocialNicheGuru
How does this committed issue ๐ User logout is vulnerable to CSRF Fixed and the change record https://www.drupal.org/node/2822514 โ affect this issue, ๐ CSRF check always fails for users without a session Needs work ?
Leaving as needs work.
- ๐บ๐ธUnited States gcalex5
#132 failed to apply for me on 10.3.0. Re-rolled against branch 10.3.x