Apply suggestions, add delete

cloudflare/src/endpoints/workers/create_secret.rs

@@ -33,5 +33,7 @@ pub struct CreateSecretParams {
     /// the variable name of the secret that will be bound to the script
     pub name: String,
     /// the string value of the secret
-    pub value: String,
+    pub text: String,
+    // type of binding (e.g.secret_text)
+    pub r#type: String,
 }

cloudflare/src/endpoints/workers/delete_secret.rs

@@ -0,0 +1,26 @@
+use super::WorkersSecret;
+
+use crate::framework::endpoint::{Endpoint, Method};
+
+/// Delete Secret
+/// https://api.cloudflare.com/#worker-delete-secret
+pub struct DeleteSecret<'a> {
+    /// account id of owner of the script
+    pub account_identifier: &'a str,
+    /// the name of the script to remove the secret from
+    pub script_name: &'a str,
+    /// the variable name of the secret
+    pub secret_name: &'a str,
+}
+
+impl<'a> Endpoint<WorkersSecret> for DeleteSecret<'a> {
+    fn method(&self) -> Method {
+        Method::Delete
+    }
+    fn path(&self) -> String {
+        format!(
+            "accounts/{}/workers/scripts/{}/secrets/{}",
+            self.account_identifier, self.script_name, self.secret_name
+        )
+    }
+}

cloudflare/src/endpoints/workers/mod.rs

@@ -1,29 +1,33 @@
 use crate::framework::response::ApiResult;
 
+use chrono::offset::Utc;
+use chrono::DateTime;
 use serde::Deserialize;
 
 mod create_route;
-pub mod create_secret;
+mod create_secret;
 mod delete_route;
-pub mod delete_secret;
+mod delete_secret;
 mod list_routes;
 
 pub use create_route::{CreateRoute, CreateRouteParams};
+pub use create_secret::{CreateSecret, CreateSecretParams};
 pub use delete_route::DeleteRoute;
+pub use delete_secret::DeleteSecret;
 pub use list_routes::ListRoutes;
 
 /// Workers KV Route
 /// Routes are basic patterns used to enable or disable workers that match requests.
 /// https://api.cloudflare.com/#worker-routes-properties
 #[derive(Deserialize, Serialize, Debug, Clone, PartialEq, Eq)]
 pub struct WorkersRoute {
-    /// Namespace identifier tag.
-    pub id: String,
-    /// The basic pattern that should map to the script
-    pub pattern: String,
-    /// Name of the script to apply when the route is matched.
-    /// The route is skipped when this is blank/missing.
-    pub script: Option<String>,
+  /// Namespace identifier tag.
+  pub id: String,
+  /// The basic pattern that should map to the script
+  pub pattern: String,
+  /// Name of the script to apply when the route is matched.
+  /// The route is skipped when this is blank/missing.
+  pub script: Option<String>,
 }
 
 impl ApiResult for WorkersRoute {}
@@ -34,8 +38,8 @@ impl ApiResult for Vec<WorkersRoute> {}
 /// but it feels wrong.
 #[derive(Deserialize, Serialize, Debug, Clone, PartialEq, Eq)]
 pub struct WorkersRouteIdOnly {
-    /// Namespace identifier tag.
-    pub id: String,
+  /// Namespace identifier tag.
+  pub id: String,
 }
 
 impl ApiResult for WorkersRouteIdOnly {}
@@ -44,10 +48,11 @@ impl ApiResult for WorkersRouteIdOnly {}
 /// https://api.cloudflare.com/#worker-secrets-properties
 #[derive(Deserialize, Serialize, Debug, Clone, PartialEq, Eq)]
 pub struct WorkersSecret {
-  /// TODO these fields depend on the API and may be wrong
+  /// TODO: these fields depend on the API and may be wrong since unable to test
   pub name: String,
-  pub modified_on: String,
-  pub script_id: String,
+  pub r#type: String,
+  pub modified_on: DateTime<Utc>,
+  pub created_on: DateTime<Utc>,
 }
 
 impl ApiResult for WorkersSecret {}