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

[server-state] Make Eliom_state.Ext.discard_state more robust #745

Merged
merged 4 commits into from
Mar 10, 2022

Conversation

ilankri
Copy link
Member

@ilankri ilankri commented Mar 9, 2022

No description provided.

@ilankri ilankri self-assigned this Mar 9, 2022
@ilankri ilankri requested a review from balat March 9, 2022 10:45
@ilankri ilankri marked this pull request as draft March 9, 2022 14:29
@ilankri ilankri force-pushed the server-state-discard-state branch from d3f67d3 to 082ad60 Compare March 9, 2022 14:58
@ilankri ilankri changed the title [server-state] Adjust the interface of Eliom_state.Ext.discard_state [server-state] Make Eliom_state.Ext.discard_state more robust Mar 9, 2022
@ilankri ilankri marked this pull request as ready for review March 9, 2022 15:00
@ilankri ilankri marked this pull request as draft March 9, 2022 16:08
So that it is possible to:

    - call successively `Eliom_state.Ext.discard_state` on several
      servers sharing the same storage for persistent cookies

    - discard different kind of states in any order
So that it is possible to call `Eliom_state.Ext.discard_state` outside
a request or the initialization phase of the Eliom module.
@ilankri ilankri force-pushed the server-state-discard-state branch from 082ad60 to ef9b16e Compare March 9, 2022 17:15
@ilankri ilankri marked this pull request as ready for review March 9, 2022 17:18
@ilankri ilankri marked this pull request as draft March 10, 2022 16:22
@ilankri ilankri marked this pull request as ready for review March 10, 2022 16:27
@balat balat merged commit 81e24ad into ocsigen:master Mar 10, 2022
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

2 participants