Skip to content

TrendMicro V1 Data Connector fix DefaultAzureCredential, retire uae site and add new uk site config#12824

Merged
v-dvedak merged 7 commits into
Azure:masterfrom
V1ManagedServices:master
Sep 18, 2025
Merged

TrendMicro V1 Data Connector fix DefaultAzureCredential, retire uae site and add new uk site config#12824
v-dvedak merged 7 commits into
Azure:masterfrom
V1ManagedServices:master

Conversation

@V1ManagedServices
Copy link
Copy Markdown
Contributor

Required items, please complete

Change(s):

  • Fix DefaultAzureCredential, replace with ManagedIdentityCredential
  • Remove uae site config
  • Add uk site config
  • Version change

Reason for Change(s):

  • Fix DefaultAzureCredential
  • Support uk site
  • Retire uae site

Version Updated:

  • No Detections/Analytic Rule templates updated.

Testing Completed:

  • Yes

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

  • Yes

@V1ManagedServices V1ManagedServices requested review from a team as code owners September 17, 2025 09:29
@v-shukore v-shukore self-assigned this Sep 17, 2025
@v-shukore v-shukore added the Connector Connector specialty review needed label Sep 17, 2025
@v-shukore
Copy link
Copy Markdown
Contributor

Hi @V1ManagedServices, could you please share the current invocation logs for the 'TrendMicroVisionOne' Data Connector?
Also, we noticed that the AzureFunctionTrendMicroXDR.zip file has been updated could you let us know what changes have been made in the ZIP? Thanks.

@V1ManagedServices
Copy link
Copy Markdown
Contributor Author

Hi @V1ManagedServices, could you please share the current invocation logs for the 'TrendMicroVisionOne' Data Connector? Also, we noticed that the AzureFunctionTrendMicroXDR.zip file has been updated could you let us know what changes have been made in the ZIP? Thanks.

thank you for quick response.
the screenshot is provided by our RD:
image

the latest zip file content can be found in "Files changed" panel:

  1. shared_code/configurations.py (1. add uk and remove uae config 2. use ManagedIdentityCredential, 3. change version info)
  2. timer_trigger_oat/init.py (change version info: 1.2.6)
  3. timer_trigger/init.py (change version info: 1.2.6)

we don't need to upload the latest zip file?

@V1ManagedServices
Copy link
Copy Markdown
Contributor Author

hi @v-shukore:

after checking, changed files (4 files) are correct.

  • TrendMicroXDR-LogPuller/azuredeploy_TrendMicroVisionOne_API_FunctionApp.json (add uk and remove uae config)
  • TrendMicroXDR-LogPuller/shared_code/configurations.py (1. add uk and remove uae config 2. use ManagedIdentityCredential, 3. change version info)
  • TrendMicroXDR-LogPuller/timer_trigger/init.py (change version info: 1.2.6)
  • TrendMicroXDR-LogPuller/timer_trigger_oat/init.py (change version info: 1.2.6)

Invocation logs appear to be normal:
image


however, RD uploaded a wrong zip file. it does not contain uk config. it only includes the following changes (3 files):

  • TrendMicroXDR-LogPuller/shared_code/configurations.py (1. use ManagedIdentityCredential, 2. change version info)
  • TrendMicroXDR-LogPuller/timer_trigger/init.py (change version info: 1.2.6)
  • TrendMicroXDR-LogPuller/timer_trigger_oat/init.py (change version info: 1.2.6)

Could you please provide your recommendation regarding this situation?

@v-shukore
Copy link
Copy Markdown
Contributor

Hi @V1ManagedServices, if the changes are correct and you're receiving the expected logs, then everything looks good—no need to worry. Regarding the ZIP file, it should be updated since the above files were modified. I believe you've already updated it, so we can proceed with the merge. Thanks!

sha256: cb100a5baf3419
@V1ManagedServices
Copy link
Copy Markdown
Contributor Author

overview:
image

configuration.py
image

init.py
image
image

azuredeploy_TrendMicroVisionOne_API_FunctionApp.json
image

@V1ManagedServices
Copy link
Copy Markdown
Contributor Author

invocation log:
image

@v-dvedak v-dvedak merged commit c6afb45 into Azure:master Sep 18, 2025
30 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Connector Connector specialty review needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants