cyclonedx-dotnet icon indicating copy to clipboard operation
cyclonedx-dotnet copied to clipboard

Github license resolution fails for repos not using "master"

Open alitheg opened this issue 2 years ago • 4 comments

As lots of repos have now moved away from calling the default branch "master", this library fails Github license resolution for those libraries now. Perhaps the support ticket referenced in the code will get fixed, but if not it might be worth failing once and trying "main" too.

alitheg avatar Oct 01 '21 08:10 alitheg

Thanks for raising the issue. We should probably be checking what the default branch is first and just using that. I'm starting to work on the next major version. So I'll probably lump it in with that unless someone else wants to create a PR in the meantime.

coderpatros avatar Oct 01 '21 08:10 coderpatros

You know, actually - I was looking deeper into the code so that I could see whether I could tackle a PR for it, and it looks like you just ignore references that aren't master, rather than using master blindly?

So that would mean there's still an issue in that it should accept main as well (which would be a simple or I think?), but it also means the issue I'm seeing isn't the fault of CycloneDX: on inspecting my nuspec file, I can see that the license is specified as coming from master, but they've changed their repository to using main on Github

alitheg avatar Oct 01 '21 09:10 alitheg

Main problem could be fixed with: https://github.com/CycloneDX/cyclonedx-dotnet/pull/515. But all other branches besides main and master won't be fixed with that.

killi199 avatar Apr 28 '22 06:04 killi199

This issue is stale because it has been open for 3 months with no activity.

github-actions[bot] avatar Jan 07 '24 02:01 github-actions[bot]