Add 'quiet' or 'silent' option for using httpie in scripts

[paulolieuthier]

Simple flag to avoid unnecessary output when using inside scripts.

[jkbrzt]

Rationale: Why do many commands provide a “quiet” option?

We can handle this by replacing our env.stdout and env.stderr with os.devnull at the appropriate moment. Instead of touching os.devnull directly at runtime, though, it should be added as Environment.devnull like we do with the standard streams:

https://github.com/jakubroztocil/httpie/blob/492687b0dafb7bec0d6281d019bb5f4f60439346/httpie/context.py#L32-L35

…and from that point only reference env.devnull. This will make it possible to write good tests. So basically there should only be one conditional somewhere and all existing behavior should remain intact.

[paulolieuthier]

What should the flag be?

[jkbrzt]

@paulolieuthier --quiet sounds pretty standard

[jkbrzt]

The replacement should probably happen here:

https://github.com/jakubroztocil/httpie/blob/492687b0dafb7bec0d6281d019bb5f4f60439346/httpie/cli/argparser.py#L123-L154

[velezbeltran]

Hey! Is someone working on this issue or can it be taken?

[jkbrzt]

@nb2838 I’m not. So, unless @paulolieuthier is looking into this, it should be up for grabs.

[paulolieuthier]

Go ahead @nb2838! I'm looking forward to the release :)

[YKo20010]

@nb2838 Are you still working on this? I'd like to pick up this issue if not!

[velezbeltran]

Yes currently working on it :)

[velezbeltran]

I've been working on the problem and I think I have a working solution. My only problem thus far is that I don't know at which point precisely to close the os.devnull file. If I don't close it and let the job be done by the garbage collectore my code works fine. However, when I try closing it by adding a
finally: env.dev.null.close() at the end of main in core.py I get an error if I use --download option. I was wondering if you have any suggestions for dealing with this issue.

[jkbrzt]

@nb2838 please open a PR and I’ll take a look

[marcelo-r]

#945 was merged and closed.