Bug 28332 - Requesting Mana token with accented name fails
Summary: Requesting Mana token with accented name fails
Status: Failed QA
Alias: None
Product: Koha
Classification: Unclassified
Component: Mana-kb (show other bugs)
Version: Main
Hardware: All All
: P5 - low normal
Assignee: Bugs List
QA Contact:
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-05-12 14:33 UTC by Alex Arnaud
Modified: 2022-01-27 16:25 UTC (History)
4 users (show)

See Also:
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 (793 bytes, patch)
2021-05-12 14:36 UTC, Alex Arnaud 		Details | Diff | Splinter Review
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl (1.01 KB, patch)
2021-12-21 23:13 UTC, Hayley Pelham 		Details | Diff | Splinter Review
Bug 28332: Fix data encoding when requesting Mana token (800 bytes, patch)
2021-12-21 23:15 UTC, Hayley Pelham 		Details | Diff | Splinter Review
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl (1.03 KB, patch)
2021-12-21 23:15 UTC, Hayley Pelham 		Details | Diff | Splinter Review
Bug 28332: Fix data encoding when requesting Mana token (825 bytes, patch)
2021-12-21 23:17 UTC, Hayley Pelham 		Details | Diff | Splinter Review
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl (1.03 KB, patch)
2021-12-21 23:18 UTC, Hayley Pelham 		Details | Diff | Splinter Review
Bug 28332: Fix data encoding when requesting Mana token (874 bytes, patch)
2021-12-31 18:30 UTC, David Nind 		Details | Diff | Splinter Review
Bug 28332: (follow-up) Include encode_json method in admin/share_content.pl (1.08 KB, patch)
2021-12-31 18:30 UTC, David Nind 		Details | Diff | Splinter Review
Show Obsolete (6) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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?