IBC core validate handler uses mutable reference

[rnbguy]

Bug Summary

The core validate handler uses a mutable reference, even though the mutability is not needed. This also adds risks of mistakenly calling a mutable method on it.

Details

&mut impl Router should be refactored to &impl Router

This is also detectable by running cargo +nightly clippy.

Version

ibc-rs/crates/ibc/src/core/handler.rs

Lines 54 to 58 in 133863d

	pub fn validate<Ctx>(
	ctx: &Ctx,
	router: &mut impl Router,
	msg: MsgEnvelope,
	) -> Result<(), RouterError>