Skip to content
This repository was archived by the owner on Jun 23, 2023. It is now read-only.

Conversation

rohe
Copy link
Collaborator

@rohe rohe commented Apr 28, 2021

We're getting rid of the oidcendpoint package having it included in oidc-op.

rohe and others added 24 commits April 12, 2021 08:40
With new sesssion management and persistent storage handling.
Use create_session from the Authorization endpoint.
The package now produces information to be placed in cookies.
This is information like the name, max-age and the value.
Whatever more needs to be put into a cookie is expected to be done by the frontend.
The package then also expects to receive cookie information in the same format as it exported.
- Wrong parameter order
- not instead of None
- need to pick up cookie name somewhere
- type in cookie not used anymore
- adapted add_cookie in views.py
Replaced oidcendpoint with oidcop where still remaining.
Added ID Token to issued_token.
* feat: added run.sh in flask_op example

some dummy code linting
@rohe rohe requested a review from peppelinux April 28, 2021 13:28
Copy link
Member

@peppelinux peppelinux left a comment

Choose a reason for hiding this comment

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

ride free Roland, ride free ...

@rohe
Copy link
Collaborator Author

rohe commented Apr 29, 2021

Could you fix the django_op/README.md conflict ?

@peppelinux peppelinux closed this Apr 29, 2021
@rohe
Copy link
Collaborator Author

rohe commented Apr 29, 2021

Why did you close it ?? Should it not have been merged ?

@peppelinux
Copy link
Member

done here:
#26

@rohe
Copy link
Collaborator Author

rohe commented Apr 29, 2021

No, #26 was into develop not into master.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants