| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
| |
We made this job easy this time around: by incrementing our MSRV, we
have forced ourselves to do at least a minor bump everywhere.
|
| |
|
|
|
| |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/861#note_2856922
|
| | |
|
| |
|
|
| |
Fixes #630
|
| | |
|
| | |
|
| |
|
|
| |
BridgeConfig is itself an Arc now, so these are redundant.
|
| |\
| |
| |
| |
| | |
Fix warnings and breakage in nightly (2022-11-18 edition)
See merge request tpo/core/arti!872
|
| | |
| |
| |
| | |
These slipped in while nightly was broken.
|
| | |
| |
| |
| |
| | |
It looks like, despite a few false starts, they've got this warning
right; there weren't any false positives.
|
| |/ |
|
| |\
| |
| |
| |
| | |
dirmgr: Expopse DirMgrStore
See merge request tpo/core/arti!851
|
| | |
| |
| |
| |
| |
| |
| | |
Now the Store is constructed by arti_client, solving the problem
described here
https://gitlab.torproject.org/tpo/core/arti/-/issues/631#note_2853665
but in a different way.
|
| | |
| |
| |
| | |
rustfmt only, broken out to ease review.
|
| | |
| |
| |
| | |
We are going to change DirMgr's constructors to also take a DirMgrStore.
|
| | | |
|
| | |
| |
| |
| | |
This will allow us to get rid of a needless trait impl.
|
| | |
| |
| |
| |
| |
| |
| |
| | |
This seems tidier to me.
(It saved a lot of typing when I was trying to make some complicated
pub supertrait with sealed private supertrait, but apparently not
now.)
|
| |\|
| |
| |
| |
| | |
bridge descriptor dormancy: Define an enum and semantics
See merge request tpo/core/arti!845
|
| | |
| |
| |
| | |
But right now, don't do anything with it. That will come in a future MR.
|
| | | |
|
| | |
| |
| |
| | |
This is part of #630.
|
| |/ |
|
| |\
| |
| |
| |
| | |
Use ErrorKind::TorAccessFailed for misbehaving bridges
See merge request tpo/core/arti!842
|
| | |
| |
| |
| | |
(Use semantic linefeed for the doc comment.)
|
| |\ \
| | |
| | |
| | |
| | | |
bridge descs: Add tests for process_document
See merge request tpo/core/arti!841
|
| | |/ |
|
| |\ \
| | |
| | |
| | |
| | | |
Remove TODO re BridgeDescDownloadConfig defaults
See merge request tpo/core/arti!839
|
| | |/ |
|
| | | |
|
| | |
| |
| |
| |
| | |
We have a separate BridgeDescMgr, rather than reusing DirMgr, so we
will not be implementing BridgeDescProvider for DirMgr.
|
| |/ |
|
| |\
| |
| |
| |
| |
| |
| | |
Persistently cache bridge descriptors
Closes #619
See merge request tpo/core/arti!831
|
| | |
| |
| |
| |
| |
| | |
This is more consistent with our naming elsewhere.
Suggested-by: Nick Mathewson <[email protected]>
|
| | | |
|
| | |
| |
| |
| |
| | |
As per
https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/831#note_2851764
|
| | | |
|
| | | |
|
| | |
| |
| |
| | |
The current test case doesn't use this.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This introduces the new API functions to Store.
But currently they are all no-ops.
So all this machinery doesn't actually do anything.
Also, it changes the API to the mockable downloader, to allow it to
support if-modified-since. So this isn't used either. But it is more
convenient to do this all at once in BridgeDescManager, since care
needs to be taken about the intraction between if-modified-since and
the persistent cache.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
We use the one in the dirmgr. That means that our constructor now has
to take a dirmgr. And, the dirmgr must have a circmgr.
This is all rather odd, TBH.
Add .. to the binding of the return values from setup, as
future-proofing.
The tests now need to provide a Store too. Make the sqlite::new_empty
function pub(crate) so we can use it.
We must retain the _db_tmp_dir, since when it goes away the tmp
directory is deleted and the db goes readonly.
|
| | |
| |
| |
| | |
This lets us see the log messages. They look fine, as it happens.
|
| |\|
| |
| |
| |
| | |
Small misc yaks for bridge descriptor persistence
See merge request tpo/core/arti!828
|
| | |
| |
| |
| | |
This allows use with tor_error::Report.
|
| | |
| |
| |
| |
| | |
This will let us share it with the bridge descriptor manager.
(As discussed with Nick.)
|
| | |
| |
| |
| |
| |
| |
| |
| |
| | |
* Centralise applying the schema updates, in a closure
* Make the schema update SQL texts be in an array so we can loop
* Make the version update statement generic, not cloned-and-hacked
This will make it possible to add another schema version without
error-prone conditions etc.
|