Attachment 82555 Details for Bug 206260 – Updated patch

Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

[patch] Updated patch

206260.txt (text/plain), 6.82 KB, created by Rohit Shetty

on 2007-11-09 13:05:58 EST

(hide)

Description:

Filename:

MIME Type:

Creator: Rohit Shetty

Created: 2007-11-09 13:05:58 EST

Size: 6.82 KB

patch

obsolete

>### Eclipse Workspace Patch 1.0
>#P org.eclipse.tptp.monitoring.logui
>Index: src/org/eclipse/tptp/monitoring/logui/internal/wizards/LogLocationUI.java
>===================================================================
>RCS file: /cvsroot/tptp/monitoring/org.eclipse.tptp.monitoring.logui/src/org/eclipse/tptp/monitoring/logui/internal/wizards/LogLocationUI.java,v
>retrieving revision 1.30
>diff -u -r1.30 LogLocationUI.java
>--- src/org/eclipse/tptp/monitoring/logui/internal/wizards/LogLocationUI.java	30 May 2007 16:18:57 -0000	1.30
>+++ src/org/eclipse/tptp/monitoring/logui/internal/wizards/LogLocationUI.java	9 Nov 2007 18:04:29 -0000
>@@ -453,7 +453,7 @@
> 				if(value!=null){	
> 					((IImportWizardWidget)widget).setValue(value);
> 					if(id.equals("file_path")){
>-						boolean hostEnabled = !(value.startsWith("ftp://") || value.startsWith("http://"));
>+						boolean hostEnabled = !(value.startsWith("ftp://") || value.startsWith("http://") || value.startsWith("https://"));
> 						details.setHostEnabled(hostEnabled);
> 					}
> 										
>@@ -614,7 +614,7 @@
> 			String key = (String) iterator.next();
> 			if(key.equals("file_path")){
> 				String value = (String)values.get(key);
>-				if(value !=  null && (value.startsWith("ftp://") || value.startsWith("http://"))){
>+				if(value !=  null && (value.startsWith("ftp://") || value.startsWith("http://") || value.startsWith("https://"))){
> 					String host = extractHostFromURL(value);
> 					if(host != null){
> 						logElem.setVirtualHost(host);
>@@ -645,7 +645,7 @@
> 			ImportWizardTextField filePathTextField = (ImportWizardTextField)activeComposite.getControlLookup().get("file_path");
> 			if(filePathTextField != null && filePathTextField.getText().equals(e.widget)){
> 				// a modification happened in the text field, verify if the file path is an ftp or http URL. If so, disable host selection and extract the host from the URL 
>-				boolean hostEnabledState = !(((Text)e.widget).getText().startsWith("ftp://") || ((Text)e.widget).getText().startsWith("http://"));
>+				boolean hostEnabledState = !(((Text)e.widget).getText().startsWith("ftp://") || ((Text)e.widget).getText().startsWith("http://") || ((Text)e.widget).getText().startsWith("https://"));
> 				details.setHostEnabled(hostEnabledState);
> 				
> 			}
>Index: src/org/eclipse/tptp/monitoring/logui/internal/wizards/ImportLogWizard.java
>===================================================================
>RCS file: /cvsroot/tptp/monitoring/org.eclipse.tptp.monitoring.logui/src/org/eclipse/tptp/monitoring/logui/internal/wizards/ImportLogWizard.java,v
>retrieving revision 1.105
>diff -u -r1.105 ImportLogWizard.java
>--- src/org/eclipse/tptp/monitoring/logui/internal/wizards/ImportLogWizard.java	26 Aug 2007 19:44:59 -0000	1.105
>+++ src/org/eclipse/tptp/monitoring/logui/internal/wizards/ImportLogWizard.java	9 Nov 2007 18:04:28 -0000
>@@ -1996,7 +1996,7 @@
> 		}
> 		LocalLogImportJob job = new LocalLogImportJob(LogUIMessages._7, handler, element);
> 		final String filePath = (String) element.getValues().get("file_path");
>-		if (filePath != null && (filePath.startsWith("ftp://") || filePath.startsWith("http://"))) {
>+		if (filePath != null && (filePath.startsWith("ftp://") || filePath.startsWith("http://")  || filePath.startsWith("https://"))) {
> 			Authenticator authenticator = new Authenticator(){
> 				ImportPasswordAuthentication importLogAuthentication = null;
> 				
>#P org.eclipse.tptp.monitoring.la.core
>Index: cbe.import/org/eclipse/tptp/monitoring/log/internal/core/CBEServiceFactoryImpl.java
>===================================================================
>RCS file: /cvsroot/tptp/monitoring/org.eclipse.tptp.monitoring.la.core/cbe.import/org/eclipse/tptp/monitoring/log/internal/core/CBEServiceFactoryImpl.java,v
>retrieving revision 1.2
>diff -u -r1.2 CBEServiceFactoryImpl.java
>--- cbe.import/org/eclipse/tptp/monitoring/log/internal/core/CBEServiceFactoryImpl.java	18 Sep 2006 19:37:05 -0000	1.2
>+++ cbe.import/org/eclipse/tptp/monitoring/log/internal/core/CBEServiceFactoryImpl.java	9 Nov 2007 18:04:33 -0000
>@@ -23,7 +23,7 @@
> 		IImportHandler handler = null;
> 		if (element.isLocalhost()){
> 			final String filePath = (String) element.getValues().get("file_path");
>-			if (filePath != null && (filePath.startsWith("ftp://") || filePath.startsWith("http://"))) {
>+			if (filePath != null && (filePath.startsWith("ftp://") || filePath.startsWith("http://") || filePath.startsWith("https://"))) {
> 				handler =  new LocalDownloadImportHandler(element ,context);				
> 			}
> 			else{
>Index: cbe.import/org/eclipse/tptp/monitoring/log/internal/core/LocalDownloadImportHandler.java
>===================================================================
>RCS file: /cvsroot/tptp/monitoring/org.eclipse.tptp.monitoring.la.core/cbe.import/org/eclipse/tptp/monitoring/log/internal/core/LocalDownloadImportHandler.java,v
>retrieving revision 1.4
>diff -u -r1.4 LocalDownloadImportHandler.java
>--- cbe.import/org/eclipse/tptp/monitoring/log/internal/core/LocalDownloadImportHandler.java	1 Mar 2007 17:13:15 -0000	1.4
>+++ cbe.import/org/eclipse/tptp/monitoring/log/internal/core/LocalDownloadImportHandler.java	9 Nov 2007 18:04:33 -0000
>@@ -19,6 +19,7 @@
> import java.io.InputStream;
> import java.io.OutputStream;
> import java.net.Authenticator;
>+import java.net.HttpURLConnection;
> import java.net.URL;
> import java.net.URLConnection;
> import java.util.Date;
>@@ -53,7 +54,7 @@
> 		String destPath = (String)context.getProperty(IImportHandler.TEMP_DIRECTORY);
> 		final String fileURL = (String)element.getValues().get("file_path"); //$NON-NLS-1$
> 		
>-		//**Need to authenticate **//
>+		//**Need to authenticate **///
> 		monitor.beginTask(element.getAgentName(), IProgressMonitor.UNKNOWN);
> 		monitor.subTask(" "+StringUtil.replace(Messages.getString("LocalDownloadImportHandler.2", locale),"{0}", fileURL)); //$NON-NLS-1$ //$NON-NLS-2$
> 		Authenticator.setDefault (authenticator);
>@@ -61,8 +62,28 @@
> 		File file;			
> 		String key = GLAHelper.getAdapterPathKey(parserInput);
> 		try {
>-			
>+ 
> 			URLConnection connection = (new URL(fileURL)).openConnection();
>+			//fix for bugzilla 206260
>+			if(connection instanceof HttpURLConnection)
>+			{
>+				// Redirect a maximum of five times. This is a generally accepted check which makes sure that
>+				// multiple redirects are supported, but this is not infinite.
>+				for(int i =0; i < 5;i++)
>+				{
>+					int retCode = ((HttpURLConnection)connection).getResponseCode();
>+					if( retCode >= 300 && retCode < 400)
>+					{
>+						String location = connection.getHeaderField("location");
>+						connection = (new URL(location)).openConnection();
>+					}
>+					else
>+					{
>+						break;
>+					}
>+				}				
>+			}
>+			//end of fix for bugzilla 206260
> 			InputStream inputStream = connection.getInputStream();
> 			SimpleDateFormat format = new SimpleDateFormat("yyyymmddhhMMss");   //$NON-NLS-1$
>

Actions: View | Diff

Attachments on bug 206260: 82417 | 82555 | 82560 | 82621 | 82640 | 83937 | 83938