Skip to content

Conversation

@Anthony-Breneliere
Copy link

  • Fix CosmosDB database name mismatch (forms-db → formsdb) to match app settings
  • Add missing AzureWebJobsStorage__credential for managed identity auth in submission function
  • Add Storage Queue Data Contributor role assignments for both functions
  • Add Storage Table Data Contributor role assignments for both functions
  • Add minTlsVersion and http20Enabled settings to notification function

- Fix CosmosDB database name mismatch (forms-db → formsdb) to match app settings
- Add missing AzureWebJobsStorage__credential for managed identity auth in submission function
- Add Storage Queue Data Contributor role assignments for both functions
- Add Storage Table Data Contributor role assignments for both functions
- Add minTlsVersion and http20Enabled settings to notification function

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings January 19, 2026 21:10
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR corrects several Azure Function deployment configuration issues to ensure proper managed identity authentication and consistent security settings across both functions.

Changes:

  • Fixed CosmosDB database name from 'forms-db' to 'formsdb' to match application settings references
  • Added missing managed identity authentication credential for submission function's storage access
  • Added RBAC role assignments for Storage Queue and Table Data Contributor to enable managed identity access for both functions
  • Added TLS and HTTP/2 security settings to notification function for consistency with submission function

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant