Skip to content

Commit

Permalink
Merge pull request #162 from rdkcentral/feature/x-subscriber-type-kurt
Browse files Browse the repository at this point in the history
Update with latest "next"
  • Loading branch information
kschrief authored Jan 9, 2024
2 parents 02204e5 + 2f16ebe commit 3009e73
Show file tree
Hide file tree
Showing 5 changed files with 33 additions and 14 deletions.
14 changes: 14 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,17 @@
## [2.1.1-next.2](https://github.com/rdkcentral/firebolt-openrpc/compare/v2.1.1-next.1...v2.1.1-next.2) (2023-11-01)


### Bug Fixes

* Errors thrown in async provider methods do not send the JSONRPC error response ([#143](https://github.com/rdkcentral/firebolt-openrpc/issues/143)) ([ec8e259](https://github.com/rdkcentral/firebolt-openrpc/commit/ec8e2599d530f9d2bd38fbb53b36e13c038edf2a))

## [2.1.1-next.1](https://github.com/rdkcentral/firebolt-openrpc/compare/v2.1.0...v2.1.1-next.1) (2023-09-14)


### Bug Fixes

* Macro not replaced for provider docs ([ef1aecd](https://github.com/rdkcentral/firebolt-openrpc/commit/ef1aecd0092a8652bc7b936604f9478b7b55df27))

# [2.1.0](https://github.com/rdkcentral/firebolt-openrpc/compare/v2.0.4...v2.1.0) (2023-09-07)


Expand Down
25 changes: 14 additions & 11 deletions languages/javascript/src/shared/ProvideManager/index.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -93,6 +93,18 @@ const provide = function(capability, provider) {
const response = {
correlationId: request.correlationId
}
let handleError = error => {
response.error = {
code: error.code || 1000, // todo: should be some reserved code for "Unknown"
message: error.message || `An error occured while calling provided ${method} method.`
}

if (error.data) {
response.error.data = JSON.parse(JSON.stringify(error.data))
}

Transport.send(module, `${method}Error`, response)
}

try {
const result = provider[method].apply(provider, providerCallArgs)
Expand All @@ -107,19 +119,10 @@ const provide = function(capability, provider) {
}

Transport.send(module, `${method}Response`, response)
})
}).catch(err => handleError(err))
}
catch(error) {
response.error = {
code: error.code || 1000, // todo: should be some reserved code for "Unknown"
message: error.message || `An error occured while calling provided ${method} method.`
}

if (error.data) {
response.error.data = JSON.parse(JSON.stringify(error.data))
}

Transport.send(module, `${method}Error`, response)
handleError(error)
}
})
})
Expand Down
4 changes: 2 additions & 2 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@firebolt-js/openrpc",
"version": "2.1.0",
"version": "2.1.1-next.2",
"description": "The Firebolt SDK Code & Doc Generator",
"main": "languages/javascript/src/sdk.mjs",
"type": "module",
Expand Down
2 changes: 2 additions & 0 deletions src/macrofier/engine.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -1512,6 +1512,8 @@ function insertProviderInterfaceMacros(template, capability, moduleJson = {}, te
}
}
methodBlock = lines.join('\n')
methodBlock = methodBlock.replace(/\$\{if\.provider\.params\}/gms, '')
methodBlock = methodBlock.replace(/\$\{end\.if\.provider\.params\}/gms, '')
}
else {
methodBlock = methodBlock.replace(/\$\{if\.provider\.params\}.*?\$\{end\.if\.provider\.params\}/gms, '')
Expand Down

0 comments on commit 3009e73

Please sign in to comment.