spring-projects / spring-integration

Spring Integration provides an extension of the Spring programming model to support the well-known Enterprise Integration Patterns (EIP)
http://projects.spring.io/spring-integration/
Apache License 2.0
1.54k stars 1.11k forks source link

directmessage id overflow [INT-1902] #5846

Closed spring-operator closed 13 years ago

spring-operator commented 13 years ago

Philippe Soares opened INT-1902 and commented

There is an overflow bug in twitter4j, which description can be found here : http://groups.google.com/group/twitter4j/browse_thread/thread/180807125fb26781

It has been fixed in twitter4j 2.1.10.

The twitter adapter fails to send directmessages when the user id is > Integer.MAXVALUE

Please upgrade the underlying api.

I managed to do a workaround by excluding the actual version shipped with the twitter adapter in my pom.xml file, and putting version 2.1.10 instead, like this :

<project>
...
<dependencies>
    <dependency>
       <groupId>org.springframework.integration</groupId>
       <artifactId>spring-integration-core</artifactId>
       <version>2.0.3.RELEASE</version>
    </dependency>
    <dependency>
       <groupId>org.springframework.integration</groupId>
       <artifactId>spring-integration-twitter</artifactId>
       <version>2.0.3.RELEASE</version>
       <exclusions>
        <exclusion>
            <groupId>org.twitter4j</groupId>
            <artifactId>twitter4j-core</artifactId>
        </exclusion>
       </exclusions>
    </dependency>
    <dependency>
         <groupId>org.twitter4j</groupId>
         <artifactId>twitter4j-core</artifactId>
         <version>2.1.10</version>
      </dependency>
  </dependencies>
  <repositories>
    <repository>
       <id>repository.springframework.maven.release</id>
       <name>Spring Framework Maven Release Repository</name>
       <url>http://maven.springframework.org/release</url>
    </repository>
    <repository>
         <id>twitter4j.org</id>
         <name>twitter4j.org Repository</name>
         <url>http://twitter4j.org/maven2</url>
         <releases>
            <enabled>true</enabled>
         </releases>
         <snapshots>
            <enabled>true</enabled>
         </snapshots>
      </repository>
  </repositories>
</project>

Affects: 2.0.3

spring-operator commented 13 years ago

Oleg Zhurakousky commented

done, should be available with tonight's nightly

spring-operator commented 13 years ago

Mark Fisher commented

As long as we are upgrading this dependency, shouldn't we go ahead and use the latest version (appears to be 2.1.12?