[GH-ISSUE #5] URL wrong for GHE use cases #4

Closed
opened 2026-02-27 20:04:51 +03:00 by kerem · 2 comments
Owner

Originally created by @commitSpectral on GitHub (Oct 14, 2025).
Original GitHub issue: https://github.com/misiektoja/github_monitor/issues/5

Appreciate addressing the missing commits in push events. However, the GitHub URL seems to have been hard-coded to a public endpoint, which breaks on GitHub Enterprise. I resolved it by updating the following line:

compare_url = f"https://github.com/{short_repo}/compare/{before_sha[:12]}...{head_sha[:12]}"
Originally created by @commitSpectral on GitHub (Oct 14, 2025). Original GitHub issue: https://github.com/misiektoja/github_monitor/issues/5 Appreciate addressing the missing commits in push events. However, the GitHub URL seems to have been hard-coded to a public endpoint, which breaks on GitHub Enterprise. I resolved it by updating the following line: ```python compare_url = f"https://github.com/{short_repo}/compare/{before_sha[:12]}...{head_sha[:12]}" ```
kerem closed this issue 2026-02-27 20:04:51 +03:00
Author
Owner

@commitSpectral commented on GitHub (Oct 14, 2025):

Just pushed a fix for that.

<!-- gh-comment-id:3402364281 --> @commitSpectral commented on GitHub (Oct 14, 2025): Just pushed a fix for that.
Author
Owner

@misiektoja commented on GitHub (Oct 14, 2025):

Thx, I totally missed that as I do my tests on public GH. It also made me realize that other parts of the code might be affected as well.

<!-- gh-comment-id:3402394338 --> @misiektoja commented on GitHub (Oct 14, 2025): Thx, I totally missed that as I do my tests on public GH. It also made me realize that other parts of the code might be affected as well.
Sign in to join this conversation.
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/github_monitor#4
No description provided.