[BitSight][Data Connector] - UI page of Function App data connector#14487
[BitSight][Data Connector] - UI page of Function App data connector#14487fenil-savani wants to merge 2 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the BitSight Function App data connector UI/instructions to align with using Azure Monitor Logs Ingestion API (DCR-based ingestion) and documents the change in the solution release notes.
Changes:
- Updated connector UI descriptions/instructions to reference Logs Ingestion API (DCR).
- Adjusted required permissions and deployment prerequisites (removed workspace shared key requirement; added Entra role assignment note).
- Updated BitSight solution release notes to capture the UI page enhancement.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| Solutions/BitSight/ReleaseNotes.md | Records the BitSight connector UI update for Logs Ingestion API. |
| Solutions/BitSight/Package/mainTemplate.json | Updates packaged connector UI text/instructions and permissions to align with Logs Ingestion API flow. |
| Solutions/BitSight/Data Connectors/BitSightDataConnector/BitSight_API_FunctionApp.json | Updates source connector definition UI text/instructions and permissions to align with Logs Ingestion API flow. |
|
Kindly review above comments. Thanks! |
|
Hi @v-maheshbh |
|
Data Connector Validations failed because Provided permissions does not match with Azure Function Connector Template. Please refer template https://github.com/Azure/Azure-Sentinel/blob/master/DataConnectors/Templates/Connector_REST_API_AzureFunctionApp_template/DataConnector_API_AzureFunctionApp_template.json Thanks! |
Change(s):
Reason for Change(s):
Version Updated:
Testing Completed: