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

[sessions] Adjust server-side interface of Os_session.disconnect_all #652

Merged
merged 3 commits into from
Mar 25, 2022

Conversation

ilankri
Copy link
Member

@ilankri ilankri commented Mar 9, 2022

So that it is possible to call Os_session.disconnect_all outside
a request or the initialization phase of the Eliom module (see
https://ocsigen.org/eliom/dev/api/server/Eliom_state.Ext#VALfold_volatile_sub_states).

Depends on ocsigen/eliom#745, waiting for the opam release of Eliom 10.0.0.

@ilankri ilankri self-assigned this Mar 9, 2022
@ilankri ilankri marked this pull request as ready for review March 21, 2022 08:26
@ilankri ilankri force-pushed the sessions-disconnect-all branch from d6dee05 to 1f6fbbb Compare March 21, 2022 08:27
@ilankri ilankri marked this pull request as draft March 21, 2022 08:47
ilankri added 2 commits March 21, 2022 09:53
So that it is possible to call `Os_session.disconnect_all` outside
a request or the initialization phase of the Eliom module (see
https://ocsigen.org/eliom/dev/api/server/Eliom_state.Ext#VALfold_volatile_sub_states).
@ilankri ilankri force-pushed the sessions-disconnect-all branch from 1f6fbbb to 8fd8b1c Compare March 21, 2022 08:54
@ilankri ilankri marked this pull request as ready for review March 21, 2022 09:26
@ilankri ilankri requested a review from balat March 21, 2022 09:26
@jrochel jrochel merged commit 676ea3c into ocsigen:master Mar 25, 2022
@ilankri ilankri deleted the sessions-disconnect-all branch April 11, 2022 10:17
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