play framework

Bug #538861
Comment #1

Comment 1 for bug 538861

Revision history for this message

AMcBain (mcbain-asm) wrote on 2010-03-14:

Patch to allow OpenID forRealm and returnTo actions to to be URLs starting with http:// Edit (965 bytes, text/plain)

Another usage for this patch is if an app wants to have a login on one URL but have it handled by another on a different domain, for example. The returnTo and forRealm actions would have to be URLs for the destination.

Obviously this patch is very simplistic, given that not all URLs accessible by the internet start with http://, but I think for OpenID, only http://-reachable locations would be handling the results.

I tried to keep the apparent code formatting in the OpenID.java file, but some spacing may be needed before, after and between the if statements. Anyone wishing to try this patch can easily do so by making the indicated changes and compiling OpenID.java against play.jar then taking the two generated classfiles and replacing those in the Play! jar.