Fix some remaining issues with WebHook key types in payloads.

Review Request #10348 — Created Dec. 5, 2018 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
b53c2c3...

Reviewers

A recent change fixed several type-related issues in WebHook payloads,
but str types for keys were still problematic. This change addresses
this, and at the same time prevents future issues with subclasses of
primitive types, OrderedDicts, and byte strings containing Unicode
content. It also keeps order preserved for dictionaries, which helps
with consistent payloads (useful for testing).

Unit tests pass.

    Loading...