Skip to content

Conversation

@paultag
Copy link

@paultag paultag commented Aug 7, 2017

This is far from complete, but I'm starting to sketch out what Session
management could look like from login1.

This is far from complete, but I'm starting to sketch out what Session
management could look like from login1.
@ghost
Copy link

ghost commented Aug 7, 2017

Can one of the admins verify this patch?

@paultag
Copy link
Author

paultag commented Aug 17, 2017

I'm not sure what action, if any, I need to take

@lucab
Copy link
Contributor

lucab commented Aug 17, 2017

My bad, I understood this was a WIP PR at the moment. Anything else you want to add here? If not, I'll go through this next.

@lucab
Copy link
Contributor

lucab commented Nov 30, 2017

@paultag ping. What's the status of this? Are you waiting for a review from me or do you have some more bits pending to be queued here?

@paultag
Copy link
Author

paultag commented Nov 30, 2017

@lucab ach, sorry! I've been hit by a metaphorical bus :)

I'm happy to make changes, but I'm not likely to add more stuff or get around to the tests if need be unless proded, so maybe it's review-able now, and if it's not a fit, we can close this out and move on :)

👍

@siavashs
Copy link
Contributor

I'm interested in this feature, but would like to also add KillSession() and TerminateSession(), ideally cover the whole logind API.

If @paultag agrees I will take ownership of the PR or open a new one.

@paultag
Copy link
Author

paultag commented Dec 15, 2017 via email

@paultag
Copy link
Author

paultag commented Jan 6, 2018

Closing in favor of #242

@paultag paultag closed this Jan 6, 2018
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.

3 participants