7:59 a.m.
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&g...
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=
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> </=
p><p>Thanks</p><p> </p><p>Justin</p>=0A</body>=0A</html>
--=_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS--
5:06 p.m.
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.
4427
days inactive
4429
days old
1 comments
2 participants
participants (2)
-
Itamar Heim -
Justin Hammond