I don’t know Rust, but trying to hack on Lemmy 0.18.1 enough to get a better error message out.

error: data did not match any variant of untagged enum AnnouncableActivities

where: crates/apub/src/activities/community/announce.rs, line: 46

https://github.com/LemmyNet/lemmy/blob/0c82f4e66065b5772fede010a879d327135dbb1e/crates/apub/src/activities/community/announce.rs#L46

That seems to be the function parameters themselves?

Is the error caused by RawAnnouncableActivities not matching the enum AnnouncableActivities and the try_into?

  warn!("zebratrace receive {:?}", self);

Works for adding logging, but I’d like the code to log self only when the enum does not match (errors). Thank you.

  • colonial@lemmy.worldEnglish
    5·
    1 year ago

    Your analysis sounds correct - data did not match any variant of untagged enum is definitely from serde IIRC - but I can’t say for certain just by looking at the code.

    but I’d like the code to log self only when the enum does not match (errors). Thank you.

    Try changing line 48 to this:

    let object = match self.clone().try_into::<AnnouncableActivities>() {
    Ok(object) => object,
    Err(e) => {
        warn!("zebratrace receive {:?}", self);
        return Err(e);
    }
}

    (This probably isn’t the best way to do this, but it’s what I came up with off the top of my head :P)

    • anlumo@feddit.deEnglish
      2·
      1 year ago

      Serde handles untagged enums in the way that it tries to deserialize every variant of that enum from top to bottom with the given data, and returns the first one that doesn’t throw an error. The error indicates that all of them threw an error when trying to deserialize the data.

    • RoundSparrow@lemmy.mlOPEnglish
      1·
      1 year ago

      let object = match self.clone().try_into::<AnnouncableActivities>() { Ok(object) => object, Err(e) => { warn!(“zebratrace receive {:?}”, self); return Err(e); } }

      Compiler didn’t like your code:

      let object = match self.clone().try_into::<AnnouncableActivities>() {
   |                                     ^^^^^^^^ expected 0 generic arguments
   |
help: consider moving this generic argument to the `TryInto` trait, which takes up to 1 argument
   |
52 |     let object = match TryInto::<AnnouncableActivities>::try_into(self.clone()) {
   |                        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
help: remove these generics
   |
52 -     let object = match self.clone().try_into::<AnnouncableActivities>() {
52 +     let object = match self.clone().try_into() {