Opened at 2013-07-13T18:13:05Z
Closed at 2013-07-13T18:55:48Z
#1077 closed defect (fixed)
[PATCH] Twitter URL length calculated wrong
Reported by: | Owned by: | ||
---|---|---|---|
Priority: | normal | Milestone: | |
Component: | BitlBee | Version: | devel |
Keywords: | Cc: | ||
IRC client+version: | Client-independent | Operating System: | Linux |
OS version/distro: |
Description
Heya,
in some corner cases, I noticed my tweets weren't getting accepted by the server because they were too long, but bitlbee didn't reject them.
It seems this is related to https://dev.twitter.com/blog/upcoming-tco-changes
The attached patch changes the default for target_url_length from 20 to 23 (not sure if 22 or 23 would be more sensible there, your decision)
Attachments (1)
Change History (2)
Changed at 2013-07-13T18:13:39Z by
Attachment: | urllen.patch added |
---|
comment:1 Changed at 2013-07-13T18:55:48Z by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Ah yes, I did notice that too sometimes. Thanks for figuring it out. I went for 22, since there's a snippet of code that adds 1 for https URLs already.
That's some serious innovation at Twitter again. So sick of their crap.
URL length patch against current BZR head