Nothing Special   »   [go: up one dir, main page]

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using MS Teams as notification channel with new Workflows results in 400 Bad Request #2664

Closed
Reduxx opened this issue Sep 18, 2024 · 1 comment · Fixed by #2666
Closed
Assignees
Labels
bug Bugs

Comments

@Reduxx
Copy link
Reduxx commented Sep 18, 2024

🐞 Bug report

Describe the bug

When trying to use MS Teams as a notification channel with the new Workflow Webhooks, which are replacing the old Incoming Webhooks, the job runs into 400 Bad Request.

Steps To Reproduce

Steps to reproduce the behavior:

  1. Create a new Workflow in MS Teams where the Trigger is "When a Teams webhook request is received"
  2. Use "HTTP POST URL" as endPoint for ms-teams Notification WebHook
  3. Run any scan (I ran the default nmap scan)
  4. See error

Expected behavior

Normally the endpoint returns a 202 Accepted in case of the Workflows

System:

  • secureCodeBox Version/Release
  • OS: MacOS 15.0
  • Kubernetes Server Version: v1.30.2 (Using kubernetes from Docker Desktop)
  • Docker Version: 27.2.0, build 3ab4256

Screenshots / Logs

nmap.log

Additional context

From what I gathered the client request from axios has the header "Content-Type: application/x-www-form-urlencoded" and when I do a curl request with that I also get a 400. If I change it in the curl request to "Content-Type: application/json" the webhook is triggered correctly and the message card appears in the channel as expected.

@Reduxx Reduxx added the bug Bugs label Sep 18, 2024
@J12934
Copy link
Member
J12934 commented Sep 18, 2024

Ah thanks for reporting this, and looking into it already 🙌
Was also trying to setup / debug this last week, will try to get this fixed in the next release

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Bugs
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants