svn commit: r628331 - in /maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira: AbstractJiraDownloader.java JiraHelper.java

[de...@apache.org]

Author: dennisl
Date: Sat Feb 16 07:42:31 2008
New Revision: 628331

URL: http://svn.apache.org/viewvc?rev=628331&view=rev
Log:
[MCHANGES-101] Consider changing log outputs to debug
Submitted by: Benjamin Bentmann
Reviewed by: Dennis Lundberg

o Applied with modifications.

Modified:
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
    maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraHelper.java

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java?rev=628331&r1=628330&r2=628331&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/AbstractJiraDownloader.java Sat Feb 16 07:42:31 2008
@@ -348,7 +348,7 @@
             {
                 if ( jiraId == null || jiraId.length() == 0 )
                 {
-                    log.info( "The JIRA URL " + project.getIssueManagement().getUrl()
+                    log.debug( "The JIRA URL " + project.getIssueManagement().getUrl()
                         + " doesn't include a pid, trying to extract it from JIRA." );
                     jiraId = JiraHelper.getPidFromJira( log, project.getIssueManagement().getUrl(), client );
                 }
@@ -428,7 +428,7 @@
         {
             jiraUrl = jiraUrl.substring( 0, jiraUrl.lastIndexOf( "/" ) );
         }
-        getLog().info( "JIRA lives at: " + jiraUrl );
+        getLog().debug( "JIRA lives at: " + jiraUrl );
 
         urlMap.put( "url", jiraUrl );
 
@@ -450,7 +450,7 @@
 
             Credentials defaultcreds = new UsernamePasswordCredentials( webUser, webPassword );
 
-            getLog().info( "Using username: " + webUser + " for Basic Authentication." );
+            getLog().debug( "Using username: " + webUser + " for Basic Authentication." );
 
             client.getState().setCredentials( new AuthScope( null, AuthScope.ANY_PORT, null, AuthScope.ANY_SCHEME ),
                                               defaultcreds );
@@ -482,7 +482,7 @@
         {
             password = StringUtils.repeat( "*", jiraPassword.length() );
         }
-        getLog().info( "Login URL: " + loginLink + "&os_password=" + password );
+        getLog().debug( "Login URL: " + loginLink + "&os_password=" + password );
 
         loginLink.append( "&os_password=" ).append( jiraPassword );
 
@@ -497,7 +497,7 @@
 
             if ( loginSucceeded( loginGet ) )
             {
-                getLog().info( "Successfully logged in into JIRA." );
+                getLog().debug( "Successfully logged in into JIRA." );
                 return true;
             }
             else
@@ -585,18 +585,18 @@
 
             proxyPass = settings.getActiveProxy().getPassword();
 
-            getLog().info( proxyPass );
+            getLog().debug( proxyPass );
         }
 
         if ( proxyHost != null )
         {
             client.getHostConfiguration().setProxy( proxyHost, proxyPort );
 
-            getLog().info( "Using proxy: " + proxyHost + " at port " + proxyPort );
+            getLog().debug( "Using proxy: " + proxyHost + " at port " + proxyPort );
 
             if ( proxyUser != null )
             {
-                getLog().info( "Using proxy user: " + proxyUser );
+                getLog().debug( "Using proxy user: " + proxyUser );
 
                 client.getState().setProxyCredentials(
                                                        new AuthScope( null, AuthScope.ANY_PORT, null,
@@ -628,7 +628,7 @@
 
             if ( sl == null )
             {
-                getLog().info( "Unknown error validating link: " + link );
+                getLog().error( "Unknown error validating link: " + link );
 
                 return;
             }
@@ -640,7 +640,7 @@
 
                 if ( locationHeader == null )
                 {
-                    getLog().info( "Site sent redirect, but did not set Location header" );
+                    getLog().warn( "Site sent redirect, but did not set Location header" );
                 }
                 else
                 {
@@ -668,7 +668,7 @@
 
                 pw.close();
 
-                getLog().info( "Downloading from JIRA was successful" );
+                getLog().debug( "Downloading from JIRA was successful" );
             }
             else
             {

Modified: maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraHelper.java
URL: http://svn.apache.org/viewvc/maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraHelper.java?rev=628331&r1=628330&r2=628331&view=diff
==============================================================================
--- maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraHelper.java (original)
+++ maven/plugins/trunk/maven-changes-plugin/src/main/java/org/apache/maven/plugin/jira/JiraHelper.java Sat Feb 16 07:42:31 2008
@@ -53,7 +53,7 @@
         try
         {
             client.executeMethod( gm );
-            log.info( "Successfully reached JIRA." );
+            log.debug( "Successfully reached JIRA." );
             projectPage = gm.getResponseBodyAsString();
         }
         catch ( Exception e )