summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | 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
| | | | | |
| * | | | | python: one import per line.Nick Mathewson2024-10-242-2/+4
| | | | | |
| * | | | | Python: use a more lint-acceptable block comment format.Nick Mathewson2024-10-241-3/+6
| | | | | |
| * | | | | python: Avoid using "l" as a variable name.Nick Mathewson2024-10-242-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | (Python coding style guide and lint tools deprecate it, even if your font distinguishes l, 1, I, and |.)
| * | | | | rpc-docs-tool: imports go at the head of the module.Nick Mathewson2024-10-241-7/+7
| | | | | |
| * | | | | python: Use "a not in b" rather than "not a in b".Nick Mathewson2024-10-243-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | Due to precedence, both work the same, but the former is preferred.
| * | | | | python: Fix warnings about unused variables and modules.Nick Mathewson2024-10-249-15/+13
| | | | | |
| * | | | | Add a pyproject.toml for arti_rpc_testsNick Mathewson2024-10-241-0/+23
| | | | | |
| * | | | | Remove obsolete "rpc_demo.py"Nick Mathewson2024-10-241-24/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It no longer reflects our current API, refers to no-longer-extant bugs, and was never a terribly good example of what you were supposed to do with RPC.
* | | | | | Merge branch 'vanguards-stems' into 'main'gabi-2502024-10-247-255/+285
|\ \ \ \ \ \ | |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | doc/dev/notes/vanguards: Use the new circuit stem terminology. Closes #1479 See merge request tpo/core/arti!2410
| * | | | | tor-circmgr: Clarify that using guarded for client rend is an exception.Gabriela Moldovan2024-10-242-1/+27
| | | | | |
| * | | | | tor-circmgr: Rename Short/Extended to Naive/Guarded.Gabriela Moldovan2024-10-244-32/+32
| | | | | |
| * | | | | tor-circmgr: Try to clarify what a "circuit stem" is.Gabriela Moldovan2024-10-241-2/+3
| | | | | |
| * | | | | tor-circmgr: Post-renaming documentation fixes.Gabriela Moldovan2024-10-244-14/+14
| | | | | |
| * | | | | tor-circmgr: s/stub/stem throughout.Gabriela Moldovan2024-10-244-100/+100
| | | | | |
| * | | | | tor-circmgr: s/HsCircStubKind/HsCircStemKind.Gabriela Moldovan2024-10-242-27/+27
| | | | | | | | | | | | | | | | | | | | | | | | As per #1479
| * | | | | tor-circmgr: s/HsCircStubKind/HsCircStemKind.Gabriela Moldovan2024-10-245-60/+60
| | | | | | | | | | | | | | | | | | | | | | | | As per #1479
| * | | | | tor-circmgr: Use the new vanguards terminology throughout.Gabriela Moldovan2024-10-243-75/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is the first step towards clarifying the questions from !2230. Corresponding torspec changes: https://gitlab.torproject.org/tpo/core/torspec/-/merge_requests/282 Part of #1479
| * | | | | doc/dev/notes/vanguards: Use the new circuit stem terminology.Gabriela Moldovan2024-10-241-25/+28
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Circuit stubs" are now called "circuit stems". We are also replacing the STUB/STUB+ terminology (STUB/STUB+ circuits are now called NAIVE/GUARDED). Part of #1479
* | | | | Merge branch 'run-black' into 'main'Alexander Hansen Færøy2024-10-2422-438/+661
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Run "black" to reformat all our python See merge request tpo/core/arti!2578
| * | | | | Run "black" to reformat all our pythonNick Mathewson2024-10-2422-438/+661
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Black" is an "opinionated" python formatter, whose opinionatedness is somewhat in the spirit of rustfmt. This MR runs black with default settings on all of our python code in Arti. It was produced by the following commands ``` # Everything in python/ black python/ # Everything with a .py extension fd '.py$' -X black # Everything with a python shebang. git grep -l '#! */usr/bin/env *python' | xargs black ```
* | | | | Merge branch 'rpc-drop' into 'main'Nick Mathewson2024-10-2414-100/+307
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | rpc: Move `rpc:release` method to apply to all object-ids. Closes #1663 See merge request tpo/core/arti!2573
| * | | | | rpc: Tweak release_owned to give an error on second removal of "connection".Nick Mathewson2024-10-232-16/+30
| | | | | |
| * | | | | rpc: Use a const for the "connection" object ID.Nick Mathewson2024-10-231-2/+9
| | | | | |
| * | | | | rpc: Simplify method-dispatch-bypass logic a bitNick Mathewson2024-10-235-34/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit removes the separate function for asking whether to bypass the dispatch code. Instead, it gives the "invoke with bypass" function an error to return when no dispatch is warranted, and moves the whole responsibility for method dispatch or non-dispatch back into tor-rpcbase. I had to add an ObjectId argument to `invoke_rpc_method` to make this work, but that's probably a good thing. Additionally, this commit tweaks the derive-deftly macro to prevent you from asking for dispatch bypass on special methods, where it isn't implemented (and doesn't really make sense).
| * | | | | rpc: Re-run cbindgen.Nick Mathewson2024-10-231-1/+0
| | | | | |
| * | | | | python: Use logging to report nonfatal errors.Nick Mathewson2024-10-232-2/+8
| | | | | |