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

logentry should support logentry object based on docs #487

Open
Tracked by #488
boxxxie opened this issue Jul 21, 2022 · 1 comment
Open
Tracked by #488

logentry should support logentry object based on docs #487

boxxxie opened this issue Jul 21, 2022 · 1 comment

Comments

@boxxxie
Copy link

boxxxie commented Jul 21, 2022

relevant docs: https://develop.sentry.dev/sdk/event-payloads/types/#logentry

Environment

sentry-cli --version  
sentry-cli 2.5.0

bug example:

{
  "logentry" : {
      "message" : "some unformatted message {var}",
      "params" : {"var" : "hey there"}
  }
}

bonus bug:

{
  "logentry" : {
      "formatted" : "this errors saying 'message' is missing"
  }
}

working example:

{
  "logentry" : {
    "message" : "This is a warning: %s",
    "params" : [ "warning" ]
  }
}

Actual Result

error: invalid type: map, expected a sequence at line 4 column 18

@boxxxie
Copy link
Author

boxxxie commented Jul 23, 2022

I made a mistake on this issue when i posted, but since then i've been using the official schema to validate my payloads, and i came across another error related to logentry, so i updated to reflect the new error i found.

@kamilogorek kamilogorek transferred this issue from getsentry/sentry-cli Jul 25, 2022
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

No branches or pull requests

1 participant