svn commit: r904547 - /jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java

[email protected]
Newsgroups gmane.comp.jakarta.jmeter.devel
Message-ID <[email protected]>
Author: sebb
Date: Fri Jan 29 16:30:50 2010
New Revision: 904547

URL: http://svn.apache.org/viewvc?rev=904547&view=rev
Log:
Disable httpsMatch field unless httpsSpoof is enabled

Modified:
    jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java

Modified: jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java?rev=904547&r1=904546&r2=904547&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java (original)
+++ jakarta/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/proxy/gui/ProxyControlGui.java Fri Jan 29 16:30:50 2010
@@ -130,6 +130,9 @@
      */
     private JCheckBox httpsSpoof;
 
+    /*
+     * Only spoof the URLs that match (optional)
+     */
     private JTextField httpsMatch;
 
     /**
@@ -170,6 +173,7 @@
 
     private static final String RESTART = "restart"; // $NON-NLS-1$
 
+    // This is applied to fields that should cause a restart when changed
     private static final String ENABLE_RESTART = "enable_restart"; // $NON-NLS-1$
 
     private static final String ADD_INCLUDE = "add_include"; // $NON-NLS-1$
@@ -295,6 +299,7 @@
         regexMatch.setSelected(model.getRegexMatch());
         httpsSpoof.setSelected(model.getHttpsSpoof());
         httpsMatch.setText(model.getHttpsSpoofMatch());
+        httpsMatch.setEnabled(httpsSpoof.isSelected()); // Only valid if Spoof is selected
         contentTypeInclude.setText(model.getContentTypeInclude());
         contentTypeExclude.setText(model.getContentTypeExclude());
 
@@ -344,6 +349,7 @@
             startProxy();
         } else if (command.equals(ENABLE_RESTART)){
             enableRestart();
+            httpsMatch.setEnabled(httpsSpoof.isSelected()); // Only valid if Spoof is selected
         } else if (command.equals(ADD_EXCLUDE)) {
             excludeModel.addNewRow();
             excludeModel.fireTableDataChanged();
@@ -501,6 +507,7 @@
         httpsMatch = new JTextField(40);
         httpsMatch.addKeyListener(this);
         httpsMatch.setName(ENABLE_RESTART);
+        httpsMatch.setEnabled(false); // Only valid if Spoof is selected
 
         JLabel matchlabel = new JLabel(JMeterUtils.getResString("proxy_httpsspoofing_match")); // $NON-NLS-1$
         matchlabel.setLabelFor(httpsMatch);
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.