• 
      

    Further bullet-proof Bitbucket webhook payload parsing.

    Review Request #10595 — Created June 17, 2019 and submitted

    Information

    Review Board
    release-3.0.x

    Reviewers

    A recent change attempted to deal with Bitbucket payload changes that
    resulted in fields sometimes not being present in the payload, depending
    on the event that fired. We've since found that sometimes the keys are
    present, but are null, breaking some of our parsing. This change further
    improves it, handling both missing keys and null values for those keys,
    and falling back to defaults in both cases.

    Unit tests pass.

    Summary ID
    Further bullet-proof Bitbucket webhook payload parsing.
    A recent change attempted to deal with Bitbucket payload changes that resulted in fields sometimes not being present in the payload, depending on the event that fired. We've since found that sometimes the keys are present, but are null, breaking some of our parsing. This change further improves it, handling both missing keys and null values for those keys, and falling back to defaults in both cases.
    3540e45921d11c8446c57ea573071c149b1f0126
    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-3.0.x (7f04e3f)