[JBIDE-22596] disable domain validation to cope with *.cdk urls (#1223)

Signed-off-by: Fred Bricon <fbricon@gmail.com>

plugins/org.jboss.tools.openshift.ui/src/org/jboss/tools/openshift/internal/ui/validator/URLValidator.java

@@ -28,7 +28,15 @@ public class URLValidator implements IValidator{
 	private static final String [] SCHEMES = new String [] {"http", "https"};
 	private final String invalidURLMessage;
 	private boolean allowEmpty = false;
-	private UrlValidator validator = new UrlValidator(SCHEMES, UrlValidator.ALLOW_LOCAL_URLS);
+	@SuppressWarnings("serial")
+	private UrlValidator validator = new UrlValidator(SCHEMES, UrlValidator.ALLOW_LOCAL_URLS) {
+		protected boolean isValidAuthority(String domain) {
+			//default implementation doesn't recognize *.cdk domains as valid authorities
+			//so we bypass that check altogether. An alternative would be to provide a RegexValidator, 
+			//but to use what regexp?
+			return true;
+		};
+	};
 
 	/**
 	 * @param urlType  The value to plug into error message of 'Please provide a valid TYPE URL'

tests/org.jboss.tools.openshift.test/src/org/jboss/tools/openshift/test/ui/validator/URLValidatorTest.java

@@ -24,7 +24,13 @@ public class URLValidatorTest {
 	public void testHttpsPassValidURL() {
 		assertEquals(ValidationStatus.ok(), validator.validate("https://foobar"));
 	}
-
+
+	@Test
+	public void testCdkDomainValidURL() {
+		assertEquals(ValidationStatus.ok(), 
+				validator.validate("https://openshift.cdk"));
+	}
+
 	@Test
 	public void testHttpPassValidURL() {
 		assertEquals(ValidationStatus.ok(), validator.validate("http://foobar"));