Activity log for bug #314426
| Date | Who | What changed | Old value | New value | Message |
|---|---|---|---|---|---|
| 2009-01-06 16:46:00 | Mihai Secasiu | bug | added bug | ||
| 2009-01-07 07:40:16 | Ryan Paul | gwibber: status | New | Confirmed | |
| 2009-01-07 07:40:16 | Ryan Paul | gwibber: importance | Undecided | Medium | |
| 2009-01-07 07:40:16 | Ryan Paul | gwibber: statusexplanation | Yep, I can replicate this. The reason why it consistently fails when you paste in a tinyurl link is because is.gd blacklists other shortening and redirection sites in order to prevent people from chaining shortened URLs. I concur with your suggestions for error handling and proper fallback behavior. I'll try to implement this soon. | ||
| 2009-01-08 21:57:56 | Evan McClain | bug | added attachment 'fix_urlshorten.diff' (fix_urlshorten.diff) | ||
| 2009-01-08 22:18:17 | Evan McClain | bug | added attachment 'fix_urlshorten2.diff' (fix_urlshorten2.diff) | ||
| 2009-02-02 03:21:19 | Ryan Paul | gwibber: status | Confirmed | Fix Released | |
| 2009-02-02 03:21:19 | Ryan Paul | gwibber: statusexplanation | Yep, I can replicate this. The reason why it consistently fails when you paste in a tinyurl link is because is.gd blacklists other shortening and redirection sites in order to prevent people from chaining shortened URLs. I concur with your suggestions for error handling and proper fallback behavior. I'll try to implement this soon. | I adapted Evan's patch and applied it in revision 210. Gwibber will automatically insert the original URL if the is.gd API call fails. It will also check the length of the URL and only try to shorten if it is longer than 20 characters. |
