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

(Fix) Handle locked sessions #180

Merged
merged 2 commits into from
Jun 7, 2018
Merged

(Fix) Handle locked sessions #180

merged 2 commits into from
Jun 7, 2018

Conversation

fvictorio
Copy link
Contributor

@fvictorio fvictorio commented Jun 6, 2018 •
edited
Loading

Closes #170.

This PR does two things:

  • It unlocks the key if something fails in notify_reg_tx
  • It unlocks all keys (in redis) on startup. Keys in memory are lost, of course, so there's nothing we can do there.

@fvictorio fvictorio requested review from garatortiz and phahulin June 6, 2018 15:08
@ghost ghost assigned fvictorio Jun 6, 2018
@ghost ghost added the in progress label Jun 6, 2018
@phahulin phahulin merged commit f988c8a into master Jun 7, 2018
@phahulin phahulin deleted the handle-locked-sessions-#170 branch June 7, 2018 10:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@garatortiz garatortiz garatortiz approved these changes

@phahulin phahulin phahulin approved these changes

Assignees

@fvictorio fvictorio

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@fvictorio @garatortiz @phahulin