tailcallhq · tusharmath · Feb 5, 2024 · Jan 23, 2024 · Jan 23, 2024 · Jan 23, 2024
diff --git a/cloudflare/src/file.rs b/cloudflare/src/file.rs
@@ -22,7 +22,9 @@ impl CloudflareFileIO {
     }
 }
 
-// TODO: avoid the unsafe impl
+// Multi-threading is not enabled in Cloudflare,
+// so this doesn't matter, and makes API compliance
+// way easier.
 unsafe impl Sync for CloudflareFileIO {}
 unsafe impl Send for CloudflareFileIO {}
 

diff --git a/cloudflare/src/handle.rs b/cloudflare/src/handle.rs
@@ -1,15 +1,12 @@
+use std::borrow::Borrow;
 use std::collections::HashMap;
 use std::rc::Rc;
 use std::sync::{Arc, RwLock};
 
-use anyhow::anyhow;
+use hyper::{Body, Method, Request, Response};
 use lazy_static::lazy_static;
 use tailcall::async_graphql_hyper::GraphQLRequest;
-use tailcall::blueprint::Blueprint;
-use tailcall::config::reader::ConfigReader;
-use tailcall::config::ConfigSet;
-use tailcall::http::{graphiql, handle_request, AppContext};
-use tailcall::target_runtime::TargetRuntime;
+use tailcall::http::{graphiql, handle_request, showcase, AppContext};
 
 use crate::http::{to_request, to_response};
 use crate::init_runtime;
