svn commit: r1396505 - /maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java

[olamy]

Author: olamy
Date: Wed Oct 10 08:07:55 2012
New Revision: 1396505

URL: http://svn.apache.org/viewvc?rev=1396505&view=rev
Log:
handle http response

Modified:
    
maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java

Modified: 
maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java
URL: 
http://svn.apache.org/viewvc/maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java?rev=1396505&r1=1396504&r2=1396505&view=diff
==============================================================================
--- 
maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java
 (original)
+++ 
maven/sandbox/trunk/plugins/maven-patch-tracker-plugin/src/main/java/org/apache/maven/plugins/patchtracker/tracking/jenkins/JenkinsPatchTracker.java
 Wed Oct 10 08:07:55 2012
@@ -64,10 +64,18 @@ public class JenkinsPatchTracker
 
             HttpResponse r = defaultHttpClient.execute( post );
 
-            log.info( "r:" + r.getStatusLine().getStatusCode() + ", status: " 
+ r.getStatusLine().getReasonPhrase()+"," + EntityUtils.toString(  
r.getEntity()) );
-
-            // FIXME verify response code !
-
+            log.debug(
+                "r:" + r.getStatusLine().getStatusCode() + ", status: " + 
r.getStatusLine().getReasonPhrase() + ","
+                    + EntityUtils.toString( r.getEntity() ) );
+
+            int statusCode = r.getStatusLine().getStatusCode();
+
+            // Jenkins returns 302
+            if ( statusCode != 200 || statusCode != 302 )
+            {
+                throw new PatchTrackerException(
+                    "Jenkins returned :" + statusCode + " with ReasonPhrase :" 
+ r.getStatusLine().getReasonPhrase() );
+            }
 
             return new PatchTrackerResult();
         }