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

Implement OpenID Connect #1180

Closed
Fale opened this issue Dec 20, 2016 · 12 comments · Fixed by #4387
Closed

Implement OpenID Connect #1180

Fale opened this issue Dec 20, 2016 · 12 comments · Fixed by #4387
Labels
Backwards incompatible The proposed change is backwards incompatible and should wait for the next major release Client Issues with the bodhi command line interface tool High priority These issues are higher priority than normal RFE Requests for Enhancement

Comments

@Fale
Copy link

Fale commented Dec 20, 2016

It would be nice if the CLI client for bodhi would use OpenID Connect instead of asking the password, as for #1179

@bowlofeggs bowlofeggs added Client Issues with the bodhi command line interface tool High priority These issues are higher priority than normal RFE Requests for Enhancement labels Dec 21, 2016
@bowlofeggs
Copy link
Contributor

I'd like to get this done in time for Fedora 26. @puiterwijk, does fedora.client have what I need to get this done already?

@bowlofeggs
Copy link
Contributor

This PR is a proposal to add OpenID Connect to fedora.client:

fedora-infra/python-fedora#188

Once fedora.client has this and is released to EPEL 7 and Fedora 26, we can add support to Bodhi's client.

@cverna cverna self-assigned this Apr 13, 2017
@cverna cverna removed their assignment Jun 21, 2017
@mcepl
Copy link

mcepl commented Feb 25, 2018

ping? having password in ~/.bash_history (my current state) is stupid.

@bowlofeggs
Copy link
Contributor

@mcepl This feature is not needed to solve that problem - simply don't use the Bodhi CLI's --password flag - it will automatically prompt you for a password if needed.

@mcepl
Copy link

mcepl commented Feb 25, 2018

Well, then either the password is so simple I can understand it, or secure and then I have to dig it up everytime from some secure store. I would prefer Kerberos (when we have it by the way of id.fedoraproject.org).

@bowlofeggs
Copy link
Contributor

bowlofeggs commented Feb 25, 2018 via email

@mcepl
Copy link

mcepl commented Feb 26, 2018

There are not plans to integrate Bodhi with kerberos, but there are plans to integrate it with OpenID.

Exactly, which was the question I had from beginning: how are these plans doing? Any ETA?

@bowlofeggs
Copy link
Contributor

@mcepl Unfortunately it's not currently very close to the top of my personal priority list so I can't give an ETA on it. It's not low either, it's just that there are some big features that are planned for Fedora 28 and I'm swamped ☺ Patches are certainly welcome if you or anybody else has the time and inclination to get it working, though it's probably not a simple project and might take some significant time.

One tricky question to figure out will be whether we want to try to make it so the server can do OpenID and OpenID Connect at the same time, or if we are going to switch to only do OpenID Connect. If the latter, that will require us to do something to handle EPEL 6/7 and Fedora < 28 bodhi clients, which might require splitting the Bodhi package into a separate client and server (because Bodhi 3 [included in F28+] is not backwards compatible with Bodhi 2). EPEL 6 will be extra tricky because it is Python 2.6 and Bodhi is about to support Python 3.

@DemiMarie
Copy link

@bowlofeggs right now this is a blocker for unattended use of Bodhi

@puiterwijk
Copy link
Contributor

@DemiMarie Unfortunately, @bowlofeggs no longer works on Bodhi. This is now the responsibility of @fedora-infra/bodhi

@bowlofeggs
Copy link
Contributor

Indeed, I now work on other things. You could potentially ask about this in #fedora-apps on Freenode if you don't get an answer here.

@cverna
Copy link
Contributor

cverna commented Dec 7, 2020

I am not really sure when this is going to be implemented, but I don't think soon :(.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backwards incompatible The proposed change is backwards incompatible and should wait for the next major release Client Issues with the bodhi command line interface tool High priority These issues are higher priority than normal RFE Requests for Enhancement
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants