(feat):The first working version of the addon #1
Conversation
webpack-sentry-plugin.js
Outdated
| @@ -21,6 +21,22 @@ module.exports = { | |||
| options: { pluginOptions, razzleOptions, webpackOptions }, | |||
| paths, | |||
| }) { | |||
| if ( | |||
| process.env.RAZZLE_SENTRY_AUTH_TOKEN && | |||
There was a problem hiding this comment.
As all RAZZLE_ environment variables are exposed also to the client window.env. we should consider renaming this sensitive env and not expose it to the client.
src/crashReporter.js
Outdated
| scope.setExtras({ | ||
| action, | ||
| state: store.getState(), | ||
| const crashReporter = (middlewares) => [ |
There was a problem hiding this comment.
Please use the old style, as a pure Redux middleware. Don't reference the middlewares in this module.
The reason is: we want to keep, as far as possible the "Volto domain" and "Redux domain" clean and separate, to make it easier to reason about them.
There was a problem hiding this comment.
@dobri1408 Please add README describing how to install this add-on and how to configure it via environment variables.
|
@dobri1408 You can take the existing documentation from older versions of @plone/volto and adapt it. |
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
Co-authored-by: Steve Piercy <web@stevepiercy.com>
|
@dobri1408 Have you tested the addon? Does it work as expected? |
The source maps are uploaded, but for some reason, sentry does not show correctly where the React code error occurred, so the source maps are still an issue. I am trying to understand why this is happening. I tried many ways, but none of them worked. |
This is the addon that will be used to integrate sentry in volto projects.
You have to set a .env file where you give these values, in order to make the addon work
SENTRY_URL= sentry url
AUTH_TOKEN=auth token
SENTRY_ORG=organization slug
SENTRY_PROJECT=project slug
SENTRY_RELEASE=release version
SENTRY_DSN=dsn