Catch bad http headers in requests to YDB monitoring

[skip ci]

Author: jepett0

ydb/core/mon/async_http_mon.cpp

@@ -200,6 +200,11 @@ class THttpMonLegacyActorRequest : public TActorBootstrapped<THttpMonLegacyActor
         if (Event->Get()->Request->Method == "OPTIONS") {
             return ReplyOptionsAndPassAway();
         }
+        try {
+            Container.GetHeaders();
+        } catch (const yexception& exception) {
+            return ReplyBadRequestAndPassAway(exception.what());
+        }
         Become(&THttpMonLegacyActorRequest::StateFunc);
         if (ActorMonPage->Authorizer) {
             NActors::IEventHandle* handle = ActorMonPage->Authorizer(SelfId(), Container);
@@ -247,6 +252,18 @@ class THttpMonLegacyActorRequest : public TActorBootstrapped<THttpMonLegacyActor
         PassAway();
     }
 
+    void ReplyBadRequestAndPassAway(const TString& message) {
+        NHttp::THttpIncomingRequestPtr request = Event->Get()->Request;
+        TStringBuilder response;
+        response << "HTTP/1.1 400 Bad Request\r\n"
+                 << "Content-Type: text/plain\r\n"
+                 << "Connection: close\r\n"
+                 << "\r\n"
+                 << message << "\r\n";
+        ReplyWith(request->CreateResponseString(response));
+        PassAway();
+    }
+
     void ReplyUnathorizedAndPassAway(const TString& error = {}) {
         NHttp::THttpIncomingRequestPtr request = Event->Get()->Request;
         NHttp::THeaders headers(request->Headers);