zhenyi
dca717be10
- Replace workspace_id parameter with Workspace object reference in all workspace service methods - Remove redundant find_workspace_by_id calls that were duplicated in each method - Update all method signatures across approval, audit, billing, branding, core, settings and stats modules - Modify SQL queries to bind ws.id instead of separate workspace_id parameter - Add Workspace import to all affected modules - Adjust method calls in API handlers to pass workspace object instead of id - Consolidate workspace retrieval logic to single location per operation flow
45 lines
1.8 KiB
Rust
45 lines
1.8 KiB
Rust
use actix_web::{HttpResponse, web};
|
|
|
|
use crate::api::response::{ApiErrorResponse, ApiResponse};
|
|
use crate::error::AppError;
|
|
use crate::models::workspaces::WorkspaceWebhook;
|
|
use crate::service::AppService;
|
|
use crate::service::workspace::webhooks::CreateWebhookParams;
|
|
use crate::session::Session;
|
|
|
|
#[utoipa::path(
|
|
post,
|
|
path = "/api/v1/workspaces/{workspace_name}/webhooks",
|
|
tag = "Workspaces",
|
|
operation_id = "workspaceCreateWebhook",
|
|
summary = "Create a webhook",
|
|
description = "Create a webhook for the workspace. Requires admin role. The URL must use HTTPS and cannot point to localhost or internal IPs.",
|
|
params(
|
|
("workspace_name" = String, Path, description = "Workspace name.")
|
|
),
|
|
request_body(
|
|
content = CreateWebhookParams,
|
|
description = "HTTPS webhook URL, optional secret ciphertext, event types, and active flag.",
|
|
content_type = "application/json"
|
|
),
|
|
responses(
|
|
(status = 200, description = "Webhook created.", body = ApiResponse<WorkspaceWebhook>),
|
|
(status = 400, description = "Invalid URL, missing events, or SSRF check failed.", body = ApiErrorResponse),
|
|
(status = 401, description = "Unauthenticated or insufficient role.", body = ApiErrorResponse),
|
|
(status = 500, description = "Database transaction failed.", body = ApiErrorResponse)
|
|
)
|
|
)]
|
|
pub async fn handle(
|
|
service: web::Data<AppService>,
|
|
session: Session,
|
|
path: web::Path<String>,
|
|
params: web::Json<CreateWebhookParams>,
|
|
) -> Result<HttpResponse, AppError> {
|
|
let ws = service.workspace.find_workspace_by_name(&path).await?;
|
|
let data = service
|
|
.workspace
|
|
.workspace_create_webhook(&session, &ws, params.into_inner())
|
|
.await?;
|
|
Ok(HttpResponse::Ok().json(ApiResponse::new(data)))
|
|
}
|