Commit Graph

8189 Commits (7568b718931a64b85461f8b89b3cb05b5c62254c)
 

Author SHA1 Message Date
Thomas Winget 04e9fd559c build fixes, clang-format, minor touch-ups 5 months ago
Thomas Winget 657214992b chahca nonce size is 24 bytes
Lots of code was using 32-byte nonces for xchacha20 symmetric
encryption, but this just means 8 extra bytes per packet wasted as
chacha is only using the first 24 bytes of that nonce anyway.

Changing this resulted in a lot of dead/dying code breaking, so this
commit also removes a lot of that (and comments a couple places with
TODO instead)

Also nounce -> nonce where it came up.
5 months ago
Thomas Winget 2bea5e7823 path control message response status changes
change path control message inner message response to take just a
string, which will be a bt-encoded response with an early key for
status.  If there is a timeout we pass a bt dict that only has that as
the status, else the response we de-onioned should have either an OK
status or some other error.

change messages to use new status key

correctly call Path::EnterState on path build response
5 months ago
Thomas Winget 62a88b3cea omit breaking RC stuff pending refactor
It seems RC refactor will obviate the need for a "get individual RC"
method, so this comments out some usage of that to sidestep build
errors, rather than correcting them in a way that will just be wasted.
5 months ago
Thomas Winget bf2665bbe7 path control messages and onioning fleshed out
- control messages can be sent along a path
- the path owner onion-encrypts the "inner" message for each hop in the
  path
- relays on the path will onion the payload in both directions, such
  that the terminal relay will get the plaintext "inner" message and the
  client will get the plaintext "response" to that.
- control messages have (mostly, see below) been changed to be invokable
  either over a path or directly to a relay, as appropriate.

TODO:
  - exit messages need looked at, so they have not yet been changed for
    this
  - path transfer messages (traffic from client to client over 2 paths
    with a shared "pivot") are not yet implemented
5 months ago
Thomas Winget 3bc8c30b22 path build message handling mostly finished
there are a few TODOs which merit further discussion
5 months ago
dr7ana d35073cc58 libquic bump 5 months ago
dr7ana 472fcc7d1a review fixes 5 months ago
dr7ana 5c6c6bcfee config addr change
- the one addr to rule them all, and its name was oxen::quic::Address
- no more vectors of inbound/outbound junk
5 months ago
dr7ana c149651d64 A soothing re-nomenclatura 5 months ago
dr7ana 47bedfd8da Review fixes + misc fixes 5 months ago
Jason Rhinelander 0de50b176b Untangle Endpoint::LookupServiceAsync
- .snodes don't need to support SRV records, so remove that
- untangle the mess of captured lambdas capturing other lambdas
  capturing other lambdas; we still need a chain of nested lambdas
  because we have a chain of callbacked events, but hiding the nesting
  by capturing them in other lambdas didn't improve anything.
5 months ago
dr7ana 68e67c881b {Remote,Local}RC's
- RemoteRC supplants most of the functionality throughout the code of RouterContact
- Next step will be to sort out CI issues, then see if we can get rid of either LocalRC (and therefore RouterContact entirely)
5 months ago
dr7ana 632f4ee70b RC refactor layout
- Local and Remote RC's now implemented with discrete functionalities and uses
5 months ago
Jason Rhinelander 431490fb8b Give up for now 5 months ago
Jason Rhinelander ca5ed01b2c Bump libquic to latest for static build fixes 5 months ago
Jason Rhinelander c3641ab22c Turn off terrible new cmake 3.24+ default 5 months ago
Jason Rhinelander 46482c03a6 Remove unused file 5 months ago
Jason Rhinelander b3c3dcf33a Bump armhf static build to bullseye
Use backports for cmake because ngtcp2 needs 3.20+.
5 months ago
Jason Rhinelander 63b8fd49cc Don't export CC
This was causing cmake to double-invoke ccache for compilation units,
and broke oxen-mq's local zmq build for the same reason.
5 months ago
dr7ana cb2a2cf6e3 Weird clang unused variable CI fix 5 months ago
Jason Rhinelander eb37796b65 Bump oxen-mq for libzmq llvm static build fix 5 months ago
dr7ana 98583b8f58 Windows CI fixes
- some weird function call business
- string formatting
- etc
5 months ago
dr7ana c46c18ea30 Bump libzmq static; remove patches 5 months ago
Jason Rhinelander 57c471e7a2 Bump WinDivert to latest
There are some bugfix releases, but also the upstream hash was wrong for
the current release anyway (upstream changed it?).
5 months ago
Jason Rhinelander b909f537fe Default libcrypt to whether or not we find it
Also deliberately don't provide the function at all if we aren't
compiled with libcrypt so that we can't link if we try to call it when
not available.
5 months ago
Jason Rhinelander 6d6301daee Disable Router Hive & DEB builds
They aren't useful right now.
5 months ago
dr7ana acf494098a squash 5 months ago
dr7ana 127ac2c9aa windows fixes 5 months ago
dr7ana 5596ffd9d7 CI Fixes, squash 5 months ago
Jason Rhinelander 19f9a8f122 Temporarily disable building/running test suite 5 months ago
dr7ana 57393ea740 Review fixes
- cleaned up include-what-you-use errors (mostly quoted includes)
- misc fixes
5 months ago
Jason Rhinelander 3d44e58e34 Fix systemd linkage 5 months ago
Jason Rhinelander 52b175772f Bring oxen-encoding back in as a submodule
As long as it's loaded before oxen-mq or libquic, this should end up
being the one that gets used in oxen-mq/libquic, and so we don't have to
update those when we want something in a newer oxen-encoding.
5 months ago
Jason Rhinelander f21facfd39 Add format.sh detection of include problems 5 months ago
dr7ana f574cd798f Clang format include sorting + CMake
- includes are now sorted in consistent, logical order; first step in an attempt to fix the tomfoolery (no relation to Tom) brought in by include-what-you-use
- shuffled around some cmake linking to simplify dependency graph
- superfluous files removed
5 months ago
dr7ana 12a89328fe Cmake restructuring 5 months ago
Jason Rhinelander b597ae5a94 "Refactor" aka delete Crypto/CryptoManager
- Get rid of CryptoManager.
- Get rid of Crypto.
- Move all the Crypto instance methods to llarp::crypto functions.
  (None of them needed to be methods at all, so this is simple).
- Move sodium/ntru initialization into static initialization.
- Add llarp::csrng, which is an available llarp::CSRNG instance which is
  a bit easier than needing to construct a `CSRNG rng{};` in various
  places.
- Various related small simplifications/cleanups.
5 months ago
dr7ana c0b19de963 Review commit 5 months ago
dr7ana 66ec886480 Compilation fixes
- almost all errors have been commented out for refactor or already refactored
- committing this prior to sorting out the cmake structure
- upcoming include-what-you-use application
5 months ago
dr7ana 88fd0852db Outbound context absorbed sendcontex
- message transmission routed through refactored handling
- still work to be done, but now to make it compile at least
5 months ago
dr7ana 6338f3b3cd introset and message transmission underway
- message handling through classes that inherit from PathSet
- cleanups around link_manager
- etc etc
5 months ago
dr7ana cc08c7cdaa initial commit for next PR 5 months ago
Thomas Winget 5441e27fa4 implement sending "path control" (onioned control) message
TODO: handle at transit hop, handle at terminal hop, create and handle responses
5 months ago
Jason Rhinelander b805f55ba8 Coalesce from_string/from_string_view
The version taking a string_view will already be perfectly callable with
a string, so just combine them into one.
5 months ago
Jason Rhinelander cb6d594a89 Remove RC key rotation
We don't have/want this anymore (and on top of that, we *never* used
it in the first place and it probably didn't even work).
5 months ago
Jason Rhinelander 570e87e1d4 Remove optional relay whitelist disabling
Whitelisting is now always-on for relays.  Disabling the option is never
used and is unsupported/unmaintained (it was, in theory, to allow
lokinet as a separate usage in a non-service-node relay mode, i.e. on a
completely separate network).

Confusingly, the option was enabled by the `[lokid]:enabled` config
parameter.
5 months ago
Jason Rhinelander 0b45024883 Remove router nickname
I don't think Kee needs it anymore.
5 months ago
Jason Rhinelander 98f2d02103 Untangle messy nested dependency tree
lokinet-cryptography depends on lokinet-base which depends on
lokinet-cryptography.

Fixed this but splitting lokinet-cryptography into the libntrup code
(which is what is really needed) and the llarp/crypto code (which isn't
needed for lokinet-base).
5 months ago
dr7ana e4315cdc69 More message handling underway 9 months ago