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

Failing to publish integration event does not include system error response details #690

Open
austinkelleher opened this issue Apr 19, 2022 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@austinkelleher
Copy link
Contributor

austinkelleher commented Apr 19, 2022

We are currently logging an error when an integration job log event fails to publish, but we are not including the error details returned by the system.

Example error/partial stack:

err.message | Request failed with status code 400
err.name | Error
err.stack | Error: Request failed with status code 400
/opt/jupiterone/app/node_modules/@jupiterone/integration-sdk-runtime/dist/src/synchronization/events.js:14:21

level | 50
msg | Failed to publish integration event

Here is an example of where we are doing this in the project:

function getSystemErrorResponseData(

@austinkelleher austinkelleher added bug Something isn't working good first issue Good for newcomers labels Apr 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

1 participant