| Commit message (Collapse) | Author | Age | Files | Lines |
| ... | |
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
safelog: Apply #[serial] to test_redacted()
Closes #675
See merge request tpo/core/arti!914
|
| | | |
| | |
| | |
| | | |
Closes #675.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Dispose of ptmgr TODOs (and remove a dead variable)
See merge request tpo/core/arti!912
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | | |
This is used exclusively by the reactor.
|
| | |/ /
| | |
| | |
| | |
| | | |
We may want to change this later, but this API is in a low level
crate, so this isn't a blocker.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Separate BridgeAddr and PtTargetAddr
See merge request tpo/core/arti!904
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/904#note_2858480
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/904#note_2858479
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/issues/668#note_2858220
This commit is difficult to split up.
The innards of BridgeAddr and PtTargetAddr are still a bit entangled.
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Resolve two bugs in our pt-socks implementation
Closes #676
See merge request tpo/core/arti!913
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
pt-spec says:
```
If the encoded argument list is less than 255 bytes in
length, the "PLEN" field must be set to "1" and the "PASSWD"
field must contain a single NUL character.
```
This is a very silly thing to say, but some PTs enforce it.
Fixes #676.
|
| | | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Our previous code would make a TCP connection to the proxy, then
wait forever for it to say something. Whoops!
Part of a fix for #676.
|
| | | | | | |
|
| | | |_|/
| |/| | |
|
| |\ \ \ \
| |_|_|/
|/| | |
| | | |
| | | | |
Abolish ChanMgr::set_default_transport
See merge request tpo/core/arti!910
|
| | | |/
| |/|
| | |
| | |
| | |
| | | |
The comment says
// TODO pt-client: It's not clear to me that we really need this method.
and empirically, deleting it, and its callee, is fine.
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Dispose of all TODO pt-client in arti-*
See merge request tpo/core/arti!909
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
Since apparently it was possible to write all the other code without
moving it.
|
| | |/ / |
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
Changelog blurb
See merge request tpo/core/arti!908
|
| | | | |
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
ptmgr: Do not construct an empty select_all future.
Closes #672 and #670
See merge request tpo/core/arti!907
|
| | |/ /
| | |
| | |
| | |
| | |
| | | |
Closes #670.
Closes #672.
|
| |\ \ \
| |_|/
|/| |
| | |
| | | |
ptmgr: error fixes
See merge request tpo/core/arti!906
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Rename it from UnconfiguredTransport. Change descriptions, kind, and
retry. Add a TODO.
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/901#note_2858455
|
| | | |
| | |
| | |
| | |
| | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/901#note_2858454
|
| | | |
| | |
| | |
| | |
| | | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/901#note_2858456
|
| |\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
Fix a logic error in Redacted.
Closes #671
See merge request tpo/core/arti!905
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | | |
Previously it was redacting exactly when safelogging was _disabled_,
which obviously isn't correct.
Fixes #671. Regression test included.
|
| |\ \ \
| | |/
| |/|
| | |
| | | |
Merge PtReactor, resolve a few of my comments.
See merge request tpo/core/arti!903
|
| | | |
| | |
| | |
| | |
| | | |
(I think this may be impossible now, since we check in ipc.rs, but
IMO it's best to check.)
|
| | | | |
|
| | | | |
|
| | | |
| | |
| | |
| | | |
This lets us avoid logging the user's path.
|
| | | | |
|
| | |\ \
| | |/
| |/|
| | |
| | | |
Resolved several conflicts while doing so; mostly with !893 and
!895.
|
| | | |
| | |
| | |
| | |
| | |
| | | |
This fleshes out the PtReactor that was previously a stub, making it
actually do reactor-ey things. Note that this is entirely untested, and
might not work at all. Hopefully it does, though.
|
| |\ \ \
| |/ /
|/| |
| | |
| | | |
arti-client: Try to take ownership of state earlier.
See merge request tpo/core/arti!902
|
| |/ /
| |
| |
| |
| |
| |
| |
| | |
There's no harm in doing this, since we'll try to take ownership of
the state soon as soon as we use it: We just want to try early, so
that we'll get it before we decide that we're using bridges.
Fixes bug #669; bug introduced by !889.
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
chanmgr: resolve several remaining "TODO pt-client" issues
See merge request tpo/core/arti!897
|
| | | | |
|
| | | | |
|
| | |/
| |
| |
| |
| | |
We were panicking if the PtMgr gave us an error, which isn't so
good.
|
| |\ \
| | |
| | |
| | |
| | | |
tor-basic-utils: list_builder: Use standard test lint block
See merge request tpo/core/arti!900
|