proxyHost and proxyHttps options do not work. about appengine-maven-plugin HOT 5 CLOSED

Hi,

I downloaded the plugin and had a look at the code. I think I found a bug in 
how the proxy parameters were managed.
Here is my proposed patch:

diff --git a/src/main/java/com/google/appengine/appcfg/AbstractAppCfgMojo.java 
b/src/main/java/com/google/appengine/appcfg/AbstractAppCfgMojo.java
index 6793085..f6c3148 100644
--- a/src/main/java/com/google/appengine/appcfg/AbstractAppCfgMojo.java
+++ b/src/main/java/com/google/appengine/appcfg/AbstractAppCfgMojo.java
@@ -236,6 +236,8 @@

     arguments.add(action);
     arguments.add(appDir);
+    
+    getLog().debug("passing arguments: " + arguments.toString());

     try {
       AppCfg.main(arguments.toArray(new String[arguments.size()]));
@@ -280,13 +282,12 @@
     }

     if (proxyHost != null && !proxyHost.isEmpty()) {
-      arguments.add("--proxy");
+      arguments.add("-p");
       arguments.add(proxyHost);
     }

     if (proxyHttps != null && !proxyHttps.isEmpty()) {
-      arguments.add("--proxy_https");
-      arguments.add(proxyHttps);
+      arguments.add("--proxy_https="+proxyHttps);
     }

     if (noCookies) {

I need this fix as well. The workaround that I used is to pass the proxy 
settings on the maven command line:

mvn -DproxySet=true -DproxyHost=proxy.example.com -DproxyPort=8080 ...

But I definitely want the fix and not my band-aid. Ideally it would be nice if 
this could use the maven proxy settings in my settings.xml. 

This bug applies to all parameters which expect an equal sign between the 
parameter name and its value. So in the current version this bug should affect 
parameters proxy, proxy_https, jar_splitting_excludes, num_days, severity e 
num_runs.

I also need this fix. 

Fix will be in 1.9.7
For now, you can 
$ git clone https://code.google.com/p/appengine-maven-plugin/
$ cd appengine-maven-plugin
$ mvn install

and use the version 1.9.7-SNAPSHOT in the plugin version as a workaround until 
1.9.7 is pushed