Skip to content

fix: IndexError when empty body response#1287

Open
pruthvi145 wants to merge 1 commit intomvantellingen:mainfrom
resilient-tech:fix-index-error
Open

fix: IndexError when empty body response#1287
pruthvi145 wants to merge 1 commit intomvantellingen:mainfrom
resilient-tech:fix-index-error

Conversation

@pruthvi145
Copy link
Copy Markdown

Issue

When there is no body element in the response, zeep is throwing an IndexError because there is eventually 0 xmlelement.

Changes Made

Added check for the not empty xmlelement in DocumentMessage._deserialize_body

Copy link
Copy Markdown

@timboode timboode left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mvantellingen We are running into this issue in our corporate environment. We have confirmed that the fix contained within this PR solves the index out of bounds issue. Any chance we can get this merged with the next zeep version?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants