mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #63437
[Bug 1945959] Re: Configuration of custom badgr server URL for Open Badges displayer sources is broken
Hi Stefan,
Thank you for having identified this hard-coded value. Yes, this should
be fixed. So it's going into our backlog. You are welcome to provide a
full solution if you like to fix it faster for Mahara core.
Technically, a better fix would be to allow for multiple Badgr instances
to be connected to one Mahara site, esp. when thinking about multi-
tenancy:
- Allow for all Badgr URLs to be displayed on the 'Connected apps' -> Badgr page so that the login form goes for the corresponding site.
- All connected Badgr backpacks are being searched in the block.
In the meantime, since it sounds like you work with one Badgr URL, you
can replace the community Badgr site URL with yours to keep going.
Thank you
Kristina
** Changed in: mahara
Importance: Undecided => Medium
** Changed in: mahara
Status: In Progress => Confirmed
** Changed in: mahara
Assignee: Doris Tam (doristam) => (unassigned)
** Summary changed:
- Configuration of custom badgr server URL for Open Badges displayer sources is broken
+ Configuration of custom Badgr server URL for Open Badges displayer sources is broken
--
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1945959
Title:
Configuration of custom Badgr server URL for Open Badges displayer
sources is broken
Status in Mahara:
Confirmed
Bug description:
Mahara 20.04.2 (same bug in current master branch on github)
Ubuntu Server 18.04 LTS
MariaDB 10.3
Firefox 92.0.1
Configuration of custom badgr server URL breaks functionality. Token
can be generated, but no collections can be accessed.
Bug in blocktype/openbadgedisplayer/lib.php
Lines 513 and 613 check for hardcoded API-URL
Replace
if ($backpack_url == 'https://api.badgr.io/') {
with
if ($host == 'badgr') {
to check for server type instead of server-URL and make it work.
Not sure what to do with line 419f though...
To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1945959/+subscriptions
References