swarm/: Rename references of protocol handler to connection handler

[maschad]

Description

Renames old references to protocol handler to the more appropriate connection handler

Links to any relevant issues

Closes #2637

Open Questions

Change checklist

• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• A changelog entry has been made in the appropriate crates

[elenaf9]

Thanks @maschad!
I think in some places the generics are also still called TProtoHandler (e.g. in swarm/connection/HandlerWarpper). Mind renaming those as well to TConnHandler or something like that?

[maschad]

Thanks @maschad!
I think in some places the generics are also still called TProtoHandler (e.g. in swarm/connection/HandlerWarpper). Mind renaming those as well to TConnHandler or something like that?

Definitely! Will do.

[maschad]

Thanks @maschad! I think in some places the generics are also still called TProtoHandler (e.g. in swarm/connection/HandlerWarpper). Mind renaming those as well to TConnHandler or something like that?

I've updated the other requested references, happy to hear any further feedback.

[elenaf9]

Thank you for the follow up @maschad! One more nit, otherwise this is good from my side.

[mxinden]

@maschad maschad force-pushed the replace-protocol-handler branch from e2fd6f2 to 2e331f9 2 days ago

No need to force-push. We squash pull requests in the end.

[maschad]

Noted @mxinden

[mxinden]

Looks good to me. Thanks @maschad.

Just missing:

• Minor version bump over swarm-derive
• Changelog entry on swarm-derive and swarm.

[maschad]

Thanks for the review @mxinden . I have made the relevant adjustments.

[mxinden]

🙏