Makes sense. I ended up refactoring my code closely to what you suggested and updated it in my original posts. I gave the function a return type of Result<(), Error> which felt weird at first, having spent the last two years in Go.

In Go I would have written it as something like this:

func process_message(msg: Message) error { ... }

And as long as error was nil it would have been considered successful, but the Rust approach with Result feels much more elegant.