Files
windmill/backend/windmill-api-integration-tests/tests/permissions.rs
Ruben Fiszel 44143e7576 replace leftover common:: references in dependency_map test (#7911)
* fix: replace leftover common:: references in dependency_map test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: add missing deno_core/mcp features and gate dead code in permissions test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-11 16:05:34 +00:00

782 lines
26 KiB
Rust

#[cfg(feature = "deno_core")]
use serde_json::json;
#[cfg(feature = "deno_core")]
use sqlx::{Pool, Postgres};
#[cfg(feature = "deno_core")]
use windmill_test_utils::*;
/// Helper to create a client authenticated as a specific user
#[cfg(feature = "deno_core")]
async fn create_client_for_user(_port: u16, token: &str) -> reqwest::Client {
let mut headers = reqwest::header::HeaderMap::new();
headers.insert(
reqwest::header::AUTHORIZATION,
reqwest::header::HeaderValue::from_str(&format!("Bearer {token}")).unwrap(),
);
reqwest::ClientBuilder::new()
.default_headers(headers)
.build()
.unwrap()
}
/// Test helper to check if a GET request succeeds
#[cfg(feature = "deno_core")]
async fn can_read(client: &reqwest::Client, url: &str) -> bool {
let resp = client.get(url).send().await.unwrap();
resp.status().is_success()
}
/// Test helper to check if a POST request succeeds (for write operations)
#[cfg(feature = "deno_core")]
async fn can_write(client: &reqwest::Client, url: &str, body: serde_json::Value) -> bool {
let resp = client.post(url).json(&body).send().await.unwrap();
let status = resp.status();
// 200, 201, 204 are success, 403 is forbidden, 401 is unauthorized
status.is_success()
}
/// Comprehensive permissions test that verifies:
/// - Non-admin users can only access their user-space (u/username)
/// - Non-admin users can only access folders they have been given access to
/// - Read access when given read permission
/// - Write access when given write permission
/// - Access through direct permissions (u/username)
/// - Access through group memberships (g/groupname)
///
/// This test is ignored in CI because it requires a full Windmill setup
/// and takes longer to run. Run it locally with:
/// `cargo test --features deno_core permissions -- --ignored`
#[ignore]
#[cfg(feature = "deno_core")]
#[sqlx::test(migrations = "../migrations", fixtures("base", "permissions_test"))]
async fn test_permissions_exhaustive(db: Pool<Postgres>) -> anyhow::Result<()> {
initialize_tracing().await;
let server = ApiServer::start(db.clone()).await?;
let port = server.addr.port();
let base_url = format!("http://localhost:{}/api", port);
// Create clients for each user (tokens must be at least 10 chars)
let admin_client = create_client_for_user(port, "ADMIN_TOKEN_TEST").await;
let alice_client = create_client_for_user(port, "ALICE_TOKEN_TEST").await;
let bob_client = create_client_for_user(port, "BOB_TOKEN_TEST12").await;
let charlie_client = create_client_for_user(port, "CHARLIE_TOKEN_01").await;
// ============================================
// TEST 1: User namespace access (u/username)
// ============================================
// Alice should be able to read her own scripts
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/my_script")
)
.await,
"Alice should be able to read her own script"
);
// Alice should NOT be able to read Bob's scripts
assert!(
!can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/bob/my_script")
)
.await,
"Alice should NOT be able to read Bob's script"
);
// Bob should be able to read his own scripts
assert!(
can_read(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/bob/my_script")
)
.await,
"Bob should be able to read his own script"
);
// Bob should NOT be able to read Alice's scripts
assert!(
!can_read(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/my_script")
)
.await,
"Bob should NOT be able to read Alice's script"
);
// Admin should be able to read any script
assert!(
can_read(
&admin_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/my_script")
)
.await,
"Admin should be able to read Alice's script"
);
assert!(
can_read(
&admin_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/bob/my_script")
)
.await,
"Admin should be able to read Bob's script"
);
// ============================================
// TEST 2: Folder access without permissions
// ============================================
// Charlie has no folder permissions, should not be able to access folder items
assert!(
!can_read(
&charlie_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/shared/public_script")
)
.await,
"Charlie should NOT be able to read shared folder script without permissions"
);
// ============================================
// TEST 3: Folder read-only access
// ============================================
// Alice has read-only access to 'shared' folder
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/shared/public_script")
)
.await,
"Alice should be able to read shared folder script (read permission)"
);
// Alice should NOT be able to create/update scripts in shared folder (read-only)
let create_script_body = json!({
"path": "f/shared/alice_new_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": ""
});
assert!(
!can_write(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_script_body.clone()
)
.await,
"Alice should NOT be able to create scripts in shared folder (read-only)"
);
// ============================================
// TEST 4: Folder write access
// ============================================
// Bob has write access to 'shared' folder
assert!(
can_read(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/shared/public_script")
)
.await,
"Bob should be able to read shared folder script (write permission includes read)"
);
// Bob should be able to create scripts in shared folder (write access)
let create_script_body = json!({
"path": "f/shared/bob_new_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": "",
"schema": {}
});
assert!(
can_write(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_script_body
)
.await,
"Bob should be able to create scripts in shared folder (write permission)"
);
// ============================================
// TEST 5: Group-based folder access
// ============================================
// Charlie is in the 'developers' group which has read access to 'team' folder
// First, let's verify Charlie has no direct access
// Then verify he gets access through group membership
assert!(
can_read(
&charlie_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/team/team_script")
)
.await,
"Charlie should be able to read team folder script via group membership"
);
// Charlie's group only has read access, so he shouldn't be able to write
let create_team_script = json!({
"path": "f/team/charlie_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": "",
"schema": {}
});
assert!(
!can_write(
&charlie_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_team_script
)
.await,
"Charlie should NOT be able to create scripts in team folder (group has read-only)"
);
// ============================================
// TEST 6: Resource permissions
// ============================================
// Alice should be able to read her own resources
// Resources use /resources/get/{path} (no /p/ prefix)
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/resources/get/u/alice/my_resource")
)
.await,
"Alice should be able to read her own resource"
);
// Alice should NOT be able to read Bob's resources
assert!(
!can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/resources/get/u/bob/my_resource")
)
.await,
"Alice should NOT be able to read Bob's resource"
);
// ============================================
// TEST 7: Variable permissions
// ============================================
// Alice should be able to read her own variables
// Variables use /variables/get/{path} (no /p/ prefix)
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/variables/get/u/alice/my_variable")
)
.await,
"Alice should be able to read her own variable"
);
// Alice should NOT be able to read Bob's variables
assert!(
!can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/variables/get/u/bob/my_variable")
)
.await,
"Alice should NOT be able to read Bob's variable"
);
// ============================================
// TEST 8: Flow permissions
// ============================================
// Alice should be able to read flows in folders she has access to
// Flows use /flows/get/{path} (no /p/ prefix)
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/flows/get/f/shared/shared_flow")
)
.await,
"Alice should be able to read shared folder flow"
);
// Charlie should NOT be able to read flows in shared folder (no access)
assert!(
!can_read(
&charlie_client,
&format!("{base_url}/w/test-workspace/flows/get/f/shared/shared_flow")
)
.await,
"Charlie should NOT be able to read shared folder flow (no permission)"
);
// ============================================
// TEST 9: Direct extra_perms grants
// ============================================
// Alice has direct read permission on a specific script owned by admin
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/admin_only/shared_with_alice")
)
.await,
"Alice should be able to read script explicitly shared with her"
);
// Bob should NOT be able to read that script (not shared with him)
assert!(
!can_read(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/admin_only/shared_with_alice")
)
.await,
"Bob should NOT be able to read script only shared with Alice"
);
// ============================================
// TEST 10: Write via group membership
// ============================================
// Bob is in the 'editors' group which has write access to 'editable' folder
let create_editable_script = json!({
"path": "f/editable/bob_editor_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": "",
"schema": {}
});
assert!(
can_write(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_editable_script
)
.await,
"Bob should be able to create scripts in editable folder via group write access"
);
// ============================================
// TEST 11: Schedule permissions
// ============================================
// Schedules follow the same permission model
// Schedules use /schedules/get/{path} (no /p/ prefix)
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/schedules/get/u/alice/my_schedule")
)
.await,
"Alice should be able to read her own schedule"
);
assert!(
!can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/schedules/get/u/bob/my_schedule")
)
.await,
"Alice should NOT be able to read Bob's schedule"
);
// ============================================
// TEST 12: App permissions
// ============================================
// Alice should be able to read apps in shared folder
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/apps/get/p/f/shared/shared_app")
)
.await,
"Alice should be able to read shared folder app"
);
// Charlie should NOT be able to read apps in shared folder
assert!(
!can_read(
&charlie_client,
&format!("{base_url}/w/test-workspace/apps/get/p/f/shared/shared_app")
)
.await,
"Charlie should NOT be able to read shared folder app"
);
// ============================================
// TEST 13: Operator permissions
// ============================================
// Note: Operators can execute but have limited management permissions
// This is tested separately if needed
// ============================================
// TEST 14: Folder owner permissions
// ============================================
// Alice is an owner of the 'alice_owned' folder
assert!(
can_read(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/alice_owned/owner_script")
)
.await,
"Alice should be able to read scripts in folder she owns"
);
// Alice should be able to write to her owned folder
let create_owned_script = json!({
"path": "f/alice_owned/new_owner_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": "",
"schema": {}
});
assert!(
can_write(
&alice_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_owned_script
)
.await,
"Alice should be able to create scripts in folder she owns"
);
// ============================================
// TEST 15: Cross-user script with extra_perms
// ============================================
// Verify that giving someone read-only permission doesn't give write
// Bob has read-only permission on alice's extra_shared_script
assert!(
can_read(
&bob_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/extra_shared_script")
)
.await,
"Bob should be able to read Alice's script shared with him"
);
// But Bob should not be able to archive/delete it
let archive_resp = bob_client
.post(&format!(
"{base_url}/w/test-workspace/scripts/archive/p/u/alice/extra_shared_script"
))
.send()
.await
.unwrap();
assert!(
!archive_resp.status().is_success(),
"Bob should NOT be able to archive Alice's script (read-only permission)"
);
Ok(())
}
/// Additional test for verifying group permission inheritance
#[ignore]
#[cfg(feature = "deno_core")]
#[sqlx::test(migrations = "../migrations", fixtures("base", "permissions_test"))]
async fn test_group_permission_inheritance(db: Pool<Postgres>) -> anyhow::Result<()> {
initialize_tracing().await;
let server = ApiServer::start(db.clone()).await?;
let port = server.addr.port();
let base_url = format!("http://localhost:{}/api", port);
let charlie_client = create_client_for_user(port, "CHARLIE_TOKEN_01").await;
// Charlie is in 'developers' group
// 'developers' group has read access to 'team' folder
// Test 1: Charlie can read from team folder via group
// Scripts use /scripts/get/p/{path}
assert!(
can_read(
&charlie_client,
&format!("{base_url}/w/test-workspace/scripts/get/p/f/team/team_script")
)
.await,
"Charlie should read team folder via group"
);
// Test 2: Add Charlie to 'editors' group (which has write to 'editable')
// This should give him write access
sqlx::query!(
"INSERT INTO usr_to_group (workspace_id, group_, usr) VALUES ('test-workspace', 'editors', 'charlie')"
)
.execute(&db)
.await?;
// Note: The AUTH_CACHE in windmill_api::auth caches authentication results
// for (workspace_id, token) tuples. Since we can't easily clear it from tests,
// we use a different token or wait for cache expiry. For this test, we create
// a new token for Charlie.
sqlx::query!(
"INSERT INTO token (token, email, label, super_admin, owner, workspace_id)
VALUES ('CHARLIE_TOKEN_NEW', 'charlie@windmill.dev', 'Charlie new token', false, 'u/charlie', 'test-workspace')"
)
.execute(&db)
.await?;
// Use the new token to get fresh permissions
let charlie_client_fresh = create_client_for_user(port, "CHARLIE_TOKEN_NEW").await;
let create_script = json!({
"path": "f/editable/charlie_editor_script",
"content": "export function main() { return 'from charlie'; }",
"language": "deno",
"summary": "Charlie's editor script",
"description": "",
"schema": {}
});
assert!(
can_write(
&charlie_client_fresh,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_script
)
.await,
"Charlie should be able to write to editable folder after joining editors group"
);
Ok(())
}
/// Test that permissions work correctly for all item types
#[ignore]
#[cfg(feature = "deno_core")]
#[sqlx::test(migrations = "../migrations", fixtures("base", "permissions_test"))]
async fn test_all_item_types_permissions(db: Pool<Postgres>) -> anyhow::Result<()> {
initialize_tracing().await;
let server = ApiServer::start(db.clone()).await?;
let port = server.addr.port();
let base_url = format!("http://localhost:{}/api", port);
let alice_client = create_client_for_user(port, "ALICE_TOKEN_TEST").await;
let bob_client = create_client_for_user(port, "BOB_TOKEN_TEST12").await;
// Test Scripts - uses /scripts/get/p/{path}
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/my_script")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/scripts/get/p/u/alice/my_script")).await);
// Test Flows - uses /flows/get/{path} (no /p/)
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/flows/get/u/alice/my_flow")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/flows/get/u/alice/my_flow")).await);
// Test Resources - uses /resources/get/{path} (no /p/)
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/resources/get/u/alice/my_resource")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/resources/get/u/alice/my_resource")).await);
// Test Variables - uses /variables/get/{path} (no /p/)
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/variables/get/u/alice/my_variable")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/variables/get/u/alice/my_variable")).await);
// Test Schedules - uses /schedules/get/{path} (no /p/)
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/schedules/get/u/alice/my_schedule")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/schedules/get/u/alice/my_schedule")).await);
// Test Apps - uses /apps/get/p/{path}
assert!(can_read(&alice_client, &format!("{base_url}/w/test-workspace/apps/get/p/u/alice/my_app")).await);
assert!(!can_read(&bob_client, &format!("{base_url}/w/test-workspace/apps/get/p/u/alice/my_app")).await);
Ok(())
}
/// Test that operators cannot create or update scripts, flows, and apps
/// Operators have limited permissions - they can execute but cannot manage resources
#[ignore]
#[cfg(feature = "deno_core")]
#[sqlx::test(migrations = "../migrations", fixtures("base", "permissions_test"))]
async fn test_operator_cannot_create_update(db: Pool<Postgres>) -> anyhow::Result<()> {
initialize_tracing().await;
let server = ApiServer::start(db.clone()).await?;
let port = server.addr.port();
let base_url = format!("http://localhost:{}/api", port);
// Create operator client
let operator_client = create_client_for_user(port, "OPERATOR_TOKEN_1").await;
// ============================================
// TEST: Operator cannot create scripts
// ============================================
let create_script = json!({
"path": "u/operator/test_script",
"content": "export function main() { return 'test'; }",
"language": "deno",
"summary": "Test script",
"description": "",
"schema": {}
});
assert!(
!can_write(
&operator_client,
&format!("{base_url}/w/test-workspace/scripts/create"),
create_script
)
.await,
"Operator should NOT be able to create scripts"
);
// ============================================
// TEST: Operator cannot create flows
// ============================================
let create_flow = json!({
"path": "u/operator/test_flow",
"summary": "Test flow",
"description": "",
"value": {"modules": []},
"schema": {}
});
assert!(
!can_write(
&operator_client,
&format!("{base_url}/w/test-workspace/flows/create"),
create_flow
)
.await,
"Operator should NOT be able to create flows"
);
// ============================================
// TEST: Operator cannot create apps
// ============================================
let create_app = json!({
"path": "u/operator/test_app",
"summary": "Test app",
"value": {"grid": []},
"policy": {
"on_behalf_of": "u/operator",
"on_behalf_of_email": "operator@windmill.dev",
"execution_mode": "viewer"
}
});
assert!(
!can_write(
&operator_client,
&format!("{base_url}/w/test-workspace/apps/create"),
create_app
)
.await,
"Operator should NOT be able to create apps"
);
// ============================================
// TEST: Operator cannot archive scripts
// ============================================
// Insert a script in operator's namespace using direct DB access
sqlx::query!(
r#"INSERT INTO script (workspace_id, hash, path, content, language, kind, created_by, schema, summary, description, lock, extra_perms)
VALUES ('test-workspace', 3001, 'u/operator/existing_script', 'export function main() { return "original"; }', 'deno', 'script', 'admin', '{}', 'Existing script', '', '', '{}')"#
)
.execute(&db)
.await?;
// Operator tries to archive this script
let archive_resp = operator_client
.post(&format!(
"{base_url}/w/test-workspace/scripts/archive/p/u/operator/existing_script"
))
.send()
.await
.unwrap();
assert!(
!archive_resp.status().is_success(),
"Operator should NOT be able to archive scripts"
);
// ============================================
// TEST: Operator cannot update flows
// ============================================
// Insert a flow in operator's namespace
sqlx::query!(
r#"INSERT INTO flow (workspace_id, path, summary, description, value, edited_by, edited_at, schema, extra_perms)
VALUES ('test-workspace', 'u/operator/existing_flow', 'Existing flow', '', '{"modules": []}', 'admin', NOW(), '{}', '{}')"#
)
.execute(&db)
.await?;
let update_flow = json!({
"path": "u/operator/existing_flow",
"summary": "Updated flow",
"description": "Updated",
"value": {"modules": []},
"schema": {}
});
let update_flow_resp = operator_client
.post(&format!(
"{base_url}/w/test-workspace/flows/update/u/operator/existing_flow"
))
.json(&update_flow)
.send()
.await
.unwrap();
assert!(
!update_flow_resp.status().is_success(),
"Operator should NOT be able to update flows"
);
// ============================================
// TEST: Operator cannot update apps
// ============================================
// Insert an app in operator's namespace
sqlx::query!(
r#"INSERT INTO app (id, workspace_id, path, summary, versions, policy, extra_perms)
VALUES (3001, 'test-workspace', 'u/operator/existing_app', 'Existing app', '{}',
'{"on_behalf_of": "u/admin", "on_behalf_of_email": "admin@windmill.dev", "execution_mode": "viewer"}', '{}')"#
)
.execute(&db)
.await?;
// Create an app version
sqlx::query!(
r#"INSERT INTO app_version (id, app_id, value, created_by, created_at)
VALUES (3001, 3001, '{"grid": []}', 'admin', NOW())"#
)
.execute(&db)
.await?;
// Update app versions
sqlx::query!(
"UPDATE app SET versions = ARRAY[3001::bigint] WHERE id = 3001"
)
.execute(&db)
.await?;
let update_app = json!({
"path": "u/operator/existing_app",
"summary": "Updated app",
"value": {"grid": []},
"policy": {
"on_behalf_of": "u/operator",
"on_behalf_of_email": "operator@windmill.dev",
"execution_mode": "viewer"
}
});
let update_app_resp = operator_client
.post(&format!(
"{base_url}/w/test-workspace/apps/update/u/operator/existing_app"
))
.json(&update_app)
.send()
.await
.unwrap();
assert!(
!update_app_resp.status().is_success(),
"Operator should NOT be able to update apps"
);
Ok(())
}