Skip to content
This repository was archived by the owner on Nov 29, 2021. It is now read-only.

[2.0.0] Set loglevel to debug for some message. - #159

Merged
bjoernricks merged 1 commit into
greenbone:ospd-2.0from
jjnicola:too-verbose-2
Oct 17, 2019
Merged

[2.0.0] Set loglevel to debug for some message.#159
bjoernricks merged 1 commit into
greenbone:ospd-2.0from
jjnicola:too-verbose-2

Conversation

@jjnicola

Copy link
Copy Markdown
Member

In case of the port list, it could flood the log file.
For the amount of result, it can be a bit confused, since many results
are "host details" which are not shown in gsa.

In case of the port list, it could be flood the log file.
For the amount of result, it can be a bit confused, since many results
are "host details" which are not shown in gsa.
@jjnicola
jjnicola requested a review from bjoernricks October 17, 2019 14:09
@codecov

codecov Bot commented Oct 17, 2019

Copy link
Copy Markdown

Codecov Report

Merging #159 into ospd-2.0 will not change coverage.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff            @@
##           ospd-2.0     #159   +/-   ##
=========================================
  Coverage     68.62%   68.62%           
=========================================
  Files            12       12           
  Lines          1756     1756           
=========================================
  Hits           1205     1205           
  Misses          551      551
Impacted Files Coverage Δ
ospd/ospd.py 73.23% <50%> (ø) ⬆️

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 c1c2780...b5ef640. Read the comment docs.

@bjoernricks
bjoernricks merged commit 0b01c21 into greenbone:ospd-2.0 Oct 17, 2019
@jjnicola
jjnicola deleted the too-verbose-2 branch November 4, 2019 14:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants