Re: State of Flask-OIDC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Merged the changes in the PyJWT PR into the fedora-infra fork, tagged a release 1.5.0, and sent in a PR against the RPM to now point at this fork + new version: https://src.fedoraproject.org/rpms/python-flask-oidc/pull-request/2



On Mon, 19 Dec 2022 at 11:19, David Kirwan <dkirwan@xxxxxxxxxx> wrote:


On Thu, 15 Dec 2022 at 08:30, Frantisek Zatloukal <fzatlouk@xxxxxxxxxx> wrote:


On Wed, Dec 7, 2022 at 3:25 AM David Kirwan <dkirwan@xxxxxxxxxx> wrote:


Thanks for the update, David!

Would it be possible to cut a new release with the pyJWT PR included at least once you have time/mood to re-focus? Or would that be wasted effort before porting to authlib?

I think it might be wasted effort, as this functionality is replaced/handled in authlib iirc, but then again, I don't think this authlib implementation is ready either so if we do cut a release of the pyJWT PR at least we'll have a working version flask-oidc!

If upstream is too busy to review, should we use our fork: https://github.com/fedora-infra/flask-oidc ? Maybe get the pyJWT code PR merged there, and then point our RPM at this fork? Will take urgency off then, I can take more time then to focus on the authlib implementation?

cheers,
David

 

Thanks a lot!
 


--
David Kirwan
Senior Software Engineer

Community Platform Engineering @ Red Hat

T: +(353) 86-8624108

_______________________________________________
infrastructure mailing list -- infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to infrastructure-leave@xxxxxxxxxxxxxxxxxxxxxxx
Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: https://lists.fedoraproject.org/archives/list/infrastructure@xxxxxxxxxxxxxxxxxxxxxxx
Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue

[Index of Archives]     [Fedora Development]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux