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.

Sign up for GitHub

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

Jump to bottom

Update dependencies #221

Open
wants to merge 45 commits into
base: develop
Choose a base branch
from
Open

Update dependencies #221

wants to merge 45 commits into from

Conversation

Filienko
Copy link
Contributor

@Filienko Filienko commented Jul 28, 2024 •
edited
Loading

Updated dependencies and Docker now uses Python 3.11

@Filienko Filienko requested a review from ivan-c July 28, 2024 03:11
ivan-c
ivan-c approved these changes Jul 29, 2024
View reviewed changes
Copy link
Member

@ivan-c ivan-c left a comment

Choose a reason for hiding this comment

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

Looks great, thanks! This is a big step towards running all the latest dependencies

I did a quick test on isacc-dev and didn't run into any issues

requirements.txt Outdated Show resolved Hide resolved
@ivan-c ivan-c requested a review from pbugni July 29, 2024 23:23
@Filienko Filienko requested a review from ivan-c August 22, 2024 23:59
Filienko added 25 commits December 4, 2024 17:36
@Filienko @ivan-c
WIP: updating dependencies
43389ca
@Filienko @ivan-c
Updating to version 3.9
5b36cdc
@Filienko @ivan-c
Adding flask_oidc
fadf536
@Filienko @ivan-c
Changing requirements
566e548
@Filienko @ivan-c
WIP: testing without libraries
20e1e1e
@Filienko @ivan-c
Returning to working libraries
e1983eb
@Filienko @ivan-c
Updating requirements
3147137
@Filienko @ivan-c
Trying to replace flask safe_join with werkzeug
95ff06b
@Filienko @ivan-c
Downgrading flask
8b7c61d
@Filienko @ivan-c
Upgrading
dd69b36
@Filienko @ivan-c
WIP: Changing safe_join
8ee3560
@Filienko @ivan-c
Updating jwt
2d5e06a
@Filienko @ivan-c
Downgrade flask
053229c
@Filienko @ivan-c
Downgrading flask
f209a24
@Filienko @ivan-c
Downgrading flask
1498ab8
@Filienko @ivan-c
Downgrading pluggy
126dc1f
@Filienko @ivan-c
Downgrading pytest
d05b697
@Filienko @ivan-c
Downgrading werkzeug
bc10861
@Filienko @ivan-c
Downgrading itsdangerous
5df077a
@Filienko @ivan-c
Downgrading jwt
43d091c
@Filienko @ivan-c
Adding six
85d341e
@Filienko @ivan-c
Cleaning up dev requirements.
ce34150
@Filienko @ivan-c
WIP: updating python to 3.11
f25d7a8
@Filienko @ivan-c
Updating python version in the readme to correspond to one in the docker
783223c
@Filienko @ivan-c
Updating JSONEncoder to import from json
b73dfdb
@ivan-c ivan-c force-pushed the update-dependencies branch from 5c42a49 to e70af92 Compare December 5, 2024 01:37
ivan-c
ivan-c requested changes Dec 6, 2024
View reviewed changes
Copy link
Member

@ivan-c ivan-c left a comment

Choose a reason for hiding this comment

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

Thanks for upgrading flask-oidc! Found a few issues

requirements.txt Outdated Show resolved Hide resolved
requirements.dev.txt Outdated Show resolved Hide resolved
requirements.txt Show resolved Hide resolved
@pbugni
Copy link
Contributor

pbugni commented Dec 10, 2024

@ivan-c we've cleared a number of hurdles, but don't yet have functioning auth since upgrading to flask-oidc 2.2.2

Looks like they replaced OpenIDConnect.validate_token with an introspection class.

Feel free to poke at this if you have cycles. Otherwise, I'll resume work here "soon".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ivan-c ivan-c ivan-c requested changes

@pbugni pbugni pbugni approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Filienko @pbugni @ivan-c