Skip to content

Add of the SecurityBridge_CL table#12847

Merged
v-atulyadav merged 9 commits into
Azure:masterfrom
SecurityBridge-DE:SecurityBridge_CL
Sep 30, 2025
Merged

Add of the SecurityBridge_CL table#12847
v-atulyadav merged 9 commits into
Azure:masterfrom
SecurityBridge-DE:SecurityBridge_CL

Conversation

@SecurityBridge-DE
Copy link
Copy Markdown
Contributor

@SecurityBridge-DE SecurityBridge-DE commented Sep 22, 2025

Required items, please complete

Change(s):

  • Update SecurityBridge_DCR.json - Added additional DCR

Reason for Change(s):

  • Adding of the table SecurityBridge_CL and the depended DCR.
  • Some clean-up

Version Updated:

  • Yes

Testing Completed:

  • Yes

Checked that the validations are passing and have addressed any issues that are present:

  • See guidance below

@v-maheshbh
Copy link
Copy Markdown
Contributor

Hi @SecurityBridge-DE Kindly check mainTemplate, Is it correct to use {{dataCollectionRuleId}} with double curly braces or should it be [parameters('dcrConfig').dataCollectionRuleId] instead?"

Thanks!

@MartinPankraz
Copy link
Copy Markdown
Contributor

MartinPankraz commented Sep 24, 2025

Hey @v-maheshbh, that is the result of the build script. See this published working connector for reference with the same output. I agree that your suggestion looks more consistent but the script produces differently and result is functional.

@v-maheshbh
Copy link
Copy Markdown
Contributor

Hi @SecurityBridge-DE Kindly attached the invocation logs of the running data connector.
Thanks!

@MartinPankraz
Copy link
Copy Markdown
Contributor

Here are the test results @v-maheshbh :

image image

@SecurityBridge-DE
Copy link
Copy Markdown
Contributor Author

Hi @v-maheshbh, do you have an update on when the changes are expected to be merged?
The app publishing in the content hub is currently failing because of the pending merge, so I just wanted to check on the timeline.

@v-atulyadav v-atulyadav merged commit 1d9cc54 into Azure:master Sep 30, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants