Skip to content

feat: Redact bearer tokens in insta snapshots - #325

Merged
spencewenski merged 3 commits into
mainfrom
redact-auth-tokens
Aug 11, 2024
Merged

feat: Redact bearer tokens in insta snapshots#325
spencewenski merged 3 commits into
mainfrom
redact-auth-tokens

Conversation

@spencewenski

@spencewenski spencewenski commented Aug 11, 2024

Copy link
Copy Markdown
Member

Also, implement From for various Subject enum variants.

@codecov

codecov Bot commented Aug 11, 2024

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.49%. Comparing base (9ede96c) to head (5823396).

Files Coverage Δ
src/middleware/http/auth/jwt/mod.rs 68.75% <100.00%> (+15.41%) ⬆️
src/testing/snapshot.rs 100.00% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9ede96c...5823396. Read the comment docs.

@spencewenski
spencewenski merged commit 9b054a6 into main Aug 11, 2024
@spencewenski
spencewenski deleted the redact-auth-tokens branch August 11, 2024 21:36
@github-actions github-actions Bot mentioned this pull request Aug 12, 2024
spencewenski added a commit that referenced this pull request Aug 12, 2024
## 🤖 New release
* `roadster`: 0.5.18 -> 0.5.19

<details><summary><i><b>Changelog</b></i></summary><p>

<blockquote>

##
[0.5.19](roadster-v0.5.18...roadster-v0.5.19)
- 2024-08-12

### Added
- Redact bearer tokens in insta snapshots
([#325](#325))

### Fixed
- Do not simply use bearer token from cookie for auth
([#326](#326))
- Set default value for Jwt cookie name
([#324](#324))
- Derive `Clone` in JWT claim types
([#323](#323))
- Use `leptos_routes` in leptos example instead of
`leptos_routes_with_context`
([#322](#322))

### Other
- *(deps)* Bump EmbarkStudios/cargo-deny-action from 1 to 2
([#319](#319))
</blockquote>


</p></details>

---
This PR was generated with
[release-plz](https://github.com/MarcoIeni/release-plz/).

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Spencer Ferris <3319370+spencewenski@users.noreply.github.com>
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.

1 participant