Opened 9 years ago

Closed 9 years ago

#767 closed enhancement (fixed)

Use CLARIN central discovery service on MPI-PL corpus1

Reported by: Sander Maijers Owned by: andmor
Priority: major Milestone:
Component: AAI Version:
Keywords: Cc: Willem Elbers, andmor, Dieter Van Uytvanck, Sander Maijers

Description


Attachments (1)

Screen Shot 2015-06-15 at 16.43.07.png (37.7 KB) - added by Willem Elbers 9 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 9 years ago by DefaultCC Plugin

Cc: Sander Maijers added

comment:2 Changed 9 years ago by Sander Maijers

lux07:/lat/shibboleth-sp/etc/shibboleth/shibboleth2.xml
contains a few MetadataProviders?, someone should decide which to keep
The SPF metadata seem commented out.

comment:3 Changed 9 years ago by Dieter Van Uytvanck

I think it should include:

  • the default (AMS) MPI IdP (static, provided by MPI)
  • the CLARIN IdP (static, provided by CLARIN)
  • all the CLARIN SPF IdPs (dynamic, provided by CLARIN)

All individual references to the national federations should be taken out, as they are now integrated into the CLARIN SPF metadata.

Changed 9 years ago by Willem Elbers

comment:4 Changed 9 years ago by Willem Elbers

I think the ticket title is a bit too restrictive.

What needs to be done in my opinion is not to replace the current discovery service with the CLARIN one, but instead change the current discovery service in such a way that the "Click here to show external identity providers" shown in the picture below, delegates/redirects to the clarin discovery service.


The current MPI discovery service can then be replaced with a static page with similar appearance (instead of the current java / javascript application).

comment:5 Changed 9 years ago by Dieter Van Uytvanck

I support Willem's suggestion as it should decrease the complexity (and maintenance) at the side of the TLA/MPI.

We should also use this opportunity to remove a bit of the jargon (identity provider, service provider) in the user interface and make it more intuitive: e.g. just two large buttons:

  • "login with your MPI for Psycholinguistics account" (default)
  • "login with your (external) academic account"

comment:6 Changed 9 years ago by andre.moreira@mpi.nl

I agree! I just have some doubts about how much implementation/configuration work will that be. But as I understand, we can take our time to do this.

comment:7 Changed 9 years ago by Sander Maijers

I just quickly made a ticket as requested, 'use' does not necessarily mean 'replace', but as you all can think of a better title, please just change it to that title then. Observation: three of us agreeing about something simple as the ticket title and/or aim of the effort in a row instead of just making the fix.

I think we (at least André, Willem, and I) should just sit together one time and do it. An hour should probably be enough. That way I'm sure it ś going to be fixed earlier than if we keep coordinating a ticket.

comment:8 Changed 9 years ago by andre.moreira@mpi.nl

Guys, yesterday I (with Sander's help) placed on lux17 a draft of what could be the solution for this request.It consists of one single html page with 3 lines of JavaScript? (one extra to fix what we've missed, Sander). This page is served directly by apache and the old disco juice servlet was shut down. I am not sure about further implications of this, perhaps Willem can elaborate a bit on it?
Anyway, take a look at the login to lux17 and let me know your thoughts.

As abput discussing the ticket. A good definition of the problem/task is always a good thing and in my case I need to know what is asked so I can estimate the workload and priority, so I can schedule the task.

comment:9 Changed 9 years ago by Dieter Van Uytvanck

Tested with my AMS IdP account and UU.nl account and it works, thanks! Now it is probably a matter of providing some more user friendly texts (see my comment above)

comment:10 Changed 9 years ago by Sander Maijers

Nice. Another string in need of fixing is:
"Your browser is not support javascript. To use the fallback discovery service" =>
"This application requires that JavaScript? is enabled in your web browser, but it isn't."

I also propose that we do not call out the hostname of the host on which this static DS happens to be hosted.

"Sign in to Lux17 Shibboleth SDS" => "Sign in"
"Select your Identity Provider here in the DiscoJuice? Shibboleth Discovery Service"
=> "Select your home organization"

Last edited 9 years ago by Sander Maijers (previous) (diff)

comment:11 Changed 9 years ago by andre.moreira@mpi.nl

I have created a new clean page from scratch with only the necessary CSS and HTML and taking into account all comments on this ticket:

Please verify the page using any of our applications on:
http://lux17.mpi.nl

I have also commited the code to the TLA SVN repository, see it here:
https://trac.mpi.nl/browser/latsvn/ShibbLoginLandingPage

I will propagate the page to corpus1 once everybody likes the page.

comment:12 Changed 9 years ago by Sander Maijers

Everbody likes the page. :)

comment:13 Changed 9 years ago by andre.moreira@mpi.nl

Resolution: fixed
Status: newclosed

Propagated to corpus1.mpi.nl

Note: See TracTickets for help on using tickets.