Comment 12 for bug 1673375

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/ocata)

Reviewed: https://review.openstack.org/447071
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=a8a1915456a86f504d23f215867da730d436fe33
Submitter: Jenkins
Branch: stable/ocata

commit a8a1915456a86f504d23f215867da730d436fe33
Author: Balazs Gibizer <email address hidden>
Date: Fri Mar 17 11:24:49 2017 +0100

    do not include context to exception notification

    The wrap_exception decorator optionally emited a notification.
    Based on the code comments the original intention was not to include the
    context to that notification due to security reasons. However the
    implementation did included the context to the payload of the legacy
    notification.

    Recently we saw circural reference errors during the payload serialization
    of this notification. Based on the logs the only complex data structure
    that could cause circural reference is the context. So this patch
    removes the context from the legacy exception notification.

    The versioned exception notification is not affected as it does not
    contain the args of the decorated function.

    Closes-Bug: #1673375
    Change-Id: I1d217620e52d45595a3e0e49ed57b4ab33cd1688
    (cherry picked from commit 3bf177a59cfd0b4e74dba256c3466ba2ea9bfbf7)