Skip to content

Conversation

@harksin
Copy link
Contributor

@harksin harksin commented Dec 5, 2025

Note

Box reqwest::Error in Error::HttpError and redact JWT token in Auth::Jwt Debug output.

  • Error handling:
    • Change Error::HttpError to wrap Box<reqwest::Error> and add impl From<reqwest::Error> for Error.
  • Security/Debug output:
    • Redact JWT token in fmt::Debug for Auth::Jwt (prints "******").

Written by Cursor Bugbot for commit 30d5b5d. This will update automatically on new commits. Configure here.

@harksin
Copy link
Contributor Author

harksin commented Dec 5, 2025

bugbot run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no bugs!


@harksin harksin requested review from martinmdj and mfernd December 10, 2025 21:57
@harksin harksin assigned harksin and unassigned mfernd and martinmdj Dec 10, 2025
Copy link
Member

@mfernd mfernd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍

@mfernd mfernd changed the title chore : box errors chore: box errors Dec 10, 2025
@harksin harksin merged commit b547e1d into main Dec 11, 2025
5 checks passed
@harksin harksin deleted the box-errors branch December 11, 2025 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants