[teams] feat(injector): add Microsoft Teams injector (#141) #142

RomuDeuxfois · 2026-01-16T21:01:00Z

Proposed changes

Add Microsoft Teams injector

Related issues

Closes Add injector: Microsoft Teams #141

Copilot

Pull request overview

This PR adds a Microsoft Teams injector to the OpenAEV platform, enabling notifications to Teams channels via Power Automate HTTP webhooks. The implementation follows existing injector patterns (nuclei, nmap, http-query) and introduces shared helper code in injector_common.

Changes:

New Teams injector with client, contracts, helpers, and main entry point
Shared DataHelpers class in injector_common for data extraction (used by Teams, can benefit others)
Docker/deployment configuration and comprehensive documentation

Reviewed changes

Copilot reviewed 12 out of 19 changed files in this pull request and generated 8 comments.

Show a summary per file

File	Description
teams/teams/openaev_teams.py	Main entry point with message processing and execution logic
teams/teams/client/teams_client.py	HTTP client for sending Teams notifications
teams/teams/helpers/teams_helper.py	Payload builder for Teams messages
teams/teams/contracts_teams.py	Contract definitions for Teams integration
injector_common/injector_common/data_helpers.py	Shared data extraction helpers (new file)
teams/pyproject.toml	Python dependencies and project configuration
teams/Dockerfile	Multi-stage Docker build configuration
teams/docker-compose.yml	Docker deployment configuration
teams/README.md	Comprehensive documentation including Power Automate setup

teams/teams/openaev_teams.py

teams/teams/client/teams_client.py

Copilot · 2026-01-16T21:19:59Z

teams/teams/client/teams_client.py

+        response = requests.post(url=url, json=payload)
+        return ExecutionResult.from_http_response(response)


HTTP errors (4xx, 5xx) with response bodies are not captured. The from_http_response only checks status codes but doesn't extract error details from response.text. This makes debugging failed webhook calls difficult. Consider including response body in error messages for non-2xx responses.

teams/teams/openaev_teams.py

teams/teams/helpers/teams_helper.py

teams/pyproject.toml

teams/teams/contracts_teams.py

teams/teams/openaev_teams.py

Copilot

Pull request overview

Copilot reviewed 12 out of 19 changed files in this pull request and generated 2 comments.

Copilot · 2026-01-19T18:47:02Z

teams/teams/client/teams_client.py

+import requests
+from requests.exceptions import RequestException, Timeout
+
+DEFAULT_TIMEOUT = 5  # secondes


The DEFAULT_TIMEOUT constant has a French comment ("secondes"). For consistency with the codebase and professional standards, this should be in English: "seconds".

Suggested change

DEFAULT_TIMEOUT = 5 # secondes

DEFAULT_TIMEOUT = 5 # seconds

Copilot · 2026-01-19T18:47:02Z

teams/teams/helpers/teams_helper.py

+from typing import Dict
+
+
+class TeamsPayloadBuilder:
+
+    @staticmethod
+    def build(content: Dict) -> Dict:
+        return {
+            "title": content["title"],
+            "message": content["message"],
+        }


The Teams injector is missing test coverage. According to the repository guidelines, each injector should have its own test suite. The http-query injector has tests for its helpers, and other injectors (nuclei, nmap) have tests. Consider adding unit tests for TeamsPayloadBuilder.build(), TeamsClient.post_message(), and the DataHelpers methods to maintain consistency with the repository's testing practices.

teams/teams/openaev_teams.py

savacano28 · 2026-01-29T17:40:13Z

Tests:

Register injector
Creation Inject with contract
Execution: Error rabbitmq (not related to code)

Send inject contract teams:

savacano28 · 2026-02-06T10:55:19Z

teams/README.md

+First, create a dedicated virtual environment to keep things isolated:
+
+```shell
+py -3.12 -m venv venv


Normally, Poetry creates a virtual environment automatically, so this step shouldn’t be necessary.

savacano28 · 2026-02-09T09:46:27Z

.circleci/config.yml

-              docker push openaev/injector-nuclei:latest
-              docker push openbas/injector-nuclei:latest
+              for injector in $INJECTORS; do
+                docker tag openaev/injector-$injector:${IMAGETAG} openaev/injector-$injector:latest


Q: Do we still need to maintain the openbas reference? @antoinemzs

savacano28 · 2026-02-09T09:47:53Z

.circleci/config.yml

+          working_directory: ~/openaev
          command: |
-            poetry install --extras prod
+            INJECTORS="nuclei nmap http-query"


S: I have also added some tests in injector_common, maybe you could include it? and maybe you could add some tests to this new injector?

savacano28

Hi, I just added a few questions to the config.yml but it’s not blocking.

RomuDeuxfois force-pushed the issue/141 branch from 2d2bab6 to f1dbef3 Compare January 16, 2026 21:14

RomuDeuxfois requested a review from Copilot January 16, 2026 21:16

Copilot started reviewing on behalf of RomuDeuxfois January 16, 2026 21:17 View session

RomuDeuxfois linked an issue Jan 16, 2026 that may be closed by this pull request

Add injector: Microsoft Teams #141

Open

Copilot AI reviewed Jan 16, 2026

View reviewed changes

RomuDeuxfois requested a review from Copilot January 19, 2026 18:43

Copilot started reviewing on behalf of RomuDeuxfois January 19, 2026 18:44 View session

Copilot AI reviewed Jan 19, 2026

View reviewed changes

RomuDeuxfois force-pushed the issue/141 branch 2 times, most recently from fe1c0f6 to 4e5aa33 Compare January 19, 2026 19:18

savacano28 self-requested a review January 26, 2026 09:40

savacano28 reviewed Jan 27, 2026

View reviewed changes

teams/teams/openaev_teams.py Outdated Show resolved Hide resolved

RomuDeuxfois force-pushed the issue/141 branch from 4e5aa33 to cda9eee Compare January 27, 2026 14:41

RomuDeuxfois marked this pull request as draft January 27, 2026 14:41

RomuDeuxfois added 2 commits January 27, 2026 16:52

[teams] feat(injector): add Microsoft Teams injector

75e46dd

[teams] feat(injector): improv code

8131aea

RomuDeuxfois force-pushed the issue/141 branch from cda9eee to 8131aea Compare January 27, 2026 16:09

RomuDeuxfois added 4 commits January 27, 2026 17:11

[teams] feat(injector): improv code

1c35131

[teams] feat(injector): improv code

91c6a79

[ci] chore(circleci): improv script

49cc352

[ci] chore(circleci): improv script

13db9b3

RomuDeuxfois marked this pull request as ready for review January 28, 2026 12:31

Base automatically changed from release/current to main February 2, 2026 13:37

savacano28 reviewed Feb 6, 2026

View reviewed changes

savacano28 reviewed Feb 9, 2026

View reviewed changes

savacano28 approved these changes Feb 9, 2026

View reviewed changes

		response = requests.post(url=url, json=payload)
		return ExecutionResult.from_http_response(response)

[teams] feat(injector): add Microsoft Teams injector (#141) #142

Are you sure you want to change the base?

[teams] feat(injector): add Microsoft Teams injector (#141) #142

Uh oh!

Conversation

RomuDeuxfois commented Jan 16, 2026

Proposed changes

Related issues

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 19, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

savacano28 commented Jan 29, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

savacano28 Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

savacano28 Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

savacano28 Feb 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

savacano28 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

savacano28 commented Jan 29, 2026 •

edited

Loading

savacano28 Feb 9, 2026 •

edited

Loading

savacano28 Feb 9, 2026 •

edited

Loading