Commit 75a87f0
committed
Implement adr-9: optionally move to the new tracing-support module for logging.
Summary of changes:
* Existing `error-support` crate re-exports `log`, `info`, `warn`, `trace` etc. All existing components are changed so that instead of using `log::` directly we use the re-exports from `error-support`.
* `error-support` also grows a `testing` feature to expose a couple of helpers for tests.
* new crate `tracing-support` which also re-exports `log`, `info`, `warn`, `trace` etc and also has a `testing` feature which re-exports the same test functions.
* `error-support` also grows new features to control whether it re-exports the log calls from the `log` and `env_logger` crates, or from `tracing-support`.
The intent here is that in app-services we will use the feature which uses `tracing` (because we also have `rust_log_forwarder` setup to support that, but initially on desktop we will enable the feature to continue using `log` while we work on setting up an appropriate tracing subscriber there.
There's also a feature to control how error reporting is done - by default it is done like has always been done on mobile, but is setup so that on Desktop it will use the tracing facility. This means we can eventually end up using a single tracing subscriber for both error reports and logs (and also any other event types we invent in the future).1 parent fa8a72a commit 75a87f0
File tree
189 files changed
+1628
-1101
lines changed- components
- autofill
- src
- db
- sync
- address
- credit_card
- tests
- context_id
- src
- example
- src
- fxa-client/src
- internal
- state_machine/internal_machines
- logins
- src
- sync
- merino
- src/curated_recommendations
- nimbus
- examples
- src
- stateful
- client
- tests
- stateful
- client
- tests
- common
- places
- src
- api
- bookmark_sync
- db
- tx
- history_sync
- import
- ios
- storage
- bookmarks
- push
- src
- internal
- communications
- storage
- relevancy
- src
- remote_settings
- src
- search
- src
- suggest
- src
- testing
- support
- error
- src
- find-places-db
- src
- jwcrypto
- restmail-client
- src
- rust-log-forwarder
- android
- src
- sql
- src
- tracing
- android
- src/main
- src
- viaduct-reqwest
- src
- sync15
- src
- bso
- clients_engine
- client
- engine
- sync_manager
- src
- tabs
- src
- sync
- viaduct
- src
- backend
- headers
- webext-storage
- ffi
- src
- src
- sync
- megazords
- full
- android
- src
- ios-rust
- focus
- src
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
189 files changed
+1628
-1101
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
99 | 99 | | |
100 | 100 | | |
101 | 101 | | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
102 | 109 | | |
103 | 110 | | |
104 | 111 | | |
| |||
0 commit comments