summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | arti_rpc_tests: use Iterator, not GeneratorNick Mathewson2024-10-281-2/+2
| | | | | | | | | | | | Generator is more abstract, and expects a possible return or send type.
| * | update-md-links: annotate a Popen generic.Nick Mathewson2024-10-281-1/+2
| |/ | | | | | | For some reason this will soon be necessary with mypy.
* | Merge branch 'chutney-artifact' into 'main'opara2024-10-291-0/+5
|\ \ | | | | | | | | | | | | ci: upload 'shadow.chutney.data/' artifact in 'integration-chutney-shadow' See merge request tpo/core/arti!2586
| * | ci: upload more artifacts in 'integration-chutney-shadow'Steven Engler2024-10-281-0/+5
| | | | | | | | | | | | Set "1 week" since I see that it's used on another shadow artifact.
* | | Merge branch 'rpc-spec-tweaks-1711' into 'main'Nick Mathewson2024-10-292-127/+147
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Final (ha!) clean-ups on rpc connect-string specs Closes #1711 See merge request tpo/core/arti!2582
| * | rpc: clarify percent-encoding rules.Nick Mathewson2024-10-281-5/+10
| | |
| * | rpc: refer to TOML tables, not sections.Nick Mathewson2024-10-281-5/+5
| | |
| * | rpc: centralize treatment of unrecognized sections/members.Nick Mathewson2024-10-281-2/+2
| | |
| * | rpc: clarify "hidden file" meaning.Nick Mathewson2024-10-281-2/+3
| | |
| * | rpc: tweak "connect string" terminologyNick Mathewson2024-10-282-75/+78
| | | | | | | | | | | | | | | With all the fanciness we've built up here here, it seems that "connect point" is a better term than "connect point".
| * | rpc-connect-sketch: convert to toml.Nick Mathewson2024-10-271-50/+61
| |/ | | | | | | | | (We've decided to make this change because these files are meant to be written by humans.)
* | Merge branch 'chutney-deps' into 'main'Jim Newsome2024-10-281-1/+16
|\ \ | | | | | | | | | | | | CI: pip install chutney to get its dependencies See merge request tpo/core/arti!2587
| * | integration-chutney-shadow: pip install chutney to get its dependenciesJim Newsome2024-10-281-0/+8
| | |
| * | .gitlab-ci: reformat long apt-install lineJim Newsome2024-10-281-1/+8
|/ /
* | Merge branch 'shadow-commit' into 'main'Jim Newsome2024-10-281-2/+3
|\ \ | | | | | | | | | | | | ci: use latest shadow commit See merge request tpo/core/arti!2585
| * | ci: use latest shadow commitSteven Engler2024-10-281-2/+3
|/ / | | | | | | | | The changes in https://github.com/shadow/shadow/pull/3428 should add more details to the CI output of Shadow simulations.
* | Merge branch 'orport' into 'main'opara2024-10-282-85/+99
|\ \ | |/ |/| | | | | tor-chanmgr: revert `PendingChannelHandle` drop behaviour, and try to prevent problematic code paths See merge request tpo/core/arti!2566
| * tor-chanmgr: rename `replace_pending_channel` to ↵Steven Engler2024-10-242-9/+10
| | | | | | | | `upgrade_pending_channel_to_open`
| * tor-chanmgr: `remove_pending` now takes a `PendingChannelHandle`Steven Engler2024-10-241-18/+8
| |
| * tor-chanmgr: improve cleanup procedure of pending channelsSteven Engler2024-10-242-47/+92
| | | | | | | | | | | | | | | | | | | | An attempt to make sure that there are no code paths which forget to remove a pending channel from the channel map. This also adds error-level log messages and panics during debug builds if a `PendingChannelHandle` is dropped without properly passing it to `MgrState::remove_pending_channel` or `MgrState::replace_pending_channel`.
| * Revert "tor-chanmgr: `PendingChannelHandle` removes the channel when dropped"Steven Engler2024-10-212-56/+34
| | | | | | | | | | | | | | | | This reverts commit f85bc3cf849109aaa2c4da9fc8c06e7173a0543f. There were some small conflcits in `AbstractChanMgr::get_or_launch_internal`, so this wasn't a clean revert.
* | Merge branch 'arti-rpcserver-remove_echo_on_session' into 'main'Nick Mathewson2024-10-271-26/+0
|\ \ | | | | | | | | | | | | arti-rpcserver: Remove echo_on_session See merge request tpo/core/arti!2549
| * | Remove echo structsNeel Chauhan2024-10-251-14/+0
| | |
| * | arti-rpcserver: Remove echo_on_sessionNeel Chauhan2024-10-251-12/+0
|/ /
* | Merge branch 'rpc-move-docs' into 'main'Nick Mathewson2024-10-243-0/+0
|\ \ | | | | | | | | | | | | Move RPC documents into a new location See merge request tpo/core/arti!2581
| * | Move RPC documents into a new locationNick Mathewson2024-10-243-0/+0
|/ / | | | | | | This will be the start of an mdbook document.
* | Merge branch 'rpc-server-sketch' into 'main'Nick Mathewson2024-10-242-15/+201
|\ \ | | | | | | | | | | | | | | | | | | Sketch RPC server and superuser operations. Closes #1702 See merge request tpo/core/arti!2563
| * | rpc-sketch: clean up some ungrammatical/hypothetical language.Nick Mathewson2024-10-241-2/+2
| | |
| * | rpc sketch: show how to override a connect point option.Nick Mathewson2024-10-241-0/+5
| | |
| * | rpc-sketch: specify and permit directories full of connect files.Nick Mathewson2024-10-241-10/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The main purpose of this commit is to RPC servers to configure connect points individually, while allowing clients to look at a configuration location that is full of multiple connect points. This change permits us to discard allow_missing_file, which is nice. This commit still pretends that the file extension is "json"; that will be fixed in #1711.
| * | rpc-connect: revise description of superuser and fs-mistrust options.Nick Mathewson2024-10-231-5/+17
| | |
| * | rpc-connect: Rewrite server behavior section.Nick Mathewson2024-10-231-16/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The most salient change here is that _everything_, including the default connect string, is a configuration option, so that `--system-arti` (hypothetical) can override it. In order to make the default connect file locations behave like regular options, I've added a `allow_missing_file` sub-option that makes the absence of a file tolerable. There is a behavioral change here: Specifying your own `rpc.listen` entries in the arti.toml _does not_ disable checking the default connect file locations.
| * | rpc-connect: Mention #1710 when talking about system-arti case.Nick Mathewson2024-10-231-0/+6
| | |
| * | rpc-connect-sketch: normalize some jsonNick Mathewson2024-10-211-6/+6
| | |
| * | Apply 2 suggestion(s) to 2 file(s)Nick Mathewson2024-10-212-2/+2
| | | | | | | | | Co-authored-by: Ian Jackson <[email protected]>
| * | rpc-*: sketch superuser operations.Nick Mathewson2024-10-212-1/+39
| | |
| * | rpc-connect-sketch: Sketch RPC server operationNick Mathewson2024-10-211-0/+44
| | |
| * | rpc-connect-sketch: Fix ambiguous use of "embedded"Nick Mathewson2024-10-211-1/+1
| | |
| * | rpc-connect-sketch: more sketching about embedded arti.Nick Mathewson2024-10-211-9/+25
| | |
| * | rpc-connect-sketch: expand on defaults.Nick Mathewson2024-10-211-1/+20
| | |
* | | Merge branch 'slug-colons' into 'main'Ian Jackson2024-10-241-0/+4
|\ \ \ | | | | | | | | | | | | | | | | tor-persist: Explain why colons will never be valid slug chars. See merge request tpo/core/arti!2576
| * | | tor-persist: Explain why colons will never be valid slug chars.Gabriela Moldovan2024-10-231-0/+4
| | | |
* | | | Merge branch 'p101-2024-q3' into 'main'Alexander Hansen Færøy2024-10-241-0/+26
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Update P101 numbers for Q3-2024. See merge request tpo/core/arti!2571
| * | | | Update P101 numbers for Q3-2024.Alexander Færøy2024-10-221-0/+26
| | | | |
* | | | | Merge branch 'python-warnings-1' into 'main'Nick Mathewson2024-10-2423-86/+128
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Fix various python warnings See merge request tpo/core/arti!2579
| * | | | | python: Make mypy accept postprocess_coverage_*Nick Mathewson2024-10-242-5/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This has a little complexity since beautifulsoup4 is fairly well duck-typed, but it also has type annotations to describe its duck-typing.
| * | | | | Python: simple annotations to allow mypy to pass.Nick Mathewson2024-10-242-4/+5
| | | | | |
| * | | | | arti_rpc_tests: Fix a type errorNick Mathewson2024-10-242-2/+11
| | | | | | | | | | | | | | | | | | | | | | | | mypy rightly complained that our context.arti_process could be None.
| * | | | | python: Split lines over 99 chars longNick Mathewson2024-10-242-3/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `black` formatter did most line wrapping for us, and we accept its default of 88 characters there, but for comments and docstrings (which black doesn't wrap) we allow up to 99 characters.
| * | | | | python: Remove a spurious blank line.Nick Mathewson2024-10-241-1/+0
| | | | | |