Skip to content

feat(logs): add command to send logs - #2708

Closed
vgrozdanic wants to merge 3 commits into
masterfrom
vg/send-logs-command
Closed

feat(logs): add command to send logs#2708
vgrozdanic wants to merge 3 commits into
masterfrom
vg/send-logs-command

Conversation

@vgrozdanic

@vgrozdanic vgrozdanic commented Aug 13, 2025

Copy link
Copy Markdown
Member

Adds BETA command to send logs, in case we would want to change the UX around it later on, and after some feedback has been collected from users.

Part of #2584

@vgrozdanic
vgrozdanic force-pushed the vg/send-logs-command branch from 10a0716 to 30b7e5f Compare August 13, 2025 08:34
@szokeasaurusrex

Copy link
Copy Markdown
Member

@vgrozdanic are we still planning to complete this? If not, please close the PR 馃檹

@vgrozdanic

Copy link
Copy Markdown
Member Author

@vgrozdanic are we still planning to complete this? If not, please close the PR 馃檹

Yep, i would like to complete it. I'll clean it up a bit, and open a PR for review

Comment thread src/commands/logs/send.rs
}

impl LogLevel {
pub(super) fn to_severity_number(&self) -> i32 {

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vgrozdanic

Copy link
Copy Markdown
Member Author

We should use Log struct from sentry-core package instead of re-implementing it here. This requires a newer version of the package that no longer has any metrics code, so we first need to clean up all the metrics code from CLI.

Closing this PR for now, we will re-open it once the metrics are properly cleaned up and sentry-core can safely be updated

@vgrozdanic vgrozdanic closed this Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants