summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | rpc: Remove prefix from RpcCode members.Nick Mathewson2024-10-171-20/+19
|/ / / /
* | | | Merge branch 'single_match_warn_in_memqueue' into 'main'Alexander Hansen Færøy2024-10-171-0/+1
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | memquota: suppress clippy::single_match warning. See merge request tpo/core/arti!2555
| * | | memquota: suppress clippy::single_match warning.Nick Mathewson2024-10-171-0/+1
|/ / / | | | | | | | | | | | | "Suppress" rather than "fix": this pattern is intentional, and makes it clear that we're doing an exhaustive match.
* | | Merge branch 'tor-key-forge-cleanup' into 'main'gabi-2502024-10-173-7/+12
|\ \ \ | | | | | | | | | | | | | | | | tor-key-forge: Minor cleanups and reformatting See merge request tpo/core/arti!2552
| * | | tor-key-forge: Fix typo in documentation.Gabriela Moldovan2024-10-171-2/+2
| | | |
| * | | tor-key-forge: Replace fully-qualified path with import.Gabriela Moldovan2024-10-171-2/+2
| | | | | | | | | | | | | | | | | | | | `KeypairData` is already in scope, so there's no need to fully-qualify it.
| * | | tor-key-forge: Reformat derives for readability.Gabriela Moldovan2024-10-171-2/+4
| | | |
| * | | tor-key-forge: Reformat a long line for readability.Gabriela Moldovan2024-10-171-1/+4
| | | |
* | | | Merge branch 'dead' into 'main'Alexander Hansen Færøy2024-10-1712-3/+41
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Disable a lot of dead code warnings and add a default-runtime feature in arti See merge request tpo/core/arti!2551
| * | | | arti: Introduce a new default-runtime Cargo featureIan Jackson2024-10-172-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now one can do a minimal build with cargo build -p arti --no-default-features --features=default-runtime rather than needing to specify tokio,native-tls.
| * | | | Disable a lot of dead code warnings (fmt)Ian Jackson2024-10-173-3/+12
| | | | |
| * | | | Disable a lot of dead code warningsIan Jackson2024-10-1710-1/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now cargo check --workspace --no-default-features --all-targets cargo build -p arti --no-default-features --features=memquota,tokio,native-tls are both clean.
* | | | | Merge branch 'chanmgr-select-docs' into 'main'David Goulet2024-10-171-0/+25
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | tor-chanmgr: update comments about selecting pending channels See merge request tpo/core/arti!2544
| * | | | | tor-chanmgr: update comments about selecting pending channelsSteven Engler2024-10-161-0/+25
| | | | | |
* | | | | | Merge branch 'orport' into 'main'David Goulet2024-10-172-224/+309
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-chanmgr: refactored to remove `with_channels` and added `PendingChannelHandle` See merge request tpo/core/arti!2538
| * | | | | | tor-chanmgr: remove `handle_build_outcome`Steven Engler2024-10-151-28/+19
| | | | | | |
| * | | | | | tor-chanmgr: `PendingChannelHandle` removes the channel when droppedSteven Engler2024-10-152-27/+56
| | | | | | |
| * | | | | | tor-chanmgr: added `PendingChannelHandle`Steven Engler2024-10-152-36/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This handle contains all of the details required to remove or replace a pending channel entry from the channel map.
| * | | | | | tor-chanmgr: refactor so we don't need `with_channels{,_and_params}`Steven Engler2024-10-152-78/+76
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These methods on `MgrState` acquire a lock, and it's easy for calling code to also try to acquire the same lock within the closure, causing a deadlock. It's better to not expose these methods.
| * | | | | | tor-chanmgr: fix some incorrect commentsSteven Engler2024-10-152-5/+3
| | | | | | |
| * | | | | | tor-chanmgr: removed use of `with_channels` within `request_channel`Steven Engler2024-10-151-61/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Best reviewed with the '-w' flag to hide whitespace changes.
| * | | | | | tor-chanmgr: refactored `AbstractChanMgr::choose_action`Steven Engler2024-10-152-109/+145
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This moves most of the channel map logic from `AbstractChanMgr::choose_action` to `MgrState::request_channel`. This is working towards being able to remove `MgrState::with_channels{,_and_params}`.
* | | | | | | Merge branch 'non-linux-fix' into 'main'Nick Mathewson2024-10-171-1/+3
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rtcompat: Fix android compilation in `general` See merge request tpo/core/arti!2553
| * | | | | | | rtcompat: Fix android compilation in `general`Nick Mathewson2024-10-171-1/+3
| | |_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously I had assumed that `std::os::linux` also existed on android, based on the documentation of `std::os::linux::net::SocketAddrExt`. That is not, however, true. This should fix the `check-targets` CI job.
* | | | | | | Merge branch 'rpc-connect-sketch' into 'main'Nick Mathewson2024-10-171-0/+464
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | RPC: Sketch for how connect strings work See merge request tpo/core/arti!2439
| * | | | | | rpc-connect-sketch: Clarify and restrict socket addr formatNick Mathewson2024-10-171-8/+16
| | | | | | |
| * | | | | | rpc-connect: add some example connect strings.Nick Mathewson2024-10-171-1/+32
| | | | | | |
| * | | | | | rpc-connect: clarify that "embedded" is not final.Nick Mathewson2024-10-171-0/+10
| | | | | | |
| * | | | | | rpc-connect: unknown builtins are "decline".Nick Mathewson2024-10-171-2/+2
| | | | | | |
| * | | | | | rpc-connect: replace a "may" with a "must"Nick Mathewson2024-10-171-1/+1
| | | | | | |
| * | | | | | Apply suggestions about authentication/socket relationships.Nick Mathewson2024-10-171-3/+6
| | | | | | |
| * | | | | | rpc-connect-sketch: Slightly clarify escaping.Nick Mathewson2024-10-171-5/+4
| | | | | | |
| * | | | | | rpc-connect-sketch: Revise connect string spec.Nick Mathewson2024-10-171-118/+130
| | | | | | |
| * | | | | | rpc-connect-sketch: Revise search-order-and-config sectionNick Mathewson2024-10-171-38/+76
| | | | | | |
| * | | | | | rpc-connect-sketch: Add suggested "security concerns" section.Nick Mathewson2024-10-171-1/+34
| | | | | | |
| * | | | | | rpc-connect-sketch: Add suggested clarifications about cookie file.Nick Mathewson2024-10-171-1/+16
| | | | | | |
| * | | | | | rpc-connect-sketch: Remove file type indicator.Nick Mathewson2024-10-171-3/+0
| | | | | | |
| * | | | | | rpc-connect-sketch: Classify our previously-undecided error casesNick Mathewson2024-10-171-6/+7
| | | | | | |
| * | | | | | rpc-connect-sketch: call setuid-detection nice-to-have.Nick Mathewson2024-10-171-0/+1
| | | | | | |
| * | | | | | RPC: Sketch for how connect strings workNick Mathewson2024-10-171-0/+316
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | This is based on conversations with Diziet, and is a followup from arti!2388 and arti!2429.
* | | | | | Merge branch 'fs-watcher-poll' into 'main'gabi-2502024-10-176-4/+57
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tor-config: Use a polling watcher on non-windows platforms that don't have inotify. Closes #1644 See merge request tpo/core/arti!2547
| * | | | | | arti: Add a warning about watch_configuration's performance issues.Gabriela Moldovan2024-10-171-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Part of #1644
| * | | | | | tor-config: Document the limitations of the FileWatcher.Gabriela Moldovan2024-10-162-0/+11
| | | | | | |
| * | | | | | arti: Enable tor-config/testing when testing.Gabriela Moldovan2024-10-161-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This enables us to make use of the unfortunately-necessary test-specific code path.
| * | | | | | tor-config: Compare the file contents when testing.Gabriela Moldovan2024-10-161-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When testing, compare the contents of the files too, not just their mtime Otherwise, because the polling backend detects changes based on mtime, if the test creates/writes files too fast, it will fail to notice changes (this can happen, for example, on a tmpfs).
| * | | | | | tor-config: Configure the polling interval.Gabriela Moldovan2024-10-161-2/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default is 30s, which feels a bit too long. Also, when testing, we use a much shorter interval. This is meant to be a temporary solution, so we hard-code the interval rather than make it configurable.
| * | | | | | tor-config: Use a polling watcher on non-windows platforms that don't have ↵Gabriela Moldovan2024-10-163-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | inotify. On windows and platforms that support inotify (i.e. linux and android), we continue using the recommended watcher. On platforms that use kqueue, we switch to a polling watcher to work around a [notify bug] that manifests when using a non-recursive watcher to watch a directory. This commit is best reviewed with `git diff --ignore-all-space`. Closes #1644 [notify bug]: https://github.com/notify-rs/notify/issues/644
| * | | | | | tor-config: Add a type alias for the underlying watcher (fmt).Gabriela Moldovan2024-10-161-2/+2
| | | | | | |
| * | | | | | tor-config: Add a type alias for the underlying watcher.Gabriela Moldovan2024-10-161-3/+6
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | This type will soon become platform-dependent, so I am preemptively adding a type alias.
* | | | | | Merge branch 'general_socketaddr_ops' into 'main'Nick Mathewson2024-10-177-2/+526
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | general::SocketAddr: Specify and implement string representations. Closes #1681 See merge request tpo/core/arti!2519