From justin at dynam.ac Mon Nov 19 08:59:53 2012 Content-Type: multipart/mixed; boundary="===============2165405942651123678==" MIME-Version: 1.0 From: Justin Hammond To: devel at ovirt.org Subject: [Engine-devel] Atlassian Crowd Support for Ovirt Date: Mon, 19 Nov 2012 13:59:43 +0000 Message-ID: --===============2165405942651123678== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable This is a multi-part message in MIME format. Your mail reader does not understand MIME message format. --=3D_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS Content-Type: text/plain; charset=3Dutf-8 Content-Transfer-Encoding: quoted-printable Hi All,=3D0D=3D0A=3D0D=3D0AI've just uploaded to Gerrit a patch that adds a= tlassi=3D an crowd authentication support to oVirt.(http://gerrit.ovirt.org/9324)=3D0= D=3D =3D0A=3D0D=3D0A=3DC2=3DA0=3D0D=3D0AAll the nitty gritty details are in the = commit messa=3D ge, but basically this adds a new authentication domain, and when selecte= =3D d sends the authentication request off to a crowd server.=3D0D=3D0A=3D0D=3D= 0A=3DC2=3DA0=3D =3D0D=3D0AWe have been using this patch on 3.1 for a while, and I thought i= ts=3D overdue to see if you guys are interested in taking this upstream=3D3F=3D0= D=3D0A=3D =3D0D=3D0A=3DC2=3DA0=3D0D=3D0AThere are two areas that I know could use imp= rovements:=3D =3D0D=3D0A=3D0D=3D0A1) caching the results from the crowd server, so we don= 't que=3D ry for every API request etc.=3D0D=3D0A=3D0D=3D0A2) adding proper memberOf = suppor=3D t so groups work correctly.=3D0D=3D0A=3D0D=3D0A=3DC2=3DA0=3D0D=3D0APlease l= et me know if =3D you guys are interested in this, and please be gentle on your reviews - I= =3D =3DE1=3DB8=3DBF not a Java Programmer by any means, and probably brought to= o ma=3D ny C++ traits over here... :)=3D0D=3D0A=3D0D=3D0A=3DC2=3DA0=3D0D=3D0AThanks= =3D0D=3D0A=3D0D=3D0A=3DC2=3D =3DA0=3D0D=3D0AJustin=3D0D=3D0A=3D0D=3D0A --=3D_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS Content-Type: text/html; charset=3Dutf-8 Content-Transfer-Encoding: quoted-printable =3D0A=3D0A =3D0A =3D0A Atlassian= Cro=3D wd Support for Ovirt=3D0A =3D0A=3D0A=3D0A

Hi All,

I= 9;ve jus=3D t uploaded to Gerrit a patch that adds atlassian crowd authentication sup= =3D port to oVirt.(http://gerrit.ovirt.org/9324)

 

All the n= =3D itty gritty details are in the commit message, but basically this adds a = =3D new authentication domain, and when selected sends the authentication req= =3D uest off to a crowd server.

 

We have been using this pa= =3D tch on 3.1 for a while, and I thought its overdue to see if you guys are = =3D interested in taking this upstream=3D3F

 

There are two ar= =3D eas that I know could use improvements:

1) caching the results from= =3D the crowd server, so we don't query for every API request etc.

2) adding proper memberOf support so groups work correctly.

 = =3D

Please let me know if you guys are interested in this, and please = =3D be gentle on your reviews - I=3DE1=3DB8=3DBF not a Java Programmer by any m= eans=3D , and probably brought too many C++ traits over here... :)

 

Thanks

 

Justin

