[sonar-dev] [VOTE] Sonar OpenID Plugin 1.0

[Simon Brandhof]

Hi,

I'd like to release the first version of the OpenID plugin. It enables user authentication and Single Sign-On via an OpenID provider.

Sonar 3.1 is required.

Download and copy into /extensions/plugins :
http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-openid-plugin/0.1-SNAPSHOT/sonar-openid-plugin-0.1-20120615.073903-1.jar

Documentation :
http://docs.codehaus.org/display/SONAR/OpenID+Plugin

Vote is open to everybody for 72 hours.
Thanks

[Evgeny Mandrikov]

Hi Simon,

I was unable to build sonar-openid, because openid4java depends on com.google.code.guice:guice:2.0 , which I didn't found in central repository.

Despite this fact plugin works well.

BTW maybe key for realm should be "OpenID" instad of "openid" ?

On Fri, Jun 15, 2012 at 1:46 PM, Simon Brandhof <[hidden email]> wrote:

Hi,

I'd like to release the first version of the OpenID plugin. It enables user authentication and Single Sign-On via an OpenID provider.

Sonar 3.1 is required.

Download and copy into /extensions/plugins :
http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-openid-plugin/0.1-SNAPSHOT/sonar-openid-plugin-0.1-20120615.073903-1.jar

Documentation :
http://docs.codehaus.org/display/SONAR/OpenID+Plugin

Vote is open to everybody for 72 hours.
Thanks

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Freddy Mallet]

+1, tested with Google account !

-----

twitter.com/FreddyMallet

Sonar for Continuous Inspection

On Mon, Jun 18, 2012 at 9:20 PM, Evgeny Mandrikov <[hidden email]> wrote:

Hi Simon,

I was unable to build sonar-openid, because openid4java depends on com.google.code.guice:guice:2.0 , which I didn't found in central repository.

Despite this fact plugin works well.

BTW maybe key for realm should be "OpenID" instad of "openid" ?

On Fri, Jun 15, 2012 at 1:46 PM, Simon Brandhof <[hidden email]> wrote:

Hi,

I'd like to release the first version of the OpenID plugin. It enables user authentication and Single Sign-On via an OpenID provider.

Sonar 3.1 is required.

Download and copy into /extensions/plugins :
http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-openid-plugin/0.1-SNAPSHOT/sonar-openid-plugin-0.1-20120615.073903-1.jar

Documentation :
http://docs.codehaus.org/display/SONAR/OpenID+Plugin

Vote is open to everybody for 72 hours.
Thanks

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Simon Brandhof]

I was unable to build sonar-openid, because openid4java depends on com.google.code.guice:guice:2.0 , which I didn't found in central repository.

Despite this fact plugin works well.

It's correctly downloaded in cloudbees but I don't understand where the google repository is declared :

Downloading: http://repo1.maven.org/maven2/com/google/code/guice/guice/2.0/guice-2.0.jar
Downloading: http://guice-maven.googlecode.com/svn/trunk/com/google/code/guice/guice/2.0/guice-2.0.jar
Downloaded: http://guice-maven.googlecode.com/svn/trunk/com/google/code/guice/guice/2.0/guice-2.0.jar (668 KB at 400.7 KB/sec)

 

BTW maybe key for realm should be "OpenID" instad of "openid" ?

The only constraint documented in API is the uniqueness. So what's the convention ? LDAP realm is upcase ("LDAP").

[Evgeny Mandrikov]

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

On Wed, Jun 20, 2012 at 12:50 PM, Simon Brandhof <[hidden email]> wrote:

I was unable to build sonar-openid, because openid4java depends on com.google.code.guice:guice:2.0 , which I didn't found in central repository.

Despite this fact plugin works well.

It's correctly downloaded in cloudbees but I don't understand where the google repository is declared :

Downloading: http://repo1.maven.org/maven2/com/google/code/guice/guice/2.0/guice-2.0.jar
Downloading: http://guice-maven.googlecode.com/svn/trunk/com/google/code/guice/guice/2.0/guice-2.0.jar
Downloaded: http://guice-maven.googlecode.com/svn/trunk/com/google/code/guice/guice/2.0/guice-2.0.jar (668 KB at 400.7 KB/sec)

 

BTW maybe key for realm should be "OpenID" instad of "openid" ?

The only constraint documented in API is the uniqueness. So what's the convention ? LDAP realm is upcase ("LDAP").

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Simon Brandhof]

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

[Evgeny Mandrikov]

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Evgeny Mandrikov]

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Fabrice Bellingard-4]

+1 Simon, works very well!

Best regards,

Fabrice BELLINGARD | SonarSource
http://sonarsource.com

On Fri, Jun 15, 2012 at 9:46 AM, Simon Brandhof <[hidden email]> wrote:

Hi,

I'd like to release the first version of the OpenID plugin. It enables user authentication and Single Sign-On via an OpenID provider.

Sonar 3.1 is required.

Download and copy into /extensions/plugins :
http://snapshots.repository.codehaus.org/org/codehaus/sonar-plugins/sonar-openid-plugin/0.1-SNAPSHOT/sonar-openid-plugin-0.1-20120615.073903-1.jar

Documentation :
http://docs.codehaus.org/display/SONAR/OpenID+Plugin

Vote is open to everybody for 72 hours.
Thanks

[Dinesh Bolkensteyn-2]

+1 Well done this plugin is really amazing!

--
Dinesh Bolkensteyn
www.SonarSource.com
twitter.com/DBolkensteyn

On Wed, Jun 20, 2012 at 10:48 AM, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[David Racodon-2]

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Simon Brandhof]

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Patroklos Papapetrou]

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Simon Brandhof]

That's the convenience to work in the same office !

On 29 June 2012 12:03, Papapetrou P.Patroklos <[hidden email]> wrote:

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Patroklos Papapetrou]

... and being the techincal lead :)

2012/6/29 Simon Brandhof <[hidden email]>

That's the convenience to work in the same office !

On 29 June 2012 12:03, Papapetrou P.Patroklos <[hidden email]> wrote:

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Evgeny Mandrikov]

You wanted to make a most voted release ? :)

P.S. I guess that I will not receive payment, since I'm not in same office and really did a test. Pity :(

On Fri, Jun 29, 2012 at 4:07 PM, Papapetrou P.Patroklos <[hidden email]> wrote:

... and being the techincal lead :)

2012/6/29 Simon Brandhof <[hidden email]>

That's the convenience to work in the same office !

On 29 June 2012 12:03, Papapetrou P.Patroklos <[hidden email]> wrote:

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[ehartmann]

... and offering some beers ? ;-)

Eric

On 29 June 2012 12:07, Papapetrou P.Patroklos <[hidden email]> wrote:

... and being the techincal lead :)

2012/6/29 Simon Brandhof <[hidden email]>

That's the convenience to work in the same office !

On 29 June 2012 12:03, Papapetrou P.Patroklos <[hidden email]> wrote:

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

[Ann Campbell]

Make him FedEx your beer, Evgeny!

On Fri, Jun 29, 2012 at 6:12 AM, Evgeny Mandrikov <[hidden email]> wrote:

You wanted to make a most voted release ? :)

P.S. I guess that I will not receive payment, since I'm not in same office and really did a test. Pity :(

On Fri, Jun 29, 2012 at 4:07 PM, Papapetrou P.Patroklos <[hidden email]> wrote:

... and being the techincal lead :)

2012/6/29 Simon Brandhof <[hidden email]>

That's the convenience to work in the same office !

On 29 June 2012 12:03, Papapetrou P.Patroklos <[hidden email]> wrote:

Woaoh!!

What a coordinated voting!!!! Tell us the secret Simon for getting mass voting!!! ;-)

Partoklos

2012/6/29 Simon Brandhof <[hidden email]>

Thanks Sonar guys for this grouped vote. As discussed I will pay you later :D
I continue with the release.

Note that there are some limitations that will be fixed in next release, including :

• support of SSL endpoints
• support of logout...
  

On 29 June 2012 11:55, David Racodon <[hidden email]> wrote:

Hello Simon,

You get my +1. Great work!

David RACODON | SonarSource
Senior Consultant

http://sonarsource.com

On 20 June 2012 10:48, Evgeny Mandrikov <[hidden email]> wrote:

And BTW : I confirm that problem with dependencies was fixed and  now I can build project, so in any case here is my +1.

On Wed, Jun 20, 2012 at 2:41 PM, Evgeny Mandrikov <[hidden email]> wrote:

That's was exactly my point :  to prevent from end-user typos - I know OpenID as "OpenID", but not as "openid" or "OPENID" ;)

On Wed, Jun 20, 2012 at 2:32 PM, Simon Brandhof <[hidden email]> wrote:

Repository declared in org.openid4java:openid4java-nodeps:0.9.6. Suggestion is following : exclude guece artifact from openid4java-nodeps and explicitely declare correct one, which can be downloaded from Maven Central.

Thanks, it's fixed.
https://github.com/SonarCommunity/sonar-openid/commit/99e3c321830c1b6a53277a3b6cf0611fe9675469
 

About key - my opinion is that it should be names as in the world : LDAP - all uppercase ( http://en.wikipedia.org/wiki/LDAP ), OpenID - first uppercase and two last ( http://en.wikipedia.org/wiki/OpenID ).

This is the value in config file, so to prevent from end-user typos, I'd prefer using up or down case only. "OPENID" is ok for me, like for java constants.
My 2 cents

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--
Best regards,
Evgeny Mandrikov aka Godin <http://godin.net.ru>
http://twitter.com/_godin_

--

G. Ann Campbell

Sr. Systems Engineer, IS Production Systems - Shop Floor Systems

Shaw Industries Inc,

201 S. Hamilton St.

Dalton Ga 30720

**********************************************************

Privileged and/or confidential information may be contained in this message. If you are not the addressee indicated in this message (or are not responsible for delivery of this message to that person) , you may not copy or deliver this message to anyone. In such case, you should destroy this message and notify the sender by reply e-mail.

If you or your employer do not consent to Internet e-mail for messages of this kind, please advise the sender.

Shaw Industries does not provide or endorse any opinions, conclusions or other information in this message that do not relate to the official business of the company  or its subsidiaries.

**********************************************************