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

talek: start adding a talek replica kaetzchen service worker #365

Draft
wants to merge 37 commits into
base: main
Choose a base branch
from

Conversation

mixmasala
Copy link
Contributor

TODO: implmement the OnCommand method and command types

mixmasala and others added 4 commits June 27, 2023 07:35
This removes the monotime package as a monotomic clock is part of golang's builtin time since golang 1.9
TODO: implmement the OnCommand method and command types
@mixmasala mixmasala marked this pull request as draft July 26, 2023 12:59
@codecov-commenter
Copy link

codecov-commenter commented Jul 26, 2023

Codecov Report

Attention: 82 lines in your changes are missing coverage. Please review.

Comparison is base (d6a031f) 47.18% compared to head (84f7714) 47.13%.

Files Patch % Lines
server/internal/provider/kaetzchen/cbor_plugins.go 0.00% 44 Missing ⚠️
server/internal/scheduler/queue_bolt.go 0.00% 17 Missing ⚠️
reunion/server/server.go 0.00% 15 Missing ⚠️
server/internal/scheduler/scheduler.go 0.00% 5 Missing ⚠️
server/internal/provider/kaetzchen/kaetzchen.go 50.00% 1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
- Coverage   47.18%   47.13%   -0.05%     
==========================================
  Files          87       85       -2     
  Lines       15117    15122       +5     
==========================================
- Hits         7133     7128       -5     
- Misses       7031     7043      +12     
+ Partials      953      951       -2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@katzenpost katzenpost deleted a comment from sonatype-lift bot Jul 26, 2023
mixmasala added 23 commits July 28, 2023 16:09
the CBOR ServerPlugin no longer returns a reply, but instead must send responses using the server.Write(Command) API.
This means that the mix server will not need to block waiting for a response from the plugin.
…nc send; refactor NewPacketFromSURB

server/cborplugin/client: add msgID to Response type
server/CBORPluginWorker: refactor NewPacketFromSURB to not depend on origin packet
This makes a pki.Document available internally
…ponse

Refactor services to use the server.Write passed by RegisterConsumer to send responses.
….Response

This changes cborplugin so that the Request and Response include the NodeDelay specififed for the Provider so that the response may be scheduled to respond within the sending client's expected ETA
this should help prevent disks filling with logfile entries
server: Add talek replica to Makefile
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.

None yet

2 participants