codacy-analysis-cli-action icon indicating copy to clipboard operation
codacy-analysis-cli-action copied to clipboard

[TS-171] pylint: Module 'http.server' has no 'ThreadingHTTPServer' member

Open mirabilos opened this issue 2 years ago • 7 comments

Erm, it does. That’s what I run.

mirabilos avatar Jan 12 '23 18:01 mirabilos

Internal ticket created : TS-171

github-actions[bot] avatar Jan 12 '23 18:01 github-actions[bot]

Hello @mirabilos

Thanks for reporting this.

Are you still experiencing the same behavior?

Kind regards, David Marinho

DMarinhoCodacy avatar May 30 '23 11:05 DMarinhoCodacy

Yes, it shows still up https://github.com/tarent/jensjs/security/code-scanning/671, together with even more error messages that are not even errors

mirabilos avatar May 30 '23 17:05 mirabilos

Hello @mirabilos,

I've made some tests with http.server and ThreadingHTTPServer is not returning any error since we bumped Pylint to a recent version (not the latest one yet but we'll do that very soon). But, either way, you're completely right. According to the docs, ThreadingHTTPServer is part of http.server since Python3.7.

Can you take some of your time and test if you still have this error?

Thanks in advance for your help.

Kind regards, David Marinho

DMarinhoCodacy avatar Oct 03 '23 14:10 DMarinhoCodacy

Ah, unfortunately it seems I cannot retest this, as it fails to run nowadays:

Error: Code Scanning could not process the submitted SARIF file:
rejecting SARIF, as there are more runs than allowed (21 > 20)

I think this is because too many test utilities run, or something.

mirabilos avatar Oct 03 '23 23:10 mirabilos

Ah hmm. https://docs.codacy.com/repositories-configure/codacy-configuration-file/#legacy-pylint-19 says to disable pylint and rely on pylintpython3.

Which is what I want, actually.

mirabilos avatar Jan 20 '24 23:01 mirabilos

However, this…

$ cat .codacy.yml                                                               
---
engines:
  pylint:
    # pylintpython3 is used instead
    enabled: false
  tsqllint:
    # for Microsoft only
    enabled: false

… does not disable either ☹☹☹

mirabilos avatar Jan 21 '24 00:01 mirabilos