Skip to content
This repository has been archived by the owner on Dec 4, 2018. It is now read-only.

Missing functionality or documentation for app type feature #126

Closed
CTOJoe opened this issue Jan 17, 2018 · 1 comment
Closed

Missing functionality or documentation for app type feature #126

CTOJoe opened this issue Jan 17, 2018 · 1 comment

Comments

@CTOJoe
Copy link

CTOJoe commented Jan 17, 2018

Expected behavior

Ability to configure app type like other platforms. ( PHP | GO | JAVA | RUBY )

Observed behavior

Unable to find in source or bugsnag documentation on how to configure it.

@bengourley
Copy link
Contributor

Thanks for the report, we should add this. In the mean time you can add a before notify callback to achieve the desired result:

bugsnag.onBeforeNotify(function (notification, originalError) {
  if (!notification.events[0].app) {
    notification.events[0].app = {}
  }
  notification.events[0].app.type = 'worker' // etc.
})

bengourley added a commit that referenced this issue Mar 15, 2018
The error reporting payload supports app.type but this notifier didn't provide a way to set it. This
change allows the app type setting to be set with the config option "appType".

Fixes #126.
bengourley added a commit that referenced this issue Mar 15, 2018
The error reporting payload supports app.type but this notifier didn't provide a way to set it. This
change allows the app type setting to be set with the config option "appType".

Fixes #126.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants