mkdocs. Mapping tables apis. Fix #1032#1043
Merged
Merged
Conversation
dgarijo
requested changes
Jul 2, 2026
|
|
||
| *1* | ||
| Extracted by fetching the LICENSE file via `GET /api/v1/repos/{owner}/{repo}/contents/{filename}` (tries `LICENSE`, `LICENSE.md`, `LICENCE`, `COPYING`). The content is base64-decoded and analyzed with `detect_license_spdx()` to obtain the SPDX identifier, name and URL. Technique: `Codeberg_API`. No newline at end of file | ||
| Extracted by fetching the LICENSE file via `GET /api/v1/repos/{owner}/{repo}/contents/{filename}` (tries `LICENSE`, `LICENSE.md`, `LICENCE`, `COPYING`). The content is base64-decoded and analyzed with `detect_license_spdx()` to obtain the SPDX identifier, name and URL. |
Collaborator
There was a problem hiding this comment.
This should not be this way. We do the inspection of the files offline, so this behavior should not happen if the API does not support licenses.
| `https://bitbucket.org/{owner}/{repo}/get/{branch}.zip` | ||
|
|
||
| Bitbucket archive URLs typically include a `Content-Length` header, so the size limit check | ||
| can be performed before downloading. |
Collaborator
There was a problem hiding this comment.
The last comment, I don't know who is targeted to. Please remove
| without byte counts per language. | ||
| - **CODEOWNERS enrichment**: Not supported for Bitbucket, as the platform does not | ||
| expose a public user API. | ||
| - **Rate limits**: Unauthenticated requests are limited to 60 requests/hour. |
Collaborator
There was a problem hiding this comment.
We already add a means to address rate limits. We indicate how to do this in the readme, no? I think this part is not needed
| - **Rate limits**: Unauthenticated requests are limited to 60 requests/hour. Authenticated | ||
| requests (via `codeberg-token`) have higher limits. Create a token at | ||
| `https://codeberg.org/user/settings/applications`. | ||
| - **License detection**: Codeberg does not provide a `license` field in the repository |
Collaborator
There was a problem hiding this comment.
Per above, this should be removed
|
|
||
| - **No Content-Length**: GitHub archive downloads lack a `Content-Length` header, so the | ||
| size limit check relies on streaming (reading 1 MB chunks until the limit is exceeded). | ||
| - **Rate limits**: Unauthenticated requests are limited to 60 requests/hour. Authenticated |
Collaborator
There was a problem hiding this comment.
We already address this in the config, so it's not a limitation
| requests (via `github-token`) are limited to 5,000 requests/hour. | ||
| - **Private repositories**: SOMEF cannot access private repositories without a valid token. | ||
| - **Asset download count**: GitHub provides `download_count` for release assets; other | ||
| providers may not offer this field. |
Collaborator
There was a problem hiding this comment.
This last thing, I am not sure we need it
| - **Self-hosted detection**: SOMEF detects self-hosted instances by checking for `gitlab.com` | ||
| in the URL. If a self-hosted instance uses a custom domain without a GitLab API endpoint, | ||
| detection may fail. | ||
| - **Rate limits**: GitLab.com unauthenticated requests are limited to 600 requests/hour. |
Collaborator
There was a problem hiding this comment.
This is already addressed. Remove
| - **Rate limits**: GitLab.com unauthenticated requests are limited to 600 requests/hour. | ||
| Authenticated requests (via `gitlab-token`)have higher limits depending on the token type. Self-hosted | ||
| instances have their own rate limits. | ||
| - **Programming languages**: GitLab returns only language names (no byte counts). |
Collaborator
There was a problem hiding this comment.
This is not a problem. Remove
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.