-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Adopt Tracing #1410
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Adopt Tracing #1410
Changes from 1 commit
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
f5a8681
WIP
hawkw debb008
filtering, nicer formatting
hawkw e0a0bfd
more trace
hawkw 6e0e902
fix filtering
hawkw 47540a9
mounting works
hawkw 88cee8c
WIP
hawkw d920738
skip span names for "rocket-ier" formatitng
hawkw bce8f14
this is cute, right?
hawkw e3398e5
fix contrib
hawkw 86c5ef5
use span names if there's no message
hawkw 88feb8e
start removing underscore macros
hawkw cbe5581
less obnoxious bold, nicer debug/trace
hawkw 43c95db
don't constantly screw up colors
hawkw 1eec8f6
unpatch
hawkw 2589dee
add docs & reexports
hawkw 533d792
cleanup, remove logger module
hawkw e7b55d7
whoops
hawkw 2dbd927
Merge branch 'master' into eliza/trace
hawkw 4bd3da2
trailing whitespace
hawkw 9905887
trailing whitespace again
hawkw 118e13e
fixup
hawkw 03533ce
use `macro_use` imports everywhere
hawkw 9e6adc6
use spans for all indentation-based messages
hawkw ec5066b
update codegen
hawkw 6144caa
add spans to generated catcher/route fns
hawkw e1b67db
make catcher spans info
hawkw cd124d8
bold span names when no message is present
hawkw bc492ba
add more info to generated spans
hawkw 88e965e
Replace `log` with `tracing` and `tracing-subscriber`.
hawkw 6fbbf94
style: remove most usage of span.enter()
jebrosen c8e4b6b
fix: macros are weird sometimes
jebrosen eeeab88
fix: use of moved value
jebrosen 41237b6
Merge remote-tracking branch 'jebrosen/adopt-tracing' into eliza/trace
hawkw 9f86214
allow libtest to capture trace logs
hawkw 9e95437
remove workaround
hawkw 9776ca3
remove workarounds, add `log` compat
hawkw fa5f30d
examples for a custom subscriber, env_logger
hawkw 73ca07c
Merge branch 'master' into eliza/trace
hawkw e68d66d
use structured fields in more places
hawkw 4e9a308
post-rebase fixup
hawkw a868cab
use `in_scope` for errors
hawkw cb1b024
allow adding directives to Rocket's filter
hawkw 6de1180
make examples a little fancier
hawkw 4ddaedf
fix trace in contrib
hawkw 0025a58
propagate spans to spawned threads
hawkw f15d432
put all request processing inside request span
hawkw dd77f67
add another span in 'Fairings::pretty_print_counts'
jebrosen d5006de
feat: use rocket::trace::info in fairings, manual_routes examples
jebrosen 4bf5f33
remove trailing whitespace
jebrosen 50a224d
fix(codegen): log parameter names in a way that will work for any kin…
jebrosen 55faca7
fix compile error in manual_routes example
jebrosen 732a613
fix(test): missing semicolon
jebrosen 8dba51d
feat: rework log output formatting
jebrosen 39ae1d5
feat: print request IDs
jebrosen 910ad1c
feat/experiment: remove connection span, making 'request' the top-lev…
jebrosen 63e0ef3
handle `log` metadata properly
hawkw 8ac9625
use tracing instead of log in http
jebrosen 2f098a7
call Paint::disable in more cases of 'tracing' initialization failure
jebrosen 2cfad83
add new log level 'support'
jebrosen d69340c
fixup: add string form to description of 'support' to make consistent…
jebrosen 0d506c1
Use 'in_scope' instead of 'enter' in most places.
jebrosen c19799b
fix: under feature="log", skip formatting of fields whose names start…
jebrosen 85f2e5b
fix: test the 'log' feature individually
jebrosen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.