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

fix: typos, uniform ctype in userinfo, jwt ES stored in TextField #291

Merged
merged 6 commits into from
Dec 14, 2023

Conversation

rglauco
Copy link
Collaborator

@rglauco rglauco commented Dec 14, 2023

fixed typo in provider settings
uniformed content-type userinfo response for SPID and CIE
changed fetched_entity_statement jwt to TextField
chore: stick to django v4

@rglauco rglauco merged commit 83f020f into italia:main Dec 14, 2023
2 of 4 checks passed
@sallner
Copy link

sallner commented Dec 18, 2023

Sorry for the late questions @peppelinux, but why are you sticking to Django 5 at the moment? Is there a specific reason for that? If you do not support Django 5 it is possible to use the trove classifier for 4 only, but this limits the installation of my application. I used it with django 5 currently and it did not seem to break. Do I miss something?

@peppelinux
Copy link
Member

=4.0,<5.0

means that you can use all the version equal or major of 4 and inferior than 5

@sallner
Copy link

sallner commented Dec 20, 2023

Thanks for that information. I know what it means, but not what the reason was for this restriction. Let's discuss it further down in #298 .

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.

None yet

3 participants