Hardcode api name for AUC (emergency)
Ken Henslee
8:34 AM
Hi
@baxter.brad
Atlanta University Center's OpenAthens EntityID and Scope are
https://idp.auctr.edu/openathens / auctr.edu, which is what we have in their GALILEO institution record. But their Redirector URL is https://go.openathens.net/redirector/aucenter.edu?url= , which is different from the EntityID and Scope...Right now their GALILEO "Redirector URL" (https://www.galileo.usg.edu/redirect?inst=auc1&url=) is using https://go.openathens.net/redirector/auctr.edu?url=. Can this be changed to https://go.openathens.net/redirector/aucenter.edu?url= ?
Brad Baxter
8:49 AM
@kenslee
This is a special case, as I understand it. AUC is the only institution whose API name (aucenter.edu) (which is used in the redirector URL) is different from their Scope (auctr.edu). So we could handle this a couple of ways:
Special case AUC in galileo to use a different value (aucenter.edu) for redirector links
Add an API name field to all institutions. In every case except AUC, the value in the API name field will be the same as the Scope.
Ken Henslee
8:55 AM
Thanks
@baxter.brad
I think option one works best. I don't see this ever being the arrangement for any institutions in the future. Would the arrangement for option one be that you hardcode aucenter.edu in the program that encodes the target URL for OpenAthens?
Brad Baxter
9:06 AM
Probably.
Mike White
9:13 AM
Good point, Ken. I like option one as well
:+1::skin-tone-2:
1
9:14
It wouldn't be GALILEO if there weren't exceptions to everything :smile:
:brent_approves:
1
Russell Palmer
9:14 AM
yeah i was just thinking of the notes
Brad Baxter
9:16 AM
There's another option, which may be better from a maintenance perspective [1]:
3. Add an API name field to the institution record. Put aucenter.edu in that field for AUC. All other institutions would be blank.
Then the logic for the redirector would be:
if API name is not blank, use API name, else use Scope.
[1] having special cases hardcoded in the program instead of stored in the database is not ideal for maintenance.
9:17
We should strive to make exceptions the exception.
:slightly_smiling_face:
1
Mike White
9:20 AM
That makes sense. Whatever is going to make it easier to maintain. I will try not to resent AUC each time I see that empty field.
Ken Henslee
9:22 AM
That makes sense to me as well. Sorry this wasn't brought up sooner. I guess you are aware they are going live with OA tomorrow? :sweat_smile:
Russell Palmer
9:23 AM
a rare case where we are asking for emergency development
Mike White
9:25 AM
If it's easier to hardcode for tomorrow, then setup the API field later, that could work
Edited by Brad Baxter