diff --git a/beacon_node/http_server/src/api.rs b/beacon_node/http_server/src/api.rs
index c89cacd9a5..11afdb9df3 100644
--- a/beacon_node/http_server/src/api.rs
+++ b/beacon_node/http_server/src/api.rs
@@ -11,6 +11,7 @@ use router::Router;
 use serde_json::json;
 use std::sync::Arc;
 
+/// Yields a handler for the HTTP API.
 pub fn build_handler<T: BeaconChainTypes + 'static>(
     beacon_chain: Arc<BeaconChain<T>>,
 ) -> impl Handler {
diff --git a/beacon_node/http_server/src/lib.rs b/beacon_node/http_server/src/lib.rs
index 8f6378e9aa..77665db8dd 100644
--- a/beacon_node/http_server/src/lib.rs
+++ b/beacon_node/http_server/src/lib.rs
@@ -87,7 +87,7 @@ pub fn start_service<T: BeaconChainTypes + 'static>(
             info!(log, "HTTP server shutting down");
 
             if let Ok(mut server) = server_start_result {
-                // According to the documentation, this function "doesn't work" and the server
+                // According to the documentation, `server.close()` "doesn't work" and the server
                 // keeps listening.
                 //
                 // It is being called anyway, because it seems like the right thing to do. If you
diff --git a/beacon_node/http_server/src/metrics.rs b/beacon_node/http_server/src/metrics.rs
index 57fa70623e..30acb88531 100644
--- a/beacon_node/http_server/src/metrics.rs
+++ b/beacon_node/http_server/src/metrics.rs
@@ -8,6 +8,7 @@ use slot_clock::SlotClock;
 use std::sync::Arc;
 use types::Slot;
 
+/// Yields a handler for the metrics endpoint.
 pub fn build_handler<T: BeaconChainTypes + 'static>(
     beacon_chain: Arc<BeaconChain<T>>,
 ) -> impl Handler {