Code review comment for lp:~bigkevmcd/goose/fix-for-swift-headers

Revision history for this message
John A Meinel (jameinel) wrote :

LGTM. Thanks for doing this. This does end up changing the public API, but I can understand that you need some way to get the headers , and we don't currently expose it.

We could return the raw http.Response object, but ATM I think this provides the best balance of preserving some flexibility in the response and giving enough information for users.

review: Approve

« Back to merge proposal