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

Refactor Credential Handling #1897

Closed
ogoffart opened this issue Jun 23, 2014 · 5 comments
Closed

Refactor Credential Handling #1897

ogoffart opened this issue Jun 23, 2014 · 5 comments

Comments

@ogoffart
Copy link
Contributor

ogoffart commented Jun 23, 2014

We have had a lot of regression in the handling of credentials. Issues with keychain, logins log off, shibboleth, interaction with the wizard, ...
We also want to support in the future multiple accounts and UI-less library for the command line client.
The code has become more complicated than it should be and some refactoring is needed.

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/2782993-refactor-credential-handling?utm_campaign=plugin&utm_content=tracker%2F216457&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F216457&utm_medium=issues&utm_source=github).
@danimo danimo changed the title Refactor credidential handling Refactor Credential Handling Jun 23, 2014
@ogoffart
Copy link
Contributor Author

I think we probably should not have AbstractCrediential. Just handle the 401 or the redirection when they come.

@danimo
Copy link
Contributor

danimo commented Jun 23, 2014

How is one related to the other?

@guruz guruz added this to the 1.9 - Multi-account milestone Jan 26, 2015
@guruz
Copy link
Contributor

guruz commented Jan 26, 2015

Also related to #43

@ogoffart ogoffart modified the milestones: backlog, 2.0 - Multi-account Jun 29, 2015
@danimo danimo removed this from the backlog milestone Feb 22, 2016
@guruz
Copy link
Contributor

guruz commented Sep 21, 2017

@ogoffart @ckamm is this (partly?) done?

@ckamm
Copy link
Contributor

ckamm commented Sep 22, 2017

Some of it is done, some if it might want to go into more specific tickets @ogoffart.

@ckamm ckamm closed this as completed Sep 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants