feat(preset): add PATCH method to Outlook preset for Graph API updates#1478
feat(preset): add PATCH method to Outlook preset for Graph API updates#1478stevenobiajulu wants to merge 1 commit intoNVIDIA:mainfrom
Conversation
Microsoft Graph API uses PATCH for standard email and calendar operations: marking messages as read, flagging for follow-up, and updating drafts. The Outlook preset currently only allows GET and POST on graph.microsoft.com, blocking these workflows inside sandboxes. Add PATCH rule to graph.microsoft.com and a regression test. Fixes NVIDIA#1477 Signed-off-by: Steven Obiajulu <steven@usejunior.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAdded a PATCH method rule to the Outlook network policy preset for Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
✨ Thanks for submitting this fix, which proposes a way to add PATCH method support to the Outlook network policy preset for Microsoft Graph API operations. Possibly related open issues: |
Summary
Add PATCH method support to the
graph.microsoft.comendpoint in the Outlook network policy preset.Microsoft Graph API uses PATCH for standard email and calendar operations — marking messages as read, flagging for follow-up, and updating drafts. The current preset only allows GET and POST, which blocks these workflows inside sandboxes.
Fixes #1477
Changes
nemoclaw-blueprint/policies/presets/outlook.yaml: Addallow: { method: PATCH, path: "/**" }tograph.microsoft.com(1 line)test/policies.test.js: Add regression test asserting PATCH is allowed on graph.microsoft.com in the Outlook preset (8 lines)Scope
graph.microsoft.com— the other endpoints in the preset do not need PATCHTesting
npx vitest run)Summary by CodeRabbit
New Features
Tests