Skip to content

Propagate DbError as Internal#630

Merged
DanGould merged 1 commit intopayjoin:masterfrom
DanGould:handle-post-err
Apr 2, 2025
Merged

Propagate DbError as Internal#630
DanGould merged 1 commit intopayjoin:masterfrom
DanGould:handle-post-err

Conversation

@DanGould
Copy link
Contributor

@DanGould DanGould commented Apr 2, 2025

Errors arising from a validated request being pushed to DB are internal 500 errors in nature.

Errors arising from a validated request being pushed to DB are
internal 500 errors in nature.
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 14228238949

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 81.608%

Changes Missing Coverage Covered Lines Changed/Added Lines %
payjoin-directory/src/lib.rs 0 1 0.0%
Totals Coverage Status
Change from base Build 14226716986: 0.0%
Covered Lines: 5258
Relevant Lines: 6443

💛 - Coveralls

Copy link
Collaborator

@arminsabouri arminsabouri left a comment

Choose a reason for hiding this comment

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

Ack bd89e57

@DanGould DanGould merged commit fde867b into payjoin:master Apr 2, 2025
7 checks passed
@DanGould DanGould deleted the handle-post-err branch April 2, 2025 20:17
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