=3D0A=3D0A --=3D_fJzGa-X67Q+KhqdUoCI9FEhv0D7g0Ni-1T1OEDm6pV+tgkUS-- --===============2165405942651123678== Content-Type: multipart/alternative MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename="attachment.bin" VGhpcyBpcyBhIG11bHRpLXBhcnQgbWVzc2FnZSBpbiBNSU1FIGZvcm1hdC4gWW91ciBtYWlsIHJl YWRlciBkb2VzIG5vdAp1bmRlcnN0YW5kIE1JTUUgbWVzc2FnZSBmb3JtYXQuCi0tPV9mSnpHYS1Y NjdRK0tocWRVb0NJOUZFaHYwRDdnME5pLTFUMU9FRG02cFYrdGdrVVMKQ29udGVudC1UeXBlOiB0 ZXh0L3BsYWluOyBjaGFyc2V0PXV0Zi04CkNvbnRlbnQtVHJhbnNmZXItRW5jb2Rpbmc6IHF1b3Rl ZC1wcmludGFibGUKCkhpIEFsbCw9MEQ9MEE9MEQ9MEFJJ3ZlIGp1c3QgdXBsb2FkZWQgdG8gR2Vy cml0IGEgcGF0Y2ggdGhhdCBhZGRzIGF0bGFzc2k9CmFuIGNyb3dkIGF1dGhlbnRpY2F0aW9uIHN1 cHBvcnQgdG8gb1ZpcnQuKGh0dHA6Ly9nZXJyaXQub3ZpcnQub3JnLzkzMjQpPTBEPQo9MEE9MEQ9 MEE9QzI9QTA9MEQ9MEFBbGwgdGhlIG5pdHR5IGdyaXR0eSBkZXRhaWxzIGFyZSBpbiB0aGUgY29t bWl0IG1lc3NhPQpnZSwgYnV0IGJhc2ljYWxseSB0aGlzIGFkZHMgYSBuZXcgYXV0aGVudGljYXRp b24gZG9tYWluLCBhbmQgd2hlbiBzZWxlY3RlPQpkIHNlbmRzIHRoZSBhdXRoZW50aWNhdGlvbiBy ZXF1ZXN0IG9mZiB0byBhIGNyb3dkIHNlcnZlci49MEQ9MEE9MEQ9MEE9QzI9QTA9Cj0wRD0wQVdl IGhhdmUgYmVlbiB1c2luZyB0aGlzIHBhdGNoIG9uIDMuMSBmb3IgYSB3aGlsZSwgYW5kIEkgdGhv dWdodCBpdHM9CiBvdmVyZHVlIHRvIHNlZSBpZiB5b3UgZ3V5cyBhcmUgaW50ZXJlc3RlZCBpbiB0 YWtpbmcgdGhpcyB1cHN0cmVhbT0zRj0wRD0wQT0KPTBEPTBBPUMyPUEwPTBEPTBBVGhlcmUgYXJl IHR3byBhcmVhcyB0aGF0IEkga25vdyBjb3VsZCB1c2UgaW1wcm92ZW1lbnRzOj0KPTBEPTBBPTBE PTBBMSkgY2FjaGluZyB0aGUgcmVzdWx0cyBmcm9tIHRoZSBjcm93ZCBzZXJ2ZXIsIHNvIHdlIGRv bid0IHF1ZT0KcnkgZm9yIGV2ZXJ5IEFQSSByZXF1ZXN0IGV0Yy49MEQ9MEE9MEQ9MEEyKSBhZGRp bmcgcHJvcGVyIG1lbWJlck9mIHN1cHBvcj0KdCBzbyBncm91cHMgd29yayBjb3JyZWN0bHkuPTBE PTBBPTBEPTBBPUMyPUEwPTBEPTBBUGxlYXNlIGxldCBtZSBrbm93IGlmID0KeW91IGd1eXMgYXJl IGludGVyZXN0ZWQgaW4gdGhpcywgYW5kIHBsZWFzZSBiZSBnZW50bGUgb24geW91ciByZXZpZXdz IC0gST0KPUUxPUI4PUJGIG5vdCBhIEphdmEgUHJvZ3JhbW1lciBieSBhbnkgbWVhbnMsIGFuZCBw cm9iYWJseSBicm91Z2h0IHRvbyBtYT0KbnkgQysrIHRyYWl0cyBvdmVyIGhlcmUuLi4gOik9MEQ9 MEE9MEQ9MEE9QzI9QTA9MEQ9MEFUaGFua3M9MEQ9MEE9MEQ9MEE9QzI9Cj1BMD0wRD0wQUp1c3Rp bj0wRD0wQT0wRD0wQQotLT1fZkp6R2EtWDY3UStLaHFkVW9DSTlGRWh2MEQ3ZzBOaS0xVDFPRURt NnBWK3Rna1VTCkNvbnRlbnQtVHlwZTogdGV4dC9odG1sOyBjaGFyc2V0PXV0Zi04CkNvbnRlbnQt VHJhbnNmZXItRW5jb2Rpbmc6IHF1b3RlZC1wcmludGFibGUKCjwhRE9DVFlQRSBIVE1MIFBVQkxJ QyAiLS8vVzNDLy9EVEQgSFRNTCA0LjAxIFRyYW5zaXRpb25hbC8vRU4iICJodHRwOi8vd3c9Cncu dzMub3JnL1RSL2h0bWw0L2xvb3NlLmR0ZCI+PGh0bWw+PTBBPGhlYWQ+PTBBICA8bWV0YSBuYW1l PTNEIkdlbmVyYXRvciI9CiBjb250ZW50PTNEIlphcmFmYSBXZWJBY2Nlc3MgdjcuMC4zLTMwNTE1 Ij49MEEgIDxtZXRhIGh0dHAtZXF1aXY9M0QiQ29udGU9Cm50LVR5cGUiIGNvbnRlbnQ9M0QidGV4 dC9odG1sOyBjaGFyc2V0PTNEdXRmLTgiPj0wQSAgPHRpdGxlPkF0bGFzc2lhbiBDcm89CndkIFN1 cHBvcnQgZm9yIE92aXJ0PC90aXRsZT49MEEgIDxzdHlsZSB0eXBlPTNEInRleHQvY3NzIj49MEEg ICAgICBib2R5PTBEPQo9MEEgICAgICB7PTBEPTBBICAgICAgICBmb250LWZhbWlseTogQXJpYWws IFZlcmRhbmEsIFNhbnMtU2VyaWYgISBpbXBvcnRhPQpudDs9MEQ9MEEgICAgICAgIGZvbnQtc2l6 ZTogMTJweDs9MEQ9MEEgICAgICAgIHBhZGRpbmc6IDVweCA1cHggNXB4IDVweDs9MEQ9Cj0wQSAg ICAgICAgbWFyZ2luOiAwcHg7PTBEPTBBICAgICAgICBib3JkZXItc3R5bGU6IG5vbmU7PTBEPTBB ICAgICAgICBiYWM9Cmtncm91bmQtY29sb3I6ICNmZmZmZmY7PTBEPTBBICAgICAgfT0wRD0wQT0w RD0wQSAgICAgIHAsIHVsLCBsaT0wRD0wQSAgICA9CiAgez0wRD0wQSAgICAgICAgbWFyZ2luLXRv cDogMHB4Oz0wRD0wQSAgICAgICAgbWFyZ2luLWJvdHRvbTogMHB4Oz0wRD0wQSA9CiAgICAgfT0w RD0wQSAgPC9zdHlsZT49MEE8L2hlYWQ+PTBBPGJvZHk+PTBBPHA+SGkgQWxsLDwvcD48cD5JJiMz OTt2ZSBqdXM9CnQgdXBsb2FkZWQgdG8gR2Vycml0IGEgcGF0Y2ggdGhhdCBhZGRzIGF0bGFzc2lh biBjcm93ZCBhdXRoZW50aWNhdGlvbiBzdXA9CnBvcnQgdG8gb1ZpcnQuKGh0dHA6Ly9nZXJyaXQu b3ZpcnQub3JnLzkzMjQpPC9wPjxwPiZuYnNwOzwvcD48cD5BbGwgdGhlIG49Cml0dHkgZ3JpdHR5 IGRldGFpbHMgYXJlIGluIHRoZSBjb21taXQgbWVzc2FnZSwgYnV0IGJhc2ljYWxseSB0aGlzIGFk ZHMgYSA9Cm5ldyBhdXRoZW50aWNhdGlvbiBkb21haW4sIGFuZCB3aGVuIHNlbGVjdGVkIHNlbmRz IHRoZSBhdXRoZW50aWNhdGlvbiByZXE9CnVlc3Qgb2ZmIHRvIGEgY3Jvd2Qgc2VydmVyLjwvcD48 cD4mbmJzcDs8L3A+PHA+V2UgaGF2ZSBiZWVuIHVzaW5nIHRoaXMgcGE9CnRjaCBvbiAzLjEgZm9y IGEgd2hpbGUsIGFuZCBJIHRob3VnaHQgaXRzIG92ZXJkdWUgdG8gc2VlIGlmIHlvdSBndXlzIGFy ZSA9CmludGVyZXN0ZWQgaW4gdGFraW5nIHRoaXMgdXBzdHJlYW09M0Y8L3A+PHA+Jm5ic3A7PC9w PjxwPlRoZXJlIGFyZSB0d28gYXI9CmVhcyB0aGF0IEkga25vdyBjb3VsZCB1c2UgaW1wcm92ZW1l bnRzOjwvcD48cD4xKSBjYWNoaW5nIHRoZSByZXN1bHRzIGZyb209CiB0aGUgY3Jvd2Qgc2VydmVy LCBzbyB3ZSBkb24mIzM5O3QgcXVlcnkgZm9yIGV2ZXJ5IEFQSSByZXF1ZXN0IGV0Yy48L3A+PHA9 Cj4yKSBhZGRpbmcgcHJvcGVyIG1lbWJlck9mIHN1cHBvcnQgc28gZ3JvdXBzIHdvcmsgY29ycmVj dGx5LjwvcD48cD4mbmJzcDs9CjwvcD48cD5QbGVhc2UgbGV0IG1lIGtub3cgaWYgeW91IGd1eXMg YXJlIGludGVyZXN0ZWQgaW4gdGhpcywgYW5kIHBsZWFzZSA9CmJlIGdlbnRsZSBvbiB5b3VyIHJl dmlld3MgLSBJPUUxPUI4PUJGIG5vdCBhIEphdmEgUHJvZ3JhbW1lciBieSBhbnkgbWVhbnM9Ciwg YW5kIHByb2JhYmx5IGJyb3VnaHQgdG9vIG1hbnkgQysrIHRyYWl0cyBvdmVyIGhlcmUuLi4gOik8 L3A+PHA+Jm5ic3A7PC89CnA+PHA+VGhhbmtzPC9wPjxwPiZuYnNwOzwvcD48cD5KdXN0aW48L3A+ PTBBPC9ib2R5Pj0wQTwvaHRtbD4KLS09X2ZKekdhLVg2N1ErS2hxZFVvQ0k5RkVodjBEN2cwTmkt MVQxT0VEbTZwVit0Z2tVUy0tCgo= --===============2165405942651123678==--