Commit 77051cc
authored
fix(security): remove pre-allowed messaging from base sandbox policy (#1705)
## Summary
- Removed Telegram (`api.telegram.org`) and Discord (`discord.com`,
`gateway.discord.gg`, `cdn.discordapp.com`) from the base sandbox
network policy
- These endpoints gave every sandboxed agent default access to external
messaging APIs without user opt-in — a data exfiltration vector
- Users who configure messaging tokens during onboarding already get
connectivity via the existing preset system (`presets/telegram.yaml`,
`presets/discord.yaml`), which is applied automatically when tokens are
detected
## Test plan
- [x] Full vitest suite passes (1243 tests, 0 failures)
- [x] Base policy YAML parses correctly
- [x] No telegram/discord references remain in base policy
- [ ] E2E: verify messaging preset auto-application when tokens are
present
<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit
* **Chores**
* Removed Telegram API endpoint from baseline network allow-list
policies
* Removed Discord API endpoints (REST and WebSocket gateway) from
sandbox network policies
* Updated network policy documentation to reflect removed endpoint
groups
<!-- end of auto-generated comment: release notes by coderabbit.ai -->
Signed-off-by: Aaron Erickson <aerickson@nvidia.com>1 parent 723f814 commit 77051cc
3 files changed
Lines changed: 0 additions & 56 deletions
File tree
- .agents/skills/nemoclaw-user-reference/references
- docs/reference
- nemoclaw-blueprint/policies
Lines changed: 0 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
73 | 73 | | |
74 | 74 | | |
75 | 75 | | |
76 | | - | |
77 | | - | |
78 | | - | |
79 | | - | |
80 | | - | |
81 | 76 | | |
82 | 77 | | |
83 | 78 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | 96 | | |
102 | 97 | | |
103 | 98 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
207 | 207 | | |
208 | 208 | | |
209 | 209 | | |
210 | | - | |
211 | | - | |
212 | | - | |
213 | | - | |
214 | | - | |
215 | | - | |
216 | | - | |
217 | | - | |
218 | | - | |
219 | | - | |
220 | | - | |
221 | | - | |
222 | | - | |
223 | | - | |
224 | | - | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | | - | |
230 | | - | |
231 | | - | |
232 | | - | |
233 | | - | |
234 | | - | |
235 | | - | |
236 | | - | |
237 | | - | |
238 | | - | |
239 | | - | |
240 | | - | |
241 | | - | |
242 | | - | |
243 | | - | |
244 | | - | |
245 | | - | |
246 | | - | |
247 | | - | |
248 | | - | |
249 | | - | |
250 | | - | |
251 | | - | |
252 | | - | |
253 | | - | |
254 | | - | |
255 | | - | |
0 commit comments