diff --git a/.circleci/config.yml b/.circleci/config.yml index fe518d3772..0c2651a487 100644 --- a/.circleci/config.yml +++ b/.circleci/config.yml @@ -170,9 +170,9 @@ jobs: make build-cmd BUILD_NUMBER='${CIRCLE_BRANCH}-${CIRCLE_BUILD_NUM}' make package BUILD_NUMBER='${CIRCLE_BRANCH}-${CIRCLE_BUILD_NUM}' curl -sL https://sentry.io/get-cli/ | bash - export VERSION=$(sentry-cli releases propose-version) - sentry-cli --log-level=debug releases new --finalize -p mattermost-server $VERSION - sentry-cli --log-level=debug releases set-commits --auto $VERSION + echo Sentry will use version `git rev-parse HEAD` + sentry-cli --log-level=debug releases new --finalize -p mattermost-server `git rev-parse HEAD` + sentry-cli --log-level=debug releases set-commits --auto `git rev-parse HEAD` - store_artifacts: path: ~/mattermost/mattermost-server/dist/mattermost-team-linux-amd64.tar.gz diff --git a/app/server.go b/app/server.go index ab76f4922a..3ff23ec8ce 100644 --- a/app/server.go +++ b/app/server.go @@ -228,6 +228,16 @@ func NewServer(options ...Option) (*Server, error) { Dsn: SENTRY_DSN, Release: model.BuildHash, AttachStacktrace: true, + BeforeSend: func(event *sentry.Event, hint *sentry.EventHint) *sentry.Event { + // sanitize data sent to sentry to reduce exposure of PII + if event.Request != nil { + event.Request.Cookies = "" + event.Request.QueryString = "" + event.Request.Headers = nil + event.Request.Data = "" + } + return event + }, }); err != nil { mlog.Warn("Sentry could not be initiated, probably bad DSN?", mlog.Err(err)) }