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.

Sign up for GitHub

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

Jump to bottom

feat(API): Set up error tracking using Sentry #4394

Merged
merged 24 commits into from
Nov 4, 2022
Merged

feat(API): Set up error tracking using Sentry #4394

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
42682ef
feat(cli): Setup error tracking using Sentry
netroy Oct 20, 2022
625a35d
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Oct 28, 2022
94b9566
make error reporting available in the workflows package
netroy Oct 28, 2022
0e428e9
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Oct 31, 2022
9c2c66f
address some of the PR comments
netroy Oct 31, 2022
a3d5f12
create a ErrorReporterProxy like LoggerProxy
netroy Nov 2, 2022
d827184
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 2, 2022
edfc6b1
remove the `captureError` helper. use ErrorReporterProxy directly
netroy Nov 2, 2022
399ef72
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 2, 2022
626ea22
fix linting issues
netroy Nov 2, 2022
aeff61d
remove ErrorReporterProxy warnings in tests
netroy Nov 2, 2022
c9ea923
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 2, 2022
ad0ceea
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 2, 2022
51207f3
check for NODE_ENV === 'production' instead
netroy Nov 2, 2022
fe076bc
IErrorReporter -> ErrorReporter
netroy Nov 2, 2022
da2cf67
ErrorReporterProxy.getInstance() -> ErrorReporter
netroy Nov 2, 2022
77a0dbd
allow capturing stacks in warnings as well
netroy Nov 2, 2022
1f6d27c
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 3, 2022
4dd816c
make n8n debugging consistent with `npm start`
netroy Nov 3, 2022
4644fc3
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 3, 2022
b611741
IReportingOptions -> ReportingOptions
netroy Nov 3, 2022
7f70ffc
use consistent signature for `error` and `warn`
netroy Nov 4, 2022
857b7bd
use Logger instead of console.log
netroy Nov 4, 2022
19d5b3a
Merge remote-tracking branch 'origin/master' into N8N-4727-setup-sentry
netroy Nov 4, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions docker/images/n8n-debian/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ ARG N8N_VERSION

RUN if [ -z "$N8N_VERSION" ] ; then echo "The N8N_VERSION argument is missing!" ; exit 1; fi

ENV N8N_VERSION=${N8N_VERSION}
RUN \
apt-get update && \
apt-get -y install graphicsmagick gosu git
Expand Down
1 change: 1 addition & 0 deletions docker/images/n8n-rhel7/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ ARG N8N_VERSION

RUN if [ -z "$N8N_VERSION" ] ; then echo "The N8N_VERSION argument is missing!" ; exit 1; fi

ENV N8N_VERSION=${N8N_VERSION}
RUN \
yum install -y gcc-c++ make

Expand Down
5 changes: 3 additions & 2 deletions docker/images/n8n/Dockerfile
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,16 @@ FROM n8nio/base:${NODE_VERSION}
ARG N8N_VERSION
RUN if [ -z "$N8N_VERSION" ] ; then echo "The N8N_VERSION argument is missing!" ; exit 1; fi

ENV N8N_VERSION=${N8N_VERSION}
ENV NODE_ENV=production
RUN set -eux; \
apkArch="$(apk --print-arch)"; \
case "$apkArch" in \
'armv7') apk --no-cache add --virtual build-dependencies python3 build-base;; \
'armv7') apk --no-cache add --virtual build-dependencies python3 build-base;; \
esac && \
npm install -g --omit=dev n8n@${N8N_VERSION} && \
case "$apkArch" in \
'armv7') apk del build-dependencies;; \
'armv7') apk del build-dependencies;; \
esac && \
find /usr/local/lib/node_modules/n8n -type f -name "*.ts" -o -name "*.js.map" -o -name "*.vue" | xargs rm && \
rm -rf /root/.npm
Expand Down
193 changes: 181 additions & 12 deletions package-lock.json
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading