For these two is it safe to do e.g. `response[:tracking_urls] ||= {}` instead? The way it is now it looks like you're trying to preserve any keys that have been defined but set to nil -- feels like we'd want to tromp nils with empty sets, no?