[PR #22438] [CLOSED] fix: prevent TypeError in Teams webhook when user data is missing #42325

New Issue

GiteaMirror · 2026-04-25T14:15:57-05:00

GiteaMirror commented

2026-04-25 14:15:57 -05:00

📋 Pull Request Information

Original PR: https://github.com/open-webui/open-webui/pull/22438
Author: @alvinttang
Created: 3/8/2026
Status: ❌ Closed

Base: dev ← Head: fix/webhook-teams-json-loads-crash

📝 Commits (1)

e99b2b9 fix: prevent TypeError in Teams webhook when user data is missing

📊 Changes

1 file changed (+6 additions, -1 deletions)

View changed files

📝 backend/open_webui/utils/webhook.py (+6 -1)

📄 Description

Description

Fixed TypeError crash in Teams webhook handler. json.loads(event_data.get("user", {})) passes a dict {} as default to json.loads() which expects a string. When the "user" key is absent from event_data, this crashes with TypeError: the JSON object must be str, bytes or bytearray, not dict.

Fixed

Teams webhook handler no longer crashes when "user" key is missing from event data

Additional Information

Tested by reviewing the code path and confirming the fix handles the missing key case
Self-reviewed the code changes
Rebased onto dev (replaces #22433)

Contributor License Agreement

By submitting this pull request, I confirm that I have read and fully agree to the Contributor License Agreement (CLA), and I am providing my contributions under its terms.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/open-webui/open-webui/pull/22438 **Author:** [@alvinttang](https://github.com/alvinttang) **Created:** 3/8/2026 **Status:** ❌ Closed **Base:** `dev` ← **Head:** `fix/webhook-teams-json-loads-crash` --- ### 📝 Commits (1) - [`e99b2b9`](https://github.com/open-webui/open-webui/commit/e99b2b9890d056268624dcaf31f9a8caf6efce91) fix: prevent TypeError in Teams webhook when user data is missing ### 📊 Changes **1 file changed** (+6 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `backend/open_webui/utils/webhook.py` (+6 -1) </details> ### 📄 Description ### Description - Fixed `TypeError` crash in Teams webhook handler. `json.loads(event_data.get("user", {}))` passes a dict `{}` as default to `json.loads()` which expects a string. When the `"user"` key is absent from `event_data`, this crashes with `TypeError: the JSON object must be str, bytes or bytearray, not dict`. ### Fixed - Teams webhook handler no longer crashes when `"user"` key is missing from event data ### Additional Information - Tested by reviewing the code path and confirming the fix handles the missing key case - Self-reviewed the code changes - Rebased onto dev (replaces #22433) ### Contributor License Agreement - [x] By submitting this pull request, I confirm that I have read and fully agree to the [Contributor License Agreement (CLA)](https://github.com/open-webui/open-webui/blob/main/CONTRIBUTOR_LICENSE_AGREEMENT), and I am providing my contributions under its terms. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-04-25 14:15:57 -05:00

GiteaMirror closed this issue

2026-04-25 14:15:58 -05:00

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/open-webui#42325