Rust: flatbuffers PlainTransport

[jmillan]

The following two points I'll handle in a different PR once this is merged.

• Use Protocol type rather than convert from/to string.
• Create a SrtpCrytoSuite type in flatbuffers to avoid converting from/to string.

[ibc]

CI failing but LGTM

[jmillan]

I'll create this alias for every element in the enum in another PR.

[nazar-pc]

Things like Clippy suggestions should ideally be squashed with commits where these complaints originate (same for TS) rather than as a separate commits before opening PR, makes it easier to review.

Also there are a lot of things remaining to be done, some of which are in your or my head and some in PR comments. They should all ideally be TODO comments right in the code, such that anyone can come later and do refactoring/cleanup and resolve those TODOs.

[nazar-pc]

Multiple comments as a review, please 🙏 Too many notifications.

[jmillan]

Multiple comments as a review, please 🙏 Too many notifications.

I'll squash all commits of this branch into a single one for clarity and force push, ok?

[nazar-pc]

I don't think I have a strong preference, but maybe do that once you're done with changes, so I can review the diff rather than re-review everything again?

[jmillan]

but maybe do that once you're done with changes, so I can review the diff rather than re-review everything again?

Yep, all concerns have been handled.

[nazar-pc]

Hm... so you have force pushed a few times anyway 😕

[jmillan]

Hm... so you have force pushed a few times anyway 😕

I know I know.., I have my hands out of the keyword.., promise 🙏

[nazar-pc]

I didn't re-read the whole thing again, but looks fine