Mailing-List: contact issues-help@camel.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@camel.apache.org
Date: Tue, 12 Apr 2016 10:36:25 +0000 (UTC)
From: "Claus Ibsen (JIRA)" <jira@apache.org>
To: issues@camel.apache.org
Message-ID: <JIRA.12957532.1460288101000.204916.1460457385479@Atlassian.JIRA>
In-Reply-To: <JIRA.12957532.1460288101000@Atlassian.JIRA>
References: <JIRA.12957532.1460288101000@Atlassian.JIRA>
 <JIRA.12957532.1460288101329@arcas>
Subject: [jira] [Updated] (CAMEL-9850) Twitter: truncation has no effect
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


     [ https://issues.apache.org/jira/browse/CAMEL-9850?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claus Ibsen updated CAMEL-9850:
-------------------------------
    Fix Version/s: 2.18.0

> Twitter: truncation has no effect
> ---------------------------------
>
>                 Key: CAMEL-9850
>                 URL: https://issues.apache.org/jira/browse/CAMEL-9850
>             Project: Camel
>          Issue Type: Bug
>          Components: camel-twitter
>    Affects Versions: 2.17.0
>            Reporter: martin scharm
>             Fix For: 2.18.0
>
>
> h3. Problem
> The current version of camel-twitter implements a truncation that does not have any effect:
> [UserProducer.java#L66|https://github.com/apache/camel/blob/83383c6318b1861d82d447165714f976e62229ab/components/camel-twitter/src/main/java/org/apache/camel/component/twitter/producer/UserProducer.java#L66]
> {code:java}
> if (status.length() > 160) {
>     log.warn("Message is longer than 160 characters. Message will be truncated!");
>     status = status.substring(0, 160);
> }
> {code}
> Status updates were truncated to 160 chars if they were longer. 
> However, the twitter limit is 140 chars. Updates with a length of 141-160 will not be truncated and fail; updates with a length of more than 160 will be truncated to 160 chars and still fail.
> Moreover, there are longer messages that are nevertheless compliant with the 140 char limit. See for example URLs: every URL is shortened using twitter's t.co shortener. Thus, even very long URLs occupy just a handful characters and the character counting, as used in camel-twitter, fails.
> h3. Proposals
> To fix this issue I prepared three proposals:
> h4. Remove Truncation
> As the truncation has no effect anyway it could simply be deleted. See my [commit b96035b on github|https://github.com/binfalse/camel/commit/b96035b239974e3ddf4cb1c9a84258e11b9a1433].
> h4. Update Truncation
> Update the current code to truncate to 140 chars. See my [commit 83b3f30 on github|https://github.com/binfalse/camel/commit/83b3f30c3ff993c50a250fd1085d7830c6570b62].
> h4. Introduce an optional Truncation
> As explained above, there are valid messages with more than 140 characters, eg. containing long URLs (see above). Using the previous solution it would not be possible to tweet them.
> Thus, based on [commit 83b3f30|https://github.com/binfalse/camel/commit/83b3f30c3ff993c50a250fd1085d7830c6570b62] I implemented optional truncation in my [commit bbbff05 on github|https://github.com/binfalse/camel/commit/bbbff051f3f982711e91ea60fb86b9218b567d63].
> That way, the client can decide whether messages should be truncated or not. There is a new flag {{truncate}} that can be used what creating an endpoint, eg.:
> {code}
> twitter://timeline/user?consumerKey=%s&...&truncate=true
> {code}
> However, if deciding for {{truncate=false}} the client should be aware that updating twitter may fail raising a {{twitter4j.TwitterException}}.
> I nevertheless set the default value to false, as this best reflects the previous behaviour and will not break other code.


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)