Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • briar briar
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 788
    • Issues 788
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 8
    • Merge requests 8
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • briarbriar
  • briarbriar
  • Merge requests
  • !1635

Defer marking messages and acks as sent

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged akwizgran requested to merge 2226-defer-marking-messages-and-acks-as-sent into master Apr 19, 2022
  • Overview 15
  • Commits 4
  • Pipelines 6
  • Changes 24
  • Add some javadocs to DB classes (rescued from !1571 (closed))
  • Refactor SimplexOutgoingSession to do all its work on a single IO thread (rescued from !1571 (closed))
  • Separate EagerSimplexOutgoingSession (used for removable drives) from SimplexOutgoingSession
  • Add MailboxOutgoingSession
  • Refactor the DB methods used by the various session types

MailboxOutgoingSession doesn't mark messages/acks as sent. Instead it records the message IDs via a DeferredSendHandler so they can be marked as sent later, when the file has been uploaded to the mailbox.

MailboxOutgoingSession monitors the amount of data it's sending and stops when it reaches a predetermined limit. This allows mailboxes to enforce a maximum file size for uploads (briar-mailbox#105 (closed)), which in turn helps to prevent DoS attacks by misbehaving contacts.

Closes #2296 (closed) (not #2226 (closed) as the branch name suggests)

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: 2226-defer-marking-messages-and-acks-as-sent