search

arabold / serverless-sentry-plugin

This plugin adds automatic forwarding of errors and exceptions to Sentry (https://sentry.io) and Serverless (https://serverless.com)
MIT License
155 stars 27 forks source link

replace String with JSON.stringify #68

Open fancywriter opened 2 years ago

fancywriter commented 2 years ago

This is clearly a bug, because when I have value of sourceMaps in my config like

sourceMaps:
  urlPrefix: /var/task

It's serialized like [object Object] and that's what I see on environment variable in lambda configuration. I am not sure what this variables is needed for (is it needed at all?), but definitely it shouldn't look like that.

Could it be that for other cases here we should replace String(...) with JSON.stringify(...)?

image

fancywriter commented 2 years ago

@arabold hey! did you have a chance to review it? perhaps it makes sense to add JSON.stringify() in other places too?