Enable to set HTTP headers

davecheney / httpstat

It's like curl -v, with colours.

MIT License

6.96k stars 382 forks source link

Enable to set HTTP headers #33

Closed tcnksm closed 7 years ago

tcnksm commented 7 years ago

This PR enables to set http headers via -H flag,

$ httpstat -H "Authorization: bearer eyJ0eXAiO..." -H "Cookie: " http://deeeet.com

davecheney commented 7 years ago

@tcnksm do you want to work together with @freeformz (either for a new PR, or just commenting on the one you two like the best).

@tcnksm you were technically first (according to the PR number that github gave), but @freeformz reused the headers type which was a nice touch and also included a test.

I'd like to see the best of both of these PR's merged into one if possible.

Also, don't forget to add a ~~strikeout~~ in the README TODO section to claim this feature as your prize.

tcnksm commented 7 years ago

Yeah, it seems @freeformz's one is better. I will close mine and comment on his PR