@@ -20,67 +17,71 @@ lazy_static! {
 ///
 /// The handler which handles requests on cloudflare
 ///
-pub async fn fetch(req: worker::Request, env: worker::Env) -> anyhow::Result<worker::Response> {
+pub async fn fetch(
+    req: worker::Request,
+    env: worker::Env,
+    _: worker::Context,
+) -> anyhow::Result<worker::Response> {
     log::info!(
         "{} {:?}",
         req.method().to_string(),
         req.url().map(|u| u.to_string())
     );
-    let env = Rc::new(env);
-    let hyper_req = to_request(req).await?;
-    if hyper_req.method() == hyper::Method::GET {
-        let response = graphiql(&hyper_req)?;
-        return to_response(response).await;
-    }
-    let query = hyper_req
-        .uri()
-        .query()
-        .ok_or(anyhow!("Unable parse extract query"))?;
-    let query = serde_qs::from_str::<HashMap<String, String>>(query)?;
-    let config_path = query
-        .get("config")
-        .ok_or(anyhow!("The key 'config' not found in the query"))?
-        .clone();
+    let req = to_request(req).await?;
 
-    log::info!("config-url: {}", config_path);
-    let app_ctx = get_app_ctx(env, config_path.as_str()).await?;
-    let resp = handle_request::<GraphQLRequest>(hyper_req, app_ctx).await?;
+    // Quick exit to GraphiQL
+    //
+    // Has to be done here, since when using GraphiQL, a config query parameter is not specified,
+    // and get_app_ctx will fail without it.
+    if req.method() == Method::GET {
+        return to_response(graphiql(&req)?).await;
+    }
 
+    let env = Rc::new(env);
+    let app_ctx = match get_app_ctx(env, &req).await? {
+        Ok(app_ctx) => app_ctx,
+        Err(e) => return to_response(e).await,
+    };
+    let resp = handle_request::<GraphQLRequest>(req, app_ctx).await?;
     to_response(resp).await
 }
 
-///
-/// Reads the configuration from the CONFIG environment variable.
-///
-async fn get_config(runtime: TargetRuntime, file_path: &str) -> anyhow::Result<ConfigSet> {
-    let reader = ConfigReader::init(runtime);
-    let config = reader.read(&file_path).await?;
-    Ok(config)
-}
-
 ///
 /// Initializes the worker once and caches the app context
 /// for future requests.
 ///
-async fn get_app_ctx(env: Rc<worker::Env>, file_path: &str) -> anyhow::Result<Arc<AppContext>> {
+async fn get_app_ctx(
+    env: Rc<worker::Env>,
+    req: &Request<Body>,
+) -> anyhow::Result<Result<Arc<AppContext>, Response<Body>>> {
     // Read context from cache
-    if let Some(app_ctx) = read_app_ctx() {
-        if app_ctx.0 == file_path {
-            log::info!("Using cached application context");
-            return Ok(app_ctx.clone().1);
+    let file_path = req
+        .uri()
+        .query()
+        .and_then(|x| serde_qs::from_str::<HashMap<String, String>>(x).ok())
+        .and_then(|x| x.get("config").cloned());
+
+    if let Some(file_path) = &file_path {
+        if let Some(app_ctx) = read_app_ctx() {
+            if app_ctx.0 == file_path.borrow() {
+                log::info!("Using cached application context");
+                return Ok(Ok(app_ctx.clone().1));
+            }
         }
     }
-    // Create new context
+
     let runtime = init_runtime(env)?;
-    let cfg = get_config(runtime.clone(), file_path).await?;
-    log::info!("Configuration read ... ok");
-    log::debug!("\n{}", cfg.to_sdl());
-    let blueprint = Blueprint::try_from(&cfg)?;
-    log::info!("Blueprint generated ... ok");
-    let app_ctx = Arc::new(AppContext::new(blueprint, runtime));
-    *APP_CTX.write().unwrap() = Some((file_path.to_string(), app_ctx.clone()));
-    log::info!("Initialized new application context");
-    Ok(app_ctx)
+    match showcase::create_app_ctx::<GraphQLRequest>(req, runtime, true).await? {
+        Ok(app_ctx) => {
+            let app_ctx: Arc<AppContext> = Arc::new(app_ctx);
+            if let Some(file_path) = file_path {
+                *APP_CTX.write().unwrap() = Some((file_path, app_ctx.clone()));
+            }
+            log::info!("Initialized new application context");
+            Ok(Ok(app_ctx))
+        }
+        Err(e) => Ok(Err(e)),
+    }
 }
 
 fn read_app_ctx() -> Option<(String, Arc<AppContext>)> {

diff --git a/cloudflare/src/lib.rs b/cloudflare/src/lib.rs
@@ -18,7 +18,6 @@ pub fn init_env(env: Rc<worker::Env>) -> Arc<dyn EnvIO> {
 }
 
 pub fn init_file(env: Rc<worker::Env>, bucket_id: String) -> anyhow::Result<Arc<dyn FileIO>> {
-    #[allow(clippy::arc_with_non_send_sync)]
     Ok(Arc::new(file::CloudflareFileIO::init(env, bucket_id)?))
 }
 
@@ -36,6 +35,9 @@ pub fn init_runtime(env: Rc<worker::Env>) -> anyhow::Result<TargetRuntime> {
     let bucket_id = env_io
         .get("BUCKET")
         .ok_or(anyhow!("BUCKET var is not set"))?;
+
+    log::debug!("R2 Bucket ID: {}", bucket_id);
+
     Ok(TargetRuntime {
         http: http.clone(),
         http2_only: http.clone(),
@@ -49,9 +51,9 @@ pub fn init_runtime(env: Rc<worker::Env>) -> anyhow::Result<TargetRuntime> {
 async fn fetch(
     req: worker::Request,
     env: worker::Env,
-    _: worker::Context,
+    ctx: worker::Context,
 ) -> anyhow::Result<worker::Response> {
-    let result = handle::fetch(req, env).await;
+    let result = handle::fetch(req, env, ctx).await;
 
     match result {
         Ok(response) => Ok(response),

diff --git a/generated/.tailcallrc.graphql b/generated/.tailcallrc.graphql
@@ -272,6 +272,10 @@ directive @server(
   """
   script: Script
   """
+  `showcase` enables the /showcase/graphql endpoint.
+  """
+  showcase: Boolean
+  """
   This configuration defines local variables for server operations. Useful for storing 
   constant configurations, secrets, or shared information.
   """

diff --git a/generated/.tailcallrc.schema.json b/generated/.tailcallrc.schema.json
@@ -1419,6 +1419,13 @@
           ],
           "description": "A link to an external JS file that listens on every HTTP request response event."
         },
+        "showcase": {
+          "description": "`showcase` enables the /showcase/graphql endpoint.",
+          "type": [
+            "boolean",
+            "null"
+          ]
+        },
         "vars": {
           "allOf": [
             {

diff --git a/src/blueprint/server.rs b/src/blueprint/server.rs
@@ -18,6 +18,7 @@ pub struct Server {
     pub enable_query_validation: bool,
     pub enable_response_validation: bool,
     pub enable_batch_requests: bool,
+    pub enable_showcase: bool,
     pub global_response_timeout: i64,
     pub worker: usize,
     pub port: u16,
@@ -100,6 +101,7 @@ impl TryFrom<crate::config::Server> for Server {
                 enable_query_validation: (config_server).enable_query_validation(),
                 enable_response_validation: (config_server).enable_http_validation(),
                 enable_batch_requests: (config_server).enable_batch_requests(),
+                enable_showcase: (config_server).enable_showcase(),
                 global_response_timeout: (config_server).get_global_response_timeout(),
                 http,
                 worker: (config_server).get_workers(),

diff --git a/src/cli/javascript/mod.rs b/src/cli/javascript/mod.rs
@@ -9,7 +9,7 @@ pub use runtime::Runtime;
 
 use crate::{blueprint, HttpIO};
 
-pub fn init_http(http: impl HttpIO, script: blueprint::Script) -> Arc<dyn HttpIO> {
+pub fn init_http(http: impl HttpIO, script: blueprint::Script) -> Arc<dyn HttpIO + Sync + Send> {
     let script_io = Runtime::new(script);
     Arc::new(HttpFilter::new(http, script_io))
 }
diff --git a/src/cli/mod.rs b/src/cli/mod.rs
@@ -30,7 +30,7 @@ pub fn init_env() -> Arc<dyn EnvIO> {
 }
 
 // Provides access to file system in native rust environment
-pub fn init_file() -> Arc<dyn FileIO> {
+pub fn init_file() -> Arc<dyn FileIO + Send + Sync> {
     Arc::new(file::NativeFileIO::init())
 }
 

diff --git a/src/config/server.rs b/src/config/server.rs
@@ -34,6 +34,9 @@ pub struct Server {
     /// `globalResponseTimeout` sets the maximum query duration before termination, acting as a safeguard against long-running queries.
     pub global_response_timeout: Option<i64>,
     #[serde(default, skip_serializing_if = "is_default")]
+    /// `showcase` enables the /showcase/graphql endpoint.
+    pub showcase: Option<bool>,
+    #[serde(default, skip_serializing_if = "is_default")]
     /// `workers` sets the number of worker threads. @default the number of system cores.
     pub workers: Option<usize>,
     #[serde(default, skip_serializing_if = "is_default")]
@@ -118,6 +121,9 @@ impl Server {
     pub fn enable_batch_requests(&self) -> bool {
         self.batch_requests.unwrap_or(false)
     }
+    pub fn enable_showcase(&self) -> bool {
+        self.showcase.unwrap_or(false)
+    }
 
     pub fn get_hostname(&self) -> String {
         self.hostname.clone().unwrap_or("127.0.0.1".to_string())
@@ -150,6 +156,7 @@ impl Server {
         self.global_response_timeout = other
             .global_response_timeout
             .or(self.global_response_timeout);
+        self.showcase = other.showcase.or(self.showcase);
         self.workers = other.workers.or(self.workers);
         self.port = other.port.or(self.port);
         self.hostname = other.hostname.or(self.hostname);

diff --git a/src/http/mod.rs b/src/http/mod.rs
@@ -7,6 +7,7 @@ mod request_context;
 mod request_handler;
 mod request_template;
 mod response;
+pub mod showcase;
 
 pub use cache::*;
 pub use data_loader::*;

diff --git a/src/http/request_context.rs b/src/http/request_context.rs
@@ -98,20 +98,20 @@ impl RequestContext {
 }
 
 impl From<&AppContext> for RequestContext {
-    fn from(server_ctx: &AppContext) -> Self {
+    fn from(app_ctx: &AppContext) -> Self {
         Self {
-            h_client: server_ctx.runtime.http.clone(),
-            h2_client: server_ctx.runtime.http2_only.clone(),
-            server: server_ctx.blueprint.server.clone(),
-            upstream: server_ctx.blueprint.upstream.clone(),
+            h_client: app_ctx.runtime.http.clone(),
+            h2_client: app_ctx.runtime.http2_only.clone(),
+            server: app_ctx.blueprint.server.clone(),
+            upstream: app_ctx.blueprint.upstream.clone(),
             req_headers: HeaderMap::new(),
-            http_data_loaders: server_ctx.http_data_loaders.clone(),
-            gql_data_loaders: server_ctx.gql_data_loaders.clone(),
-            cache: server_ctx.runtime.cache.clone(),
-            grpc_data_loaders: server_ctx.grpc_data_loaders.clone(),
+            http_data_loaders: app_ctx.http_data_loaders.clone(),
+            gql_data_loaders: app_ctx.gql_data_loaders.clone(),
+            cache: app_ctx.runtime.cache.clone(),
+            grpc_data_loaders: app_ctx.grpc_data_loaders.clone(),
             min_max_age: Arc::new(Mutex::new(None)),
             cache_public: Arc::new(Mutex::new(None)),
-            env_vars: server_ctx.runtime.env.clone(),
+            env_vars: app_ctx.runtime.env.clone(),
         }
     }
 }