[Engine-devel] Atlassian Crowd Support for Ovirt

2) adding proper memberOf support so groups work correctly.</p><p> = </p><p>Please let me know if you guys are interested in this, and please = be gentle on your reviews - I=E1=B8=BF not a Java Programmer by any means= , and probably brought too many C++ traits over here... :)</p><p> </=
This is a multi-part message in MIME format. Your mail reader does not understand MIME message format. --=_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Hi All,=0D=0A=0D=0AI've just uploaded to Gerrit a patch that adds atlassi= an crowd authentication support to oVirt.(http://gerrit.ovirt.org/9324)=0D= =0A=0D=0A=C2=A0=0D=0AAll the nitty gritty details are in the commit messa= ge, but basically this adds a new authentication domain, and when selecte= d sends the authentication request off to a crowd server.=0D=0A=0D=0A=C2=A0= =0D=0AWe have been using this patch on 3.1 for a while, and I thought its= overdue to see if you guys are interested in taking this upstream=3F=0D=0A= =0D=0A=C2=A0=0D=0AThere are two areas that I know could use improvements:= =0D=0A=0D=0A1) caching the results from the crowd server, so we don't que= ry for every API request etc.=0D=0A=0D=0A2) adding proper memberOf suppor= t so groups work correctly.=0D=0A=0D=0A=C2=A0=0D=0APlease let me know if = you guys are interested in this, and please be gentle on your reviews - I= =E1=B8=BF not a Java Programmer by any means, and probably brought too ma= ny C++ traits over here... :)=0D=0A=0D=0A=C2=A0=0D=0AThanks=0D=0A=0D=0A=C2= =A0=0D=0AJustin=0D=0A=0D=0A --=_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS Content-Type: text/html; charset=utf-8 Content-Transfer-Encoding: quoted-printable <!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://ww= w.w3.org/TR/html4/loose.dtd"><html>=0A<head>=0A <meta name=3D"Generator"= content=3D"Zarafa WebAccess v7.0.3-30515">=0A <meta http-equiv=3D"Conte= nt-Type" content=3D"text/html; charset=3Dutf-8">=0A <title>Atlassian Cro= wd Support for Ovirt</title>=0A <style type=3D"text/css">=0A body=0D= =0A {=0D=0A font-family: Arial, Verdana, Sans-Serif ! importa= nt;=0D=0A font-size: 12px;=0D=0A padding: 5px 5px 5px 5px;=0D= =0A margin: 0px;=0D=0A border-style: none;=0D=0A bac= kground-color: #ffffff;=0D=0A }=0D=0A=0D=0A p, ul, li=0D=0A = {=0D=0A margin-top: 0px;=0D=0A margin-bottom: 0px;=0D=0A = }=0D=0A </style>=0A</head>=0A<body>=0A<p>Hi All,</p><p>I've jus= t uploaded to Gerrit a patch that adds atlassian crowd authentication sup= port to oVirt.(http://gerrit.ovirt.org/9324)</p><p> </p><p>All the n= itty gritty details are in the commit message, but basically this adds a = new authentication domain, and when selected sends the authentication req= uest off to a crowd server.</p><p> </p><p>We have been using this pa= tch on 3.1 for a while, and I thought its overdue to see if you guys are = interested in taking this upstream=3F</p><p> </p><p>There are two ar= eas that I know could use improvements:</p><p>1) caching the results from= the crowd server, so we don't query for every API request etc.</p><p= p><p>Thanks</p><p> </p><p>Justin</p>=0A</body>=0A</html> --=_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS--

On 11/19/2012 03:59 PM, Justin Hammond wrote:
Hi All,
I've just uploaded to Gerrit a patch that adds atlassian crowd authentication support to oVirt.(http://gerrit.ovirt.org/9324)
All the nitty gritty details are in the commit message, but basically this adds a new authentication domain, and when selected sends the authentication request off to a crowd server.
We have been using this patch on 3.1 for a while, and I thought its overdue to see if you guys are interested in taking this upstream?
There are two areas that I know could use improvements:
1) caching the results from the crowd server, so we don't query for every API request etc.
2) adding proper memberOf support so groups work correctly.
Please let me know if you guys are interested in this, and please be gentle on your reviews - Iḿ not a Java Programmer by any means, and probably brought too many C++ traits over here... :)
i thought worth mentioning to anyone seeing this thread that there is an active discussion on the patch. main non technical issue is trying to understand which license the crowd library would be under.
participants (2)
-
Itamar Heim
-
Justin Hammond