REVERTME: debug the InvalidProof issue

polkadot/node/core/dispute-coordinator/src/initialized.rs

@@ -504,7 +504,7 @@ impl Initialized {
 					target: LOG_TARGET,
 					?session_index,
 					?candidate_hash,
-					key_ownership_proof_len = key_ownership_proof.len(),
+					?key_ownership_proof,
 					"Trying to submit a slashing report",
 				);
 

polkadot/runtime/parachains/src/disputes/slashing.rs

@@ -369,6 +369,7 @@ pub use pallet::*;
 #[frame_support::pallet]
 pub mod pallet {
 	use super::*;
+	use alloc::fmt::Debug;
 	use frame_support::pallet_prelude::*;
 	use frame_system::pallet_prelude::*;
 
@@ -377,7 +378,7 @@ pub mod pallet {
 		/// The proof of key ownership, used for validating slashing reports.
 		/// The proof must include the session index and validator count of the
 		/// session at which the offence occurred.
-		type KeyOwnerProof: Parameter + GetSessionNumber + GetValidatorCount;
+		type KeyOwnerProof: Parameter + GetSessionNumber + GetValidatorCount + Debug + Encode;
 
 		/// The identification of a key owner, used when reporting slashes.
 		type KeyOwnerIdentification: Parameter;
@@ -617,9 +618,18 @@ fn is_known_offence<T: Config>(
 	dispute_proof: &DisputeProof,
 	key_owner_proof: &T::KeyOwnerProof,
 ) -> Result<(), TransactionValidityError> {
+	use codec::Encode;
 	// check the membership proof to extract the offender's id
 	let key = (polkadot_primitives::PARACHAIN_KEY_TYPE_ID, dispute_proof.validator_id.clone());
 
+	let proof_encoded: Vec<u8> = key_owner_proof.encode();
+	log::debug!(
+		target: LOG_TARGET,
+		"Validating key ownership proof key={:?}, proof={:?}, proof_encoded={:?}",
+		key,
+		key_owner_proof,
+		proof_encoded,
+	);
 	let offender = T::KeyOwnerProofSystem::check_proof(key, key_owner_proof.clone())
 		.ok_or(InvalidTransaction::BadProof)?;
 

substrate/primitives/session/src/lib.rs

@@ -39,7 +39,7 @@ pub use runtime_api::*;
 pub type ValidatorCount = u32;
 
 /// Proof of membership of a specific key in a given session.
-#[derive(Encode, Decode, Clone, Eq, PartialEq, Default, RuntimeDebug, scale_info::TypeInfo)]
+#[derive(Encode, Decode, Clone, Eq, PartialEq, Default, Debug, scale_info::TypeInfo)]
 pub struct MembershipProof {
 	/// The session index on which the specific key is a member.
 	pub session: SessionIndex,