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

feat(*): unified error log output #601

Merged
merged 11 commits into from
Oct 29, 2024
Merged

feat(*): unified error log output #601

merged 11 commits into from
Oct 29, 2024

Conversation

aborovsky
Copy link
Contributor

Closes #600

@aborovsky aborovsky added the Type: enhancement New feature or request. label Oct 28, 2024
@aborovsky aborovsky self-assigned this Oct 28, 2024
@aborovsky aborovsky enabled auto-merge (squash) October 28, 2024 07:27
@aborovsky aborovsky changed the title feat(*): unified-error-output feat(*): unified error log output Oct 28, 2024
@aborovsky aborovsky force-pushed the feat/unified-error-output branch 2 times, most recently from 2cecd55 to b753c8d Compare October 28, 2024 07:57
.github/workflows/deploy.yml Outdated Show resolved Hide resolved
src/Utils/ErrorMessageBuilder.ts Outdated Show resolved Hide resolved
src/Utils/ErrorMessageBuilder.ts Outdated Show resolved Hide resolved
src/Commands/UploadArchive.ts Outdated Show resolved Hide resolved
src/Utils/ErrorMessageBuilder.ts Outdated Show resolved Hide resolved
@aborovsky aborovsky force-pushed the feat/unified-error-output branch from b753c8d to 5826684 Compare October 28, 2024 07:58
@derevnjuk derevnjuk disabled auto-merge October 28, 2024 08:01
@derevnjuk
Copy link
Member

@aborovsky, change the base branch to next, then re-enable auto-merge. According to Git flow, only maintenance changes may go directly to master.

@aborovsky aborovsky changed the base branch from master to next October 28, 2024 12:07
@aborovsky aborovsky enabled auto-merge (squash) October 28, 2024 12:08
@aborovsky aborovsky requested a review from derevnjuk October 28, 2024 12:38
src/Utils/ErrorMessageFactory.ts Outdated Show resolved Hide resolved
src/Utils/ErrorMessageFactory.ts Outdated Show resolved Hide resolved
src/Utils/ErrorMessageFactory.ts Outdated Show resolved Hide resolved
src/Utils/ErrorMessageFactory.ts Outdated Show resolved Hide resolved
derevnjuk
derevnjuk previously approved these changes Oct 28, 2024
Co-authored-by: Artem Derevnjuk <artem.derevnjuk@brightsec.com>
aborovsky and others added 4 commits October 29, 2024 11:35
Co-authored-by: Artem Derevnjuk <artem.derevnjuk@brightsec.com>
Co-authored-by: Artem Derevnjuk <artem.derevnjuk@brightsec.com>
@aborovsky aborovsky requested a review from derevnjuk October 29, 2024 10:37
@aborovsky aborovsky merged commit 27abbfe into next Oct 29, 2024
6 checks passed
@aborovsky aborovsky deleted the feat/unified-error-output branch October 29, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: enhancement New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Print out original error response payload on command execution malfunction
2 participants