🧹 Improve redirect status code logging readability#60
🧹 Improve redirect status code logging readability#60ToolchainLab wants to merge 2 commits intomainfrom
Conversation
Changed the redirect status code logging in `src/main.rs` to use the `StatusCode`'s `Display` implementation instead of explicitly calling `.as_u16()`. This provides a more descriptive log message that includes both the numeric code and its canonical reason phrase (e.g., "308 Permanent Redirect" instead of "308"). Co-authored-by: ToolchainLab <263750431+ToolchainLab@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
Changed the redirect status code logging in `src/main.rs` to use the `StatusCode`'s `Display` implementation instead of explicitly calling `.as_u16()`. This provides a more descriptive log message that includes both the numeric code and its canonical reason phrase (e.g., "308 Permanent Redirect" instead of "308"). Co-authored-by: ToolchainLab <263750431+ToolchainLab@users.noreply.github.com>
1 participant
The current logging for the redirect status code only showed the numeric value (e.g., 308). By using the
Displayimplementation ofaxum::http::StatusCode, the logs now include the descriptive reason phrase as well, improving maintainability and readability.🎯 What: Changed
redirect_status.as_u16()toredirect_statusin theinfo!macro insrc/main.rs.💡 Why: This leverages the
Displayimplementation ofStatusCode, which is more informative for developers monitoring the logs.✅ Verification: Verified the change in
src/main.rsand rancargo fmt. Attemptedcargo test, which was blocked by the environment but is safe given the trivial nature of the change. Removed temporary experimental files.✨ Result: More readable startup logs for the redirect server.
PR created automatically by Jules for task 16525599607447945517 started by @ToolchainLab