Skip to content

Have the ability to add WellKnownEndpoints for OpenID handling #903

Closed
olger opened this Issue Feb 19, 2011 · 2 comments

3 participants

@olger
olger commented Feb 19, 2011

At this moment there is a fixed list of well known endpoints available.
It is handy to have the option to add your own known endpoints to that.

Mail thread pasted in this message (as I cannot find the thread in the google group.

On Fri, Feb 18, 2011 at 10:24 AM, Olger Warnier olger.warnier@gmail.com wrote:
Hi List,

I'd like to support logon based on a google apps domain in my application.
In order to do such a thing, you need a setup (actually a single file called: openid) on your domain webserver that will redirect the actual openId check to the google openid server for app domains.

The liftweb openid supports the general google openid host, but in the WellKnownEndpoints there is no entry for the Google App handling.

Could we add the following to the WellKnownEndpoints ?

val GoogleApps = new DiscoveredEndpoint("Google","https://www\.google\.com/a/.+\..+/o8/.+") {
override def makeAttributeExtension(attrs: List[Attribute]): Box[MessageExtension] =
Full(fetchRequestExtension(attrs.zipAll(Nil, null, true)))
}

(and to the endpoints list of course)

  • or -

In what way do I register an aditional endpoint in order to have it found and mapped as done for the WellKnownEndpoints ?

For completeness:
This code is used with the beforeAuth = Full(ext _) stuff as found on the wiki. (Could add some info there on the support for apps domain)

Regards,

Olger

Afair, there's currently no support for registering additional endpoints. This seems like a useful feature, so please open a ticket at ticket.liftweb.net, referencing this thread (you must be a watcher of the Liftweb space)

Also, you can update the wiki yourself once you become a watcher :-)

/Jeppe

@Shadowfiend
Lift Web Framework member

Don't know the latest state of this ticket, but if you run across it and it's still an issue, please file it on the liftmodules/openid repository, as the openid module is no longer in the main Lift repo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.