sentry-rust icon indicating copy to clipboard operation
sentry-rust copied to clipboard

logentry should support logentry object based on docs

Open boxxxie opened this issue 3 years ago • 1 comments
trafficstars

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 avatar Jul 21 '22 19:07 boxxxie

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.

boxxxie avatar Jul 23 '22 00:07 boxxxie