- Notifications
You must be signed in to change notification settings - Fork1.1k
feat(enterprise): implement organization "disable workspace sharing" option#21376
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:geokat/internal-1073-make-org-member-role-customizable-per-org
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
geokat commentedDec 22, 2025
Now I'm not sure if we want to expose It's a UX question but maybe also an authz one: as a setting (the former case) it could be easily guarded with a separate RBAC resource type if necessary (e.g. |
aslilac commentedDec 22, 2025
just as a heads up, we usually try to limit prs to about +500 lines. a little bit bigger is fine, but this is significantly over the usual limit, which will make it much harder for us to review quickly/effectively. |
geokat commentedDec 22, 2025
My bad, I should have used#21359 as the base branch because these two are stacked. Fixed. |
Uh oh!
There was an error while loading.Please reload this page.
Part of a series (2 stacked PRs) thatclosescoder/internal#1073
Adds a per-organization setting to disable workspace sharing. When enabled,
all existing workspace ACLs in the organization are cleared and workspace
sharing API endpoints return
403 Forbidden.This complements the existing site-wide
--disable-workspace-sharingflag byproviding more granular control at the organization level.
Changes
/organizations/{org}/settings/workspace-sharing/organizations/{org}/settings/workspace-sharingcoder organizations settings show workspace-sharingand
coder organizations settings set workspace-sharingDeleteWorkspaceACLsByOrganization)/workspaces/{id}/acl) return403 ForbiddenImplementation notes
organization edit --disable-workspace-sharingbut chose the settingsendpoint pattern for extensibility (e.g., future settings like default share level, allowed
share levels)
workspace_sharing_disabled