server bug fix

1. delete hottier should be restricted for internal streams
2. POST /logstream/{logstream} should be restricted for query mode

Author: nikhilsinhaparseable

server/src/handlers/http/ingest.rs

@@ -186,6 +186,12 @@ pub async fn post_event(req: HttpRequest, body: Bytes) -> Result<HttpResponse, P
     if !STREAM_INFO.stream_exists(&stream_name) {
         return Err(PostError::StreamNotFound(stream_name));
     }
+
+    if CONFIG.parseable.mode == Mode::Query {
+        return Err(PostError::Invalid(anyhow::anyhow!(
+            "Ingestion is not allowed in Query mode"
+        )));
+    }
     flatten_and_push_logs(req, body, stream_name).await?;
     Ok(HttpResponse::Ok().finish())
 }

server/src/handlers/http/logstream.rs

@@ -1057,6 +1057,13 @@ pub async fn delete_stream_hot_tier(req: HttpRequest) -> Result<impl Responder,
         return Err(StreamError::HotTierNotEnabled(stream_name));
     }
 
+    if STREAM_INFO.stream_type(&stream_name).unwrap() == Some(StreamType::Internal.to_string()) {
+        return Err(StreamError::Custom {
+            msg: "Hot tier can not be deleted for internal stream".to_string(),
+            status: StatusCode::BAD_REQUEST,
+        });
+    }
+
     if let Some(hot_tier_manager) = HotTierManager::global() {
         hot_tier_manager.delete_hot_tier(&stream_name).await?;
     }