Skip to content

Disable response body validation by default#43

Merged
stanislav-zaprudskiy merged 2 commits intostable/2025.1-m3from
cherry-pick-bug/2126676
Dec 5, 2025
Merged

Disable response body validation by default#43
stanislav-zaprudskiy merged 2 commits intostable/2025.1-m3from
cherry-pick-bug/2126676

Conversation

@stanislav-zaprudskiy
Copy link
Copy Markdown
Collaborator

@stanislav-zaprudskiy stanislav-zaprudskiy commented Dec 5, 2025

This should not be used in production for the reasons given inline.

Change-Id: Ie40f41f57e316888c2b33f2952edcbac702c1c79
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Depends-on: https://review.opendev.org/c/openstack/devstack/+/962852
Closes-bug: #2126676
@stanislav-zaprudskiy stanislav-zaprudskiy marked this pull request as ready for review December 5, 2025 15:51
bbobrov
bbobrov previously approved these changes Dec 5, 2025
rajivmucheli
rajivmucheli previously approved these changes Dec 5, 2025
For some reason a bug went unnoticed where in the schema validation we
log the message from the decorators, but logging itself in not imported.

Change-Id: I6ddb69d21d22eafbfcde5c8952a63e39750e6328
Signed-off-by: Artem Goncharov <artem.goncharov@gmail.com>
Copy link
Copy Markdown
Collaborator

@bbobrov bbobrov left a comment

Choose a reason for hiding this comment

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

ok if pep8 and the rest of the jobs are happy

@stanislav-zaprudskiy stanislav-zaprudskiy merged commit ad13ce9 into stable/2025.1-m3 Dec 5, 2025
3 checks passed
@stanislav-zaprudskiy stanislav-zaprudskiy deleted the cherry-pick-bug/2126676 branch December 5, 2025 16:45
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.

6 participants