Update javadoc to ignore errors of java8 + add support for environment variabled defined in the build of jenkins.

Author: Amine Fattouch

src/main/java/com/amazonaws/codedeploy/AWSClients.java

@@ -89,6 +89,10 @@ public static AWSClients fromBasicCredentials(String region, String awsAccessKey
     /**
      * Via the default provider chain (i.e., global keys for this Jenkins instance),  return the account ID for the
      * currently authenticated user.
+     *
+     * @param proxyHost
+     * @param proxyPort
+     *
      * @return 12-digit account id
      */
     public static String getAccountId(String proxyHost, int proxyPort) {

src/main/java/com/amazonaws/codedeploy/AWSCodeDeployPublisher.java

@@ -70,7 +70,7 @@
 /**
  * The AWS CodeDeploy Publisher is a post-build plugin that adds the ability to start a new CodeDeploy deployment
  * with the project's workspace as the application revision.
- * <p/>
+ * <p></p>
  * To configure, users must create an IAM role that allows "S3" and "CodeDeploy" actions and must be assumable by
  * the globally configured keys. This allows the plugin to get temporary credentials instead of requiring permanent
  * credentials to be configured for each project.
@@ -199,12 +199,21 @@ public boolean perform(AbstractBuild build, Launcher launcher, BuildListener lis
                         this.proxyHost,
                         this.proxyPort);
             } else {
-                aws = AWSClients.fromBasicCredentials(
+                if (StringUtils.isEmpty(envVars.get(this.awsAccessKey)) && StringUtils.isEmpty(envVars.get(this.awsSecretKey))) {
+                    aws = AWSClients.fromBasicCredentials(
                         this.region,
                         this.awsAccessKey,
                         this.awsSecretKey,
                         this.proxyHost,
                         this.proxyPort);
+                } else {
+                    aws = AWSClients.fromBasicCredentials(
+                        this.region,
+                        envVars.get(this.awsAccessKey),
+                        envVars.get(this.awsSecretKey),
+                        this.proxyHost,
+                        this.proxyPort);
+                }
             }
         } else {
             aws = AWSClients.fromIAMRole(
@@ -495,8 +504,8 @@ public BuildStepMonitor getRequiredMonitorService() {
     /**
      * Descriptor for {@link AWSCodeDeployPublisher}. Used as a singleton.
      * The class is marked as public so that it can be accessed from views.
-     * <p/>
-     * <p/>
+     * <p></p>
+     * <p></p>
      * See <tt>src/main/resources/com/amazonaws/codedeploy/AWSCodeDeployPublisher/*.jelly</tt>
      * for the actual HTML fragment for the configuration screen.
      */

src/main/resources/com/amazonaws/codedeploy/AWSCodeDeployPublisher/help-awsAccessKey.html

@@ -1,6 +1,8 @@
 <div>
   <p>AWS Access and Secret keys to use for this deployment. At minimum the keys must be allowed to execute
-  <code>codedeploy:*</code> and <code>s3:Put*</code>. It's a best practice to have these keys be from an IAM role
+  <code>codedeploy:*</code> and <code>s3:Put*</code>. 
+  You can also use environment variables defined in jenkins's build for this fields.
+  It's a best practice to have these keys be from an IAM role
   with limited scope.</p>
 
   <p>

src/main/resources/com/amazonaws/codedeploy/AWSCodeDeployPublisher/help-awsSecretKey.html

@@ -1,6 +1,8 @@
 <div>
   <p>AWS Access and Secret keys to use for this deployment. At minimum the keys must be allowed to execute
-  <code>codedeploy:*</code> and <code>s3:Put*</code>. It's a best practice to have these keys be from an IAM role
+  <code>codedeploy:*</code> and <code>s3:Put*</code>. 
+  You can also use environment variables defined in jenkins's build for this fields.
+  It's a best practice to have these keys be from an IAM role
   with limited scope.</p>
 
   <p>