Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Moves all login logic from the activator to the openToxLoginOutListener #5

Merged
merged 2 commits into from Feb 7, 2013

Conversation

KlasJoensson
Copy link
Contributor

It also:

  • Makes OpenToxs authorization service an account property.
  • Adds a new login method to openToxs manager: login(String user, String pass, String authService)
  • Adds a new method for geting the authorization service from the opentox manager
  • Adds a new metod the reset the authorization service
  • Adds javaDoc to the openToxLoginOutListener-class
  • Adds a logger to the openToxLoginOutListener-class
  • Makes the openToxLoginOutListener-class a singleton

And will probably solve bug 3069 as well...

…Listener-class.

It also:
* Makes OpenToxs authorization service an account property.
* Adds a new login method to openToxs manager: login(String user, String pass, String authService)
* Adds a new method for geting the authorization service from the opentox manager
* Adds a new metod the reset the authorization service
* Adds javaDoc to the openToxLoginOutListener-class
* Adds a logger to the openToxLoginOutListener-class
* Makes the openToxLoginOutListener-class a singleton
@KlasJoensson
Copy link
Contributor Author

Forget my hope about solven bug 3069... =(

<property
defaultValue="http://opensso.in-silico.ch/opensso/identity"
name="auth. service"
required="false"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't it be required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mm, thinking about it it should probably be that...

@ghost ghost assigned valyo Feb 5, 2013
goglepox added a commit that referenced this pull request Feb 7, 2013
Moves all login logic from the activator to the openToxLoginOutListener
@goglepox goglepox merged commit 2ae5cd2 into bioclipse:develop Feb 7, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants