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.

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

refactor(repeater): send errors to sentry #458

Merged
merged 7 commits into from Oct 4, 2023

Conversation

lsndr
Copy link
Contributor

@lsndr lsndr commented Sep 21, 2023

No description provided.

@lsndr lsndr force-pushed the refactor/send-errors-to-sentry branch from 49f0696 to 0133fae Compare September 21, 2023 07:13
@lsndr lsndr self-assigned this Sep 21, 2023
@lsndr lsndr force-pushed the refactor/send-errors-to-sentry branch from 0133fae to a6e6d79 Compare September 21, 2023 07:52
@lsndr
Copy link
Contributor Author

lsndr commented Sep 21, 2023

This PR is ready to be reviewed, but not ready to be merged

src/Utils/Sentry.ts Outdated Show resolved Hide resolved
src/Config/CliBuilder.ts Show resolved Hide resolved
src/Repeater/ServerRepeaterLauncher.ts Outdated Show resolved Hide resolved
@derevnjuk derevnjuk changed the title feat(repeater): send errors to sentry refactor(repeater): send errors to sentry Sep 22, 2023
@derevnjuk derevnjuk added Type: refactoring Changes in the way the code works internally without changing the output produced. Type: technical debt Changes that remove or significantly update old unused code and/or dependencies. labels Sep 22, 2023
src/Utils/Logger.ts Outdated Show resolved Hide resolved
src/Utils/Logger.ts Outdated Show resolved Hide resolved
@lsndr lsndr force-pushed the refactor/send-errors-to-sentry branch from 91df5f2 to 026e0f2 Compare September 26, 2023 07:46
@lsndr lsndr force-pushed the refactor/send-errors-to-sentry branch from d499b9a to 54609cf Compare October 3, 2023 10:23
@lsndr lsndr changed the base branch from next to alpha October 3, 2023 10:23
@lsndr lsndr marked this pull request as ready for review October 3, 2023 10:27
@derevnjuk derevnjuk force-pushed the refactor/send-errors-to-sentry branch from 54609cf to 21ce83c Compare October 4, 2023 07:08
@derevnjuk derevnjuk force-pushed the refactor/send-errors-to-sentry branch from c793f09 to c9c640d Compare October 4, 2023 08:05
@derevnjuk derevnjuk merged commit 3723bad into alpha Oct 4, 2023
4 checks passed
@derevnjuk derevnjuk deleted the refactor/send-errors-to-sentry branch October 4, 2023 08:08
derevnjuk pushed a commit that referenced this pull request Oct 7, 2023
derevnjuk pushed a commit that referenced this pull request Oct 23, 2023
Some errors are logged in `printf`-like format, so despite we send all
logged arguments to Sentry, the captured message is too generic. At the
same time some passed logged arguments are being filtered by Sentry,
which makes issue investigation harder.

relates-to: #458
derevnjuk pushed a commit that referenced this pull request Oct 26, 2023
derevnjuk pushed a commit that referenced this pull request Oct 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: refactoring Changes in the way the code works internally without changing the output produced. Type: technical debt Changes that remove or significantly update old unused code and/or dependencies.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants