Skip to content

feat(server): introduces serializeError and redactError#32

Merged
reggi merged 9 commits intomainfrom
reggi/safe-errors
Jan 29, 2025
Merged

feat(server): introduces serializeError and redactError#32
reggi merged 9 commits intomainfrom
reggi/safe-errors

Conversation

@reggi
Copy link
Contributor

@reggi reggi commented Jan 23, 2025

No description provided.

@reggi reggi requested a review from a team as a code owner January 23, 2025 17:00
@reggi reggi requested a review from hashtagchris January 24, 2025 20:26
hashtagchris
hashtagchris previously approved these changes Jan 28, 2025
Copy link
Contributor

@hashtagchris hashtagchris left a comment

Choose a reason for hiding this comment

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

Looks good.

If you're going to squash, consider updating the title to describe the new functionality being exported.

@reggi reggi requested a review from hashtagchris January 29, 2025 14:45
@reggi reggi changed the title feat: breakout error handling into separate file feat(server): introduces serializeError and redactError Jan 29, 2025
Copy link
Contributor

@hashtagchris hashtagchris left a comment

Choose a reason for hiding this comment

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

Fantastic!

@reggi reggi merged commit d1837d1 into main Jan 29, 2025
3 checks passed
@reggi reggi deleted the reggi/safe-errors branch January 29, 2025 15:02
@github-actions github-actions bot mentioned this pull request Jan 29, 2025
@reggi reggi restored the reggi/safe-errors branch January 29, 2025 15:05
reggi pushed a commit that referenced this pull request Jan 29, 2025
🤖 I have created a release *beep* *boop*
---


## [3.1.0](v3.0.0...v3.1.0)
(2025-01-29)
### Features
*
[`d1837d1`](d1837d1)
[#32](#32) server: introduces
serializeError and redactError (#32) (@reggi)

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@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.

2 participants