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

migration to new OIDCOP ver 2.x #70

Merged
merged 2 commits into from
Jul 1, 2023

Conversation

MdreW
Copy link
Collaborator

@MdreW MdreW commented Jun 27, 2023

  • updated proxy.conf for new frontend
  • updated frontend configurazion
  • update run.sh for docker configuration

@MdreW MdreW requested a review from peppelinux June 27, 2023 09:38
@MdreW
Copy link
Collaborator Author

MdreW commented Jun 27, 2023

Hi @scolagreco , I have updated run.sh, you can check the docker configuration work fine?

@peppelinux
Copy link
Member

@MdreW I think that's time to drop my forks here
https://github.com/italia/Satosa-Saml2Spid/blob/master/requirements.txt

it seems to me that both satosa and pysaml2 can now be used as they are release in the mainline. this requires tests anyway but spid-sp-test would help in CI for that

I'd also suggest to put the versions of the packages if you agree.

hint: don't use == but always >= and/or <= like >=2.1.0,<=2.2.0

@MdreW
Copy link
Collaborator Author

MdreW commented Jun 28, 2023

Ok, @peppelinux , I try to remove your fork and we try to make a single new version 👍

@MdreW
Copy link
Collaborator Author

MdreW commented Jun 30, 2023

Hi @peppelinux , I have moved Satosa requirement to official package and I have removed pysaml requirements because si part of Satosa requirements.

  • Minimum satosa version is set to 8.2.0
  • Minimum oidcop version is set to 2.0.0

@MdreW
Copy link
Collaborator Author

MdreW commented Jun 30, 2023

@peppelinux, what do you think? Is time for a new release?

@peppelinux peppelinux merged commit 39f2c8a into italia:master Jul 1, 2023
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.

2 participants