ossf/cve-bin-tool

"cve_bin_tool.log.LOGGER is not a valid type"

Aberta

#2.870 aberto em 29 de mar. de 2023

 (7 comentários) (0 reação) (0 responsável)Python (646 forks)auto 404
good first issue

Métricas do repositório

Stars
 (1.738 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

I've just merged a fix for some type errors in #2786 but one of the changes the contributor made was switching us from using our customized LOGGER to the built-in logger that it's based on.

I'd prefer a solution that fixed cve_bin_tool.log.LOGGER itself. You can find it in https://github.com/intel/cve-bin-tool/blob/main/cve_bin_tool/log.py -- basically it's just setting up the logger with some consistent arguments. I'm not sure what the right solution here is but if I had to guess it's the fact that LOGGER itself can be Null so you might need to explicitly set a type for LOGGER and make sure that it is not null.

Guia do colaborador