Add boolean "no_details" option to "get_reports()"

[Kars-de-Jong]

GMP v7 and up support the undocumented boolean parameter "details", which can be set to "0" to suppress results. As of GMP v8 this is the preferred way of getting a list of reports related to a task, since the reports are no longer included in get_tasks().

This closes #128.

Checklist:

• Tests
• CHANGELOG Entry
• Documentation

[codecov]

Codecov Report

Merging #129 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage    96.4%   96.41%   +<.01%     
==========================================
  Files          10       10              
  Lines        2478     2480       +2     
==========================================
+ Hits         2389     2391       +2     
  Misses         89       89

Impacted Files	Coverage Δ
gvm/protocols/gmpv7.py	99.74% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 181982d...09bf5fb. Read the comment docs.

[codecov]

Codecov Report

Merging #129 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage    96.4%   96.41%   +<.01%     
==========================================
  Files          10       10              
  Lines        2478     2480       +2     
==========================================
+ Hits         2389     2391       +2     
  Misses         89       89

Impacted Files	Coverage Δ
gvm/protocols/gmpv7.py	99.74% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 181982d...a1d7252. Read the comment docs.

[bjoernricks]

Do I understand your PR correctly that details="1" is the default? Means if the details attribute is not included in the request gvmd assumes it is 1?

[Kars-de-Jong]

Do I understand your PR correctly that details="1" is the default? Means if the details attribute is not included in the request gvmd assumes it is 1?

Yes, it seems to be the default.

[bjoernricks]

The PR looks very good!

I am not a fan of the details parameter and therefore I've not added it by intention. But if your use case really requires it, we should merge this PR.

[Kars-de-Jong]

It makes quite a big difference in the size of the xml returned, so it would be better for us.

[bjoernricks]

Yes, it seems to be the default.

😒

[bjoernricks]

@wiegandm could you take care of this PR next week?

[wiegandm]

@bjoernricks Ack.

@Kars-de-Jong, I think the comment should be the other way around, could you please check?

[bjoernricks]

I really don't like the no_details argument. We need to fix the protocol for this param!