search

kessler / license-report

create a short report about a project's dependencies (license, url etc)
MIT License
236 stars 39 forks source link

Added limit option to avoid too many concurrent connections #14

Closed ospfranco closed 5 years ago

ospfranco commented 5 years ago

Hey, I'm on vacation and my internet connection is not too fast, I tried to run this package and I was getting no output and the program simply hanged, I checked the code and noticed all the request to fetch the packages licenses go out at once, so I added a flag "limit" to limit the amount of concurrent connections.

Let me know what you think.

FYI, I need some help testing the config, I only tested changes in the code not the config, not sure how since test folder does not contain tests for the entire package only for certain files.

ospfranco commented 5 years ago

Is anybody still maintaining this project? is this PR going to be reviewed/approved?