From c1dc552b1cbd190d127444dd728d1071e238f6a5 Mon Sep 17 00:00:00 2001 From: Ethan Donowitz <8703826+ethowitz@users.noreply.github.com> Date: Fri, 5 Nov 2021 11:52:09 -0400 Subject: [PATCH] docs: add comments about Tokenserver state being an Option (#1161) Closes #1102 --- src/tokenserver/extractors.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/tokenserver/extractors.rs b/src/tokenserver/extractors.rs index 3e0d1c83..01f59d92 100644 --- a/src/tokenserver/extractors.rs +++ b/src/tokenserver/extractors.rs @@ -134,6 +134,9 @@ impl FromRequest for TokenserverRequest { Box::pin(async move { let token_data = TokenData::extract(&req).await?; + + // XXX: Tokenserver state will no longer be an Option once the Tokenserver + // code is rolled out, so we will eventually be able to remove this unwrap(). let state = get_server_state(&req)?.as_ref().as_ref().unwrap(); let shared_secret = get_secret(&req)?; let fxa_metrics_hash_secret = &state.fxa_metrics_hash_secret.as_bytes(); @@ -249,6 +252,8 @@ impl FromRequest for Box { let req = req.clone(); Box::pin(async move { + // XXX: Tokenserver state will no longer be an Option once the Tokenserver + // code is rolled out, so we will eventually be able to remove this unwrap(). let state = get_server_state(&req)?.as_ref().as_ref().unwrap(); let db = state.db_pool.get().map_err(|_| { error!("⚠️ Could not acquire database connection");