Skip to content

Conversation

@skavanagh
Copy link
Contributor

Added operations to support multiple authorizations (removeAuthorization, getAuthorization, getAuthorizations).

Added operations to support multiple authorizations
(removeAuthorization, getAuthorization, getAuthorizations).
@adietish
Copy link
Member

sorry for the delay I have here, too. But once again in the cycle of releasing JBoss Tools CR2 I have to fight critical bugs that keep me all busy. Should be back on track pretty soon though.

@skavanagh
Copy link
Contributor Author

Theres no need to hurry with this at all.

Thanks Andre!

On Tue, Sep 30, 2014 at 10:44 AM, André Dietisheim <notifications@github.com

wrote:

sorry for the delay I have here, too. But once again in the cycle of
releasing JBoss Tools CR2 I have to fight critical bugs that keep me all
busy. Should be back on track pretty soon though.


Reply to this email directly or view it on GitHub
#153 (comment)
.

@adietish
Copy link
Member

adietish commented Oct 6, 2014

rebased and created a new PR: #164

@adietish adietish closed this Oct 6, 2014
@adietish
Copy link
Member

adietish commented Oct 6, 2014

If I now look at the methods in OpenShiftConnectionFactory I see us stacking the different authorization methods in additional parameters. We should separate the 3 authentication forms we have, ideally via a connection builder: https://issues.jboss.org/browse/OSJC-128

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.

2 participants