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

Feature/payload sync revisited #24

Merged
merged 7 commits into from
May 29, 2022
Merged

Conversation

toxyl
Copy link
Owner

@toxyl toxyl commented May 29, 2022

  • replaced fingerprints with payloads
  • payloads are now synced using the default mechanism
  • updated some errors
  • UI: payloads list can now be navigated with up/down arrow keys
  • UI: payloads list is now sorted by creation date
  • fixed overlayfs path leak (resolves OverlayFS path leak #20)
  • fixed stuck mounts (resolves OverlayFS stuck mounts #22)
  • implemented session cleanup goroutine

Tox added 7 commits May 18, 2022 21:22
…ing on the opposite mechanism

- added a couple session removals in an attempt to clean up (seems to have partially worked)
- denigrated sync client connection errors to warnings if the "connect: connection refused" error is encountered (happens frequently if sync nodes are busy)
- fixed overlayfs path leak
- UI: changed payload list to be sorted by file creation date (also displayed)
Copy link
Collaborator

@dylandreimerink dylandreimerink left a comment

Choose a reason for hiding this comment

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

LGTM

@toxyl toxyl merged commit 171a450 into master May 29, 2022
@toxyl toxyl deleted the feature/payload-sync-revisited branch May 29, 2022 08:30
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.

OverlayFS stuck mounts OverlayFS path leak
2 participants