Fix logging of ADNL key activation on QUIC server

[cleverfox]

No description provided.

[Copilot]

Pull request overview

Adjusts QUIC key activation behavior to avoid logging “Activated” when the activation doesn’t actually change the server’s SNI fallback identity.

Changes:

• Track whether last_added_name actually changes during activate_key_inner.
• Emit the “Activated QUIC identity …” info log only when the fallback identity is updated.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

If endpoint_state.last_added_name mutex is poisoned, changed becomes false and the activation request is silently ignored (and no log is emitted). Consider logging a warn/error in the Err(_) branch, and optionally recovering via PoisonError::into_inner() so the fallback identity can still be updated after a panic.

Suggested change

	let changed = if let Ok(mut last) = endpoint_state.last_added_name.lock() {
	if last.as_deref() == Some(name.as_str()) {
	false
	} else {
	*last = Some(name);
	true
	}
	} else {
	false
	let changed = match endpoint_state.last_added_name.lock() {
	Ok(mut last) => {
	if last.as_deref() == Some(name.as_str()) {
	false
	} else {
	*last = Some(name);
	true
	}
	}
	Err(poisoned) => {
	log::warn!(
	target: TARGET,
	"activate_key: last_added_name lock poisoned for key {} on port {}, recovering",
	key_id,
	port
	);
	let mut last = poisoned.into_inner();
	if last.as_deref() == Some(name.as_str()) {
	false
	} else {
	*last = Some(name);
	true
	}
	}