-
Notifications
You must be signed in to change notification settings - Fork 207
Added functionlaity for logging information #241
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
base: main
Are you sure you want to change the base?
Conversation
examples/transport/Cargo.toml
Outdated
"time", | ||
] } | ||
tokio = { version = "1", features = ["full"] } | ||
# tokio = { version = "1", features = [ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just delete these
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@4t145 i have removed line 19 nd reverted to 20, i don't see any ci error again, kindly check
Could you fix the ci errors? |
Cool , I will delete them
…On Tue, 3 June 2025, 3:42 am 4t145, ***@***.***> wrote:
*4t145* left a comment (modelcontextprotocol/rust-sdk#241)
<#241 (comment)>
Could you fix the ci errors?
—
Reply to this email directly, view it on GitHub
<#241 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AE4COXAEAUT3WEJO7N2343D3BUDRLAVCNFSM6AAAAAB6MTXWKOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDSMZTGE3TGMRRHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
@4t145 kindly view, |
CI failed again. Seems like hyper implmented cargo clippy --all-targets --all-features
cargo +nightly fmt --all |
@4t145 thanks, i will look into this |
Added server startup log with bound address Cloned service handler for proper connection handling Improved connection logging with client addresses
Added enhanced logging function
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context