Skip to content

Fix: CSRF check failed#54

Merged
Rom1-B merged 3 commits intomainfrom
support_40368
Nov 18, 2025
Merged

Fix: CSRF check failed#54
Rom1-B merged 3 commits intomainfrom
support_40368

Conversation

@Rom1-B
Copy link
Copy Markdown

@Rom1-B Rom1-B commented Nov 18, 2025

Checklist before requesting a review

Please delete options that are not relevant.

  • I have performed a self-review of my code.
  • I have added tests (when available) that prove my fix is effective or that my feature works.
  • I have updated the CHANGELOG with a short functional description of the fix or new feature.
  • This change requires a documentation update.

Description

  • It fixes !40368

Avoids a CSRF check error if print is clicked multiple times.

CSRF check failed for User ID: 2 at .../plugins/pdf/ajax/export.php
User ID: `2` tried to execute an invalid request on `/plugins/pdf/ajax/export.php`.

  Backtrace :
  ./src/Session.php:1783                             
  ...ner/ControllerListener/CheckCsrfListener.php:89 Session::checkCSRF()
  .../event-dispatcher/Debug/WrappedListener.php:116 Glpi\Kernel\Listener\ControllerListener\CheckCsrfListener->onKernelController()
  ...ymfony/event-dispatcher/EventDispatcher.php:220 Symfony\Component\EventDispatcher\Debug\WrappedListener->__invoke()
  ...symfony/event-dispatcher/EventDispatcher.php:56 Symfony\Component\EventDispatcher\EventDispatcher->callListeners()
  ...spatcher/Debug/TraceableEventDispatcher.php:139 Symfony\Component\EventDispatcher\EventDispatcher->dispatch()
  ./vendor/symfony/http-kernel/HttpKernel.php:169    Symfony\Component\EventDispatcher\Debug\TraceableEventDispatcher->dispatch()
  ./vendor/symfony/http-kernel/HttpKernel.php:76     Symfony\Component\HttpKernel\HttpKernel->handleRaw()
  ./vendor/symfony/http-kernel/Kernel.php:197        Symfony\Component\HttpKernel\HttpKernel->handle()
  ./public/index.php:70                              Symfony\Component\HttpKernel\Kernel->handle()

AND Fixes some SQL errors during export

Screenshots (if appropriate):

image

@Rom1-B Rom1-B requested a review from stonebuzz November 18, 2025 13:10
@Rom1-B Rom1-B merged commit e68890f into main Nov 18, 2025
3 checks passed
@Rom1-B Rom1-B deleted the support_40368 branch November 18, 2025 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants