Bugzilla – Attachment 145303 Details for
Bug 31492
Patron image upload fails on first attempt with CSRF failure
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 31492: Set CSRF token in circulation.pl
Bug-31492-Set-CSRF-token-in-circulationpl.patch (text/plain), 1.35 KB, created by
Lucas Gass (lukeg)
on 2023-01-13 19:03:46 UTC
(
hide
)
Description:
Bug 31492: Set CSRF token in circulation.pl
Filename:
MIME Type:
Creator:
Lucas Gass (lukeg)
Created:
2023-01-13 19:03:46 UTC
Size:
1.35 KB
patch
obsolete
>From 5941a15b469fddf7d052f86529405cae1a60c1e2 Mon Sep 17 00:00:00 2001 >From: Lucas Gass <lucas@bywatersolutions.com> >Date: Fri, 13 Jan 2023 19:01:55 +0000 >Subject: [PATCH] Bug 31492: Set CSRF token in circulation.pl > >To test: >* Turn on patronimages system preferences >* Go to any patron account >* Hover over the 'dummy image' for the Add button >* Browse and select an image for upload >* Saving results in the following error: > >The form submission failed (Wrong CSRF token). Try to come back, refresh the page, then try again. > >Apply patch and try again. You shouldn't need to try twice, it should work on the first attempt now. >--- > circ/circulation.pl | 6 ++++++ > 1 file changed, 6 insertions(+) > >diff --git a/circ/circulation.pl b/circ/circulation.pl >index b7270bfe56a..4bbc92b5346 100755 >--- a/circ/circulation.pl >+++ b/circ/circulation.pl >@@ -54,6 +54,7 @@ use Koha::Items; > use Koha::SearchEngine; > use Koha::SearchEngine::Search; > use Koha::Patron::Modifications; >+use Koha::Token; > > use List::MoreUtils qw( uniq ); > >@@ -636,4 +637,9 @@ $template->param( > logged_in_user => $logged_in_user, > ); > >+# Generate CSRF token for upload and delete image buttons >+$template->param( >+ csrf_token => Koha::Token->new->generate_csrf({ session_id => $query->cookie('CGISESSID'),}), >+); >+ > output_html_with_http_headers $query, $cookie, $template->output; >-- >2.30.2
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 31492
:
145303
|
145346
|
145347