Request response pretty printing - second attempt #58

merged 11 commits into from Apr 2, 2013


None yet
2 participants

davidapgar commented Mar 28, 2013

second attempt at this - I had deleted my fork, and on recreating it I can't get it to reference the previous pull request. See pull #52 for comments

Dave Apgar added some commits Mar 11, 2013

Dave Apgar
Add PDPrettyStringPrinting Protocol and text implementation
Handles the non-binary MIME types and defines the interface for
other pretty string processors
Dave Apgar
Add JSON pretty printer, and have it handle redacting
The PDDebugger loads this printer by default on monitoring all network
Dave Apgar
Parse text/binary responses with the default pretty string printer
Instead of hard-coding what is text or binary, if there isn't a pretty
string printer for the content type, treat it as binary
Dave Apgar
Move default text pretty printer registration to not depend on a netw…
…ork domain controller

Otherwise, there is a dependency the default instance being used prior to any prettyStringPrinters
being registered (which isn't a requirement)
Dave Apgar
Better handling of intial prettyPrinter registration
Also, add license information to added headers

Should be #pragma mark - Blah, then Xcode will show the line over it in that menu.


wlue commented Mar 29, 2013

Other than the mark issue, LGTM!


davidapgar commented Mar 29, 2013

Fixed the transposed pragma statement. Sorry about that

wlue added a commit that referenced this pull request Apr 2, 2013

Merge pull request #58 from davidapgar/request-response-pretty-printing
Request response pretty printing - second attempt

@wlue wlue merged commit db702af into square:master Apr 2, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment