Bug 28332

Summary: Requesting Mana token with accented name fails
Product: Koha Reporter: Alex Arnaud <alex.arnaud>
Component: Mana-kbAssignee: Bugs List <koha-bugs>
Status: Failed QA --- QA Contact:
Severity: normal    
Priority: P5 - low CC: alex.arnaud, david, hayleypelham, jonathan.druart
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
This fixes the Mana configuration so that accented names (such as Université Foo) can be used when requesting the Mana token. Before this, the account registration would fail and no token was received back.
Version(s) released in:
Circulation function:
Attachments: Bug 28332: Fix data encoding when requesting Mana token
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl
Bug 28332: Fix data encoding when requesting Mana token
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl
Bug 28332: Fix data encoding when requesting Mana token
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl
Bug 28332: Fix data encoding when requesting Mana token
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl

Description Alex Arnaud 2021-05-12 14:33:54 UTC
Test plan:
  - Go to Administration  Share content with Mana KB (cgi-bin/koha/admin/share_content.pl)
  - in "Configure Mana KB" part, set your email and a name with accent (Université Foo),
  - click on "Send to Mana KB"
  => Fail. No mana token.
Comment 1 Alex Arnaud 2021-05-12 14:36:15 UTC
Created attachment 120884 [details] [review]
Bug 28332: Fix data encoding when requesting Mana token
Comment 2 Hayley Pelham 2021-12-21 23:02:34 UTC
I get an error after applying the patch and using a name with an accent to request the token

Undefined subroutine &CGI::Compile::ROOT::home_vagrant_kohaclone_admin_share_content_2epl::encode_json called at /home/vagrant/kohaclone/admin/share_content.pl line 70
Comment 3 Hayley Pelham 2021-12-21 23:13:47 UTC
Created attachment 128851 [details] [review]
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl

Test plan:
  - Go to Administration  Share content with Mana KB (cgi-bin/koha/admin/share_content.pl)
  - In "Configure Mana KB" part, set your email and a name with accent (Université Foo),
  - Click on "Send to Mana KB"
  => Fail. No mana token.
Comment 4 Hayley Pelham 2021-12-21 23:15:19 UTC
Created attachment 128852 [details] [review]
Bug 28332: Fix data encoding when requesting Mana token
Comment 5 Hayley Pelham 2021-12-21 23:15:43 UTC
Created attachment 128853 [details] [review]
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl

Test plan:
  - Go to Administration  Share content with Mana KB (cgi-bin/koha/admin/share_content.pl)
  - In "Configure Mana KB" part, set your email and a name with accent (Université Foo),
  - Click on "Send to Mana KB"
  => Fail. No mana token.

Sponsored-by: Catalyst IT
Comment 6 Hayley Pelham 2021-12-21 23:17:54 UTC
Created attachment 128854 [details] [review]
Bug 28332: Fix data encoding when requesting Mana token

Authored-by: Alex Arnaud
Comment 7 Hayley Pelham 2021-12-21 23:18:41 UTC
Created attachment 128855 [details] [review]
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl

Test plan:
  - Go to Administration  Share content with Mana KB (cgi-bin/koha/admin/share_content.pl)
  - In "Configure Mana KB" part, set your email and a name with accent (Université Foo),
  - Click on "Send to Mana KB"
  => Fail. No mana token.

Sponsored-by: Catalyst IT
Comment 8 David Nind 2021-12-31 18:30:13 UTC
Created attachment 128944 [details] [review]
Bug 28332: Fix data encoding when requesting Mana token

Authored-by: Alex Arnaud

Signed-off-by: David Nind <david@davidnind.com>
Comment 9 David Nind 2021-12-31 18:30:18 UTC
Created attachment 128945 [details] [review]
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl

Test plan:
  - Go to Administration  Share content with Mana KB (cgi-bin/koha/admin/share_content.pl)
  - In "Configure Mana KB" part, set your email and a name with accent (Université Foo),
  - Click on "Send to Mana KB"
  => Fail. No mana token.

Sponsored-by: Catalyst IT

Signed-off-by: David Nind <david@davidnind.com>
Comment 10 Jonathan Druart 2022-01-07 15:30:17 UTC
Alex, I've tried with "✔ ❤ ★" and I get an error from the server: "An error occurred, mana server returned: Internal Server Error". Do you have access to the test server to investigate what's happening?