Conversation
Fixes #224
|
@xsuchy @swinslow @jlovejoy - FYI OSI changed their URLs for licenses and made several breaking changes to their API. The LicenseListPublisher currently in the license-list-XML repo is generating bad data for the cross-refs / URLs for the OSI. Then changes are recent so the published licenses should be fine. We just need to fix this before the next release. This PR addresses the changes and should work with the new APIs and URLs. I'll create a release and update the license-list-XML repo in the next couple days. |
| * @return normalized URL | ||
| */ | ||
| public static String normalizeOsiUrl(String url) { | ||
| return url.toLowerCase() // OSI changed the case on URLs like Apache-2.0 (now apache-2.0) |
There was a problem hiding this comment.
The old variant still works. Do you have an idea if this is just for the transition period?
There was a problem hiding this comment.
The old variants seem to redirect to the new version. In the API, they only store the latest URLs (not the previous variants), so we have to normalize. A bit of historical context - we use to check the URL's directly, but the OSI website would fail due to rate limiting, so we now use their API which is much friendlier to their website infrastructure.
2 participants
Fixes #224