Right now these functions return an `Arc<...>`. But on https://gitlab.torproject.org/tpo/core/arti/-/merge_requests/347?commit_id=2f2ba8db5f45c71b2d792102914c485623beab40#note_2782761 , @diziet says: > I would usually have a method like this return `&Arc`, so that it doesn't clone. (Call sites that need a clone can make one of their own (albeit with some syntatic vinegar, unless we take !352)) But I don't know whether your call sites all (or mostly) need owned data. I think it's a good change to make, so that callers only make this change when needed. Since these methods are under `experimental-api`, this isn't a semver-breaking change.