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

python3-saml vaihto kirjastoon PySAML2 #3233

Closed
dezhidki opened this issue Nov 1, 2022 · 0 comments · Fixed by #3249
Closed

python3-saml vaihto kirjastoon PySAML2 #3233

dezhidki opened this issue Nov 1, 2022 · 0 comments · Fixed by #3249
Assignees
@dezhidki
Labels
kehitystyö TIM-kehitystyö (kehitys, joka ei ole suoraan näkyvissä loppukäyttäjälle)

Comments

@dezhidki
Copy link
Member

dezhidki commented Nov 1, 2022

SAML-autentikaatioon (esim. HAKA-kirjautuminen) tällä hetkellä käytetään kirjastoa python3-saml. Valitettavasti kirjastoa ei enää ylläpidetä virallisesti, katso SAML-Toolkits/python3-saml#320. Tämä estää muiden kirjastojen päivittämistä, esim lxml (HTML:n sanitointi ja muokkaus) on nyt lukittu versioon 4.7.0. Lisäksi python3-saml on vaatinut hieman kikkailua toimiaakseen tim.educationin kanssa.

Ylläpidetty kirjasto on sen sijaan tällä hetkellä PySAML2, ja se näyttää olevan yhteisön ylläpitämä eikä yksittäisen yhtiön kirjasto. Jatkon kannalta kirjasto kannattaakin vaihtaa. Kirjastojen rajapinnat ovat hieman erilaiset, joten muutos vaatii samalla koodin refaktorointia.
@dezhidki dezhidki added the kehitystyö TIM-kehitystyö (kehitys, joka ei ole suoraan näkyvissä loppukäyttäjälle) label Nov 1, 2022
@dezhidki dezhidki self-assigned this Nov 4, 2022
@dezhidki dezhidki moved this to 🏗 In progress in @dezhidki:n hommat Nov 4, 2022
@dezhidki dezhidki mentioned this issue Nov 7, 2022
Merged
@dezhidki dezhidki moved this from 🏗 In progress to 🔖 Ready (not merged) in @dezhidki:n hommat Nov 7, 2022
@dezhidki dezhidki moved this from 🔖 Ready (not merged) to ✅ Done in @dezhidki:n hommat Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dezhidki dezhidki

Labels
kehitystyö TIM-kehitystyö (kehitys, joka ei ole suoraan näkyvissä loppukäyttäjälle)
Projects
@dezhidki:n hommat
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refactor SAML to use PySAML2 TIM-JYU/TIM
1 participant
@dezhidki