Support Policies
Nobody has claimed this yet.
Assessment
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Newbie friendliness
- 42/100
- Issue type
- Feature
- Clarity
- Needs clarification
- Activity status
- Quiet
- Tech stack
- typescript, vscode
- Domain
- devtools
Research direction
Start by comparing VS Code's policy-enabled configuration service in src/vs/code/electron-main/main.ts with the VS Code Server setup in src/vs/server/node/serverServices.ts, especially the NullPolicyService. Trace how code-server incorporates the server configuration, then verify that policies from the expected Linux JSON location are applied in code-server.
Written by the indexing model from the issue text.
Description
Reposting this for visibility, as I'm not sure anyone will see a comment on a closed issue and I haven't heard anything for a week.
The upstream issue was resolved, but unfortunately VSCode disables policies (the partial solution to the problem) for VSCode Web. Would it be reasonable to make policies apply to code-server?
https://github.com/microsoft/vscode/issues/27972 is still open, but it would be nice to at least get the policy stuff over to code-server.
While I was trying to hack around this locally my AI coding assistant narrowed down to what it thinks the problem is (and which seems like a reasonable guess), which is that the
ConfigurationServicethat is instantiated for the electron app includes apolicyService
https://github.com/microsoft/vscode/blob/main/src/vs/code/electron-main/main.ts#L226
But the configuration service that is instantiated for VSCode Server uses aNullPolicyService.
https://github.com/microsoft/vscode/blob/main/src/vs/server/node/serverServices.ts#L145Of note, the policy service on Linux is just a JSON file in a particular location, which I think would all that would be needed here.
In theory, the only patch necessary to fix this for code-server would be to change VSCode Server to instantiate a policyService and use it. I suspect that reality is quite different from theory, as always, but this feels like it has potential to be a low-lift improvement.
Originally posted by @MicahZoltu in #6995
- Dominant language
- TypeScript
- Stars
- 79.4k
- Forks
- 6.9k
- Avg merge
- 2d 13h
- Merged PRs (30d)
- 39
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from coder/code-server
-
Difficulty 1/5 Under an hour Newbie friendliness 92/100
coder/code-server#8017 · 2 comments ·
-
security
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
coder/code-server#8013 · 4 comments ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
coder/code-server#7976 · 2 comments ·
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 35/100
coder/code-server#7962 · 3 comments ·
-
bug needs-investigation
Difficulty 4/5 3-5 days Newbie friendliness 55/100
coder/code-server#7955 · 1 comment ·
All issues in coder/code-server
Similar issues
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Open
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
-
Difficulty 1/5 Under an hour Newbie friendliness 85/100
-
Mend: dependency security vulnerability untriaged
Difficulty 2/5 1-3 hours Newbie friendliness 70/100