Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dont make peopleapacheorg apicalls #1

Open
wants to merge 16 commits into
base: release/v0.57
Choose a base branch
from
7 changes: 7 additions & 0 deletions pkg/dependency/parser/java/pom/parse.go
Original file line number Diff line number Diff line change
Expand Up @@ -703,6 +703,13 @@ func (p *Parser) remoteRepoRequest(repo string, paths []string) (*http.Request,
paths = append([]string{repoURL.Path}, paths...)
repoURL.Path = path.Join(paths...)

// don't do requests to `people.apache.org`. as it
if strings.Contains(repoURL.Host, "people.apache.org") {
SemProvoost marked this conversation as resolved.
Show resolved Hide resolved
// this domain is used to fetch pom files in some cases, but it just nearly always times out and makes everything extremely slow
// so we just skip it
return nil, xerrors.Errorf("people.apache.org is blocklisted")
}

req, err := http.NewRequest(http.MethodGet, repoURL.String(), http.NoBody)
if err != nil {
return nil, xerrors.Errorf("unable to create HTTP request: %w", err)
Expand Down