Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Fix error handling on tracking operations [DEV-3527] #461

Merged
merged 4 commits into from
Jan 11, 2024

Conversation

DaevMithran
Copy link
Contributor

@DaevMithran DaevMithran commented Jan 5, 2024

The earlier approach used to throw an INTERNAL ERROR if a tracking operations fails even if the api request was successfull.

The errors are just logged silently for now.

@ankurdotb
Copy link
Contributor

Copy link
Contributor

@lampkin-diet lampkin-diet left a comment

Choose a reason for hiding this comment

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

Seems like this PR includes part of #454
which is not related to tracking resource fix.
Let's merge #454 first and continue with tracking fix after.

Copy link
Contributor

@lampkin-diet lampkin-diet left a comment

Choose a reason for hiding this comment

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

In general, removing result checking and sending 500 error looks fine

@DaevMithran DaevMithran changed the title fix: Fix error handling on tracking operations [DEV-3494] fix: Fix error handling on tracking operations [DEV-3527] Jan 11, 2024
src/controllers/credential-status.ts Outdated Show resolved Hide resolved
src/controllers/credential-status.ts Outdated Show resolved Hide resolved
src/controllers/credentials.ts Outdated Show resolved Hide resolved
src/controllers/credentials.ts Outdated Show resolved Hide resolved
@lampkin-diet lampkin-diet merged commit 53d7dfd into develop Jan 11, 2024
14 checks passed
@lampkin-diet lampkin-diet deleted the track-result branch January 11, 2024 12:58
cheqd-bot bot pushed a commit that referenced this pull request Jan 11, 2024
## [2.15.1-develop.1](2.15.0...2.15.1-develop.1) (2024-01-11)

### Bug Fixes

* Fix error handling on tracking operations [DEV-3527] ([#461](#461)) ([53d7dfd](53d7dfd))
@cheqd-bot
Copy link

cheqd-bot bot commented Jan 11, 2024

🎉 This PR is included in version 2.15.1-develop.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

lampkin-diet pushed a commit that referenced this pull request Jan 11, 2024
* fix: Fix error handling on tracking operations [DEV-3527] (#461)

* fix: Fix error handling on tracking operations

* Silently log tracking errors

* Remove duplicate return statements

---------

Co-authored-by: Andrew Nikitin <andrew.nikitin@cheqd.io>

* chore(release): 2.15.1-develop.1 [skip ci]

## [2.15.1-develop.1](2.15.0...2.15.1-develop.1) (2024-01-11)

### Bug Fixes

* Fix error handling on tracking operations [DEV-3527] ([#461](#461)) ([53d7dfd](53d7dfd))

* Simplyfing setting up user or customer from fetcher

* Fix it for account creating

---------

Co-authored-by: DaevMithran <61043607+DaevMithran@users.noreply.github.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
DaevMithran added a commit that referenced this pull request Jan 12, 2024
* feat: Add account create api

* feat: Support m2m token in auth

* feat: Add getAppScopes in auth

* Update account/create api

* Update create api flow & auth

* Add request validator

* Delegate tokens only to testnet account

* chore: Suggestion to simplifying the user or customer getting (#469)

* fix: Fix error handling on tracking operations [DEV-3527] (#461)

* fix: Fix error handling on tracking operations

* Silently log tracking errors

* Remove duplicate return statements

---------

Co-authored-by: Andrew Nikitin <andrew.nikitin@cheqd.io>

* chore(release): 2.15.1-develop.1 [skip ci]

## [2.15.1-develop.1](2.15.0...2.15.1-develop.1) (2024-01-11)

### Bug Fixes

* Fix error handling on tracking operations [DEV-3527] ([#461](#461)) ([53d7dfd](53d7dfd))

* Simplyfing setting up user or customer from fetcher

* Fix it for account creating

---------

Co-authored-by: DaevMithran <61043607+DaevMithran@users.noreply.github.com>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>

---------

Co-authored-by: Andrew Nikitin <andrew.nikitin@cheqd.io>
Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net>
cheqd-bot bot pushed a commit that referenced this pull request Jan 12, 2024
## [2.16.0-develop.1](2.15.1-develop.1...2.16.0-develop.1) (2024-01-12)

### Features

* Add M2MTokenUserInfoFetcher [DEV-3516] ([#468](#468)) ([279ec36](279ec36)), closes [#469](#469) [#461](#461)
cheqd-bot bot pushed a commit that referenced this pull request Jan 18, 2024
## [2.16.0](2.15.0...2.16.0) (2024-01-18)

### Features

* Add M2MTokenUserInfoFetcher [DEV-3516] ([#468](#468)) ([279ec36](279ec36)), closes [#469](#469) [#461](#461)

### Bug Fixes

* Fix error handling on tracking operations [DEV-3527] ([#461](#461)) ([53d7dfd](53d7dfd))
* Fix for update did [DEV-3494] ([#472](#472)) ([8a7682e](8a7682e))
* Make all the response bodies in predictable format [DEV-3494] ([#454](#454)) ([b3549d8](b3549d8))
* Update email to username in account/create ([#470](#470)) ([b19a353](b19a353))
@cheqd-bot
Copy link

cheqd-bot bot commented Jan 18, 2024

🎉 This PR is included in version 2.16.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

@cheqd-bot cheqd-bot bot added the released label Jan 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants