Thomas Neidhart

Results 160 comments of Thomas Neidhart

I could reproduce the problem with firefox. Debugging it on firefox shows a CORS Missing Allow origin error. Also looking at the request being made to github.com, there are no...

ty for chiming in. I tested locally to update the cors-filter by explicitly specifying the methods, but it did not change a thing, so the request did not include a...

I wonder if its related to the hosting provider, see a similar SO question about cloudflare: https://stackoverflow.com/questions/74446830/how-to-fix-403-forbidden-errors-with-python-requests-even-with-user-agent-head This sounds like some protection mechanism against data scraping.

I tested the above snippet and can confirm that I also receive a 403 error. However, by simply adding a User-Agent header like that: ``` import requests headers = {...

I dont have a technical solution in mind as of now, but wanted to create this ticket to raise awareness that we will need some kind of generic way to...

[jsonnet-bundler](https://github.com/jsonnet-bundler/jsonnet-bundler) has a small utility method to generate the hash of a directory which might be useful here as well: https://github.com/jsonnet-bundler/jsonnet-bundler/blob/master/pkg/packages.go#L351

ok I did not realize that the directory hash should be also taking that into account. Maybe tree hashes as calculated by git would be useful. Here is some test...

Sorry for the misunderstanding, I did not intend to suggest to use git itself, but rather its mechanism to generate tree hashes.

Using the current version in my project results in this: ``` tn@proteus:~/workspace/netomi/ort$ time python-inspector --python-version 311 --operating-system linux --json-pdt /tmp/ort-PythonInspector676305276266227952/python-inspector16065407924165681261.json --analyze-setup-py-insecurely --requirement /tmp/ort-Poetry6870289546925138000/requirements.txt9586036788288994416.tmp real 0m29,104s user 0m12,745s sys 0m0,353s ```...

No that would only make subsequent runs of the action faster imho. The initial run might still be slow. However, I am not sure if this PR makes sense atm,...