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

[github] Error parsing URLs with trailing slash and #1159

Closed
sduenas opened this issue Aug 2, 2024 · 0 comments · Fixed by #1160
Closed

[github] Error parsing URLs with trailing slash and #1159

sduenas opened this issue Aug 2, 2024 · 0 comments · Fixed by #1160
Labels

Comments

@sduenas
Copy link
Member

sduenas commented Aug 2, 2024

When GitHub URL repository has a trailing slash and a dash, ELK doesn't split the name of the organization and the name of the repository correctly. For example, for https://github.com/rocky-linux/documentation/, elk generates this error:

mordred-1            | 2024-08-01 16:30:26,436 - GrimoireLab - sirmordred.task_collection - INFO - [github] collection starts for https://github.com/rocky-linux/documentation/
mordred-1            | 2024-08-01 16:30:26,479 - GrimoireLab - grimoire_elk.raw.elastic - INFO - [github] Incremental from: None until None for https://github.com/documentation/
mordred-1            | 2024-08-01 16:30:30,626 - GrimoireLab - grimoire_elk.elk - ERROR - Error feeding raw from github (https://github.com/documentation/): 404 Client Error: Not Found for url: https://api.github.com/repos/documentation//issues?state=all&per_page=100&direction=asc&sort=updated&since=1970-01-01T00%3A00%3A00%2B00%3A00
<snip long python error>
mordred-1            | requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://api.github.com/repos/documentation//issues?state=all&per_page=100&direction=asc&sort=updated&since=1970-01-01T00%3A00%3A00%2B00%3A00

The error can be found here.

ELK parses correctly the URL if the trailing slash is removed from the URL: https://github.com/rocky-linux/documentation.

This bug was reported by Jonas Rosland on the GrimoireLab CHAOSS channel on Slack.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant