search

TkTech / notifico

IRC Message Relay
http://n.tkte.ch
MIT License
155 stars 39 forks source link

Git.io no longer supports shortening links #204

Closed SadieCat closed 2 years ago

SadieCat commented 2 years ago

This breaks Notifico generating short links for events.

Ref: https://twitter.com/GHchangelog/status/1481041131530080260

TkTech commented 2 years ago

Did I miss a warning blog post or did they really just shut this off without notice :|

SadieCat commented 2 years ago

I think they shut it off with absolutely no notice. 😬

TkTech commented 2 years ago

Doesn't look like this is causing any downtime. 9 years ago I added a tiny bit of failure logic to it (https://github.com/TkTech/notifico/blob/0c4892f60959e0e679e1123e004adb16b035a040/notifico/services/hooks/github.py#L900) and from what I see it's working. I'll evaluate a different link shortener, since this is actually important for IRC. With the limit on message length in IRC, we want to try to fit as much as we can on one line or we have to send more lines, which means more throttling.