Andre Dietisheim created JBIDE-13114: ---------------------------------------- Summary: openshift-java-client: improve how userAgent is passed to UrlConnectionClient Key: JBIDE-13114 URL: https://issues.jboss.org/browse/JBIDE-13114 Project: Tools (JBoss Tools) Issue Type: Enhancement Affects Versions: 4.0.0.CR1 Reporter: Andre Dietisheim Assignee: Andre Dietisheim Fix For: 4.1.x clarify how the user-agent is set to the UrlConnectionHttpClient. You can actually set it to the constructor but also by a setter: {code:title=com.openshift.internal.client.httpclient.UrlConnectionHttpClient} public UrlConnectionHttpClient(String username, String password, String userAgent, boolean sslChecks, IMediaType requestMediaType, String acceptedMediaType, String version, String authKey, String authIV) { {code} {code:title=com.openshift.internal.client.httpclient.UrlConnectionHttpClient} public void setUserAgent(String userAgent) { this.userAgent = userAgent; } {code} Both variants are actually used in the library code: {code:title=com.openshift.internal.client.RestService} public RestService(String baseUrl, String clientId, IHttpClient client) { this.baseUrl = UrlUtils.ensureStartsWithHttps(baseUrl); this.client = client; client.setUserAgent(new RestServiceProperties().getUseragent(clientId)); {code} {code:title=com.openshift.client.OpenShiftConnectionFactory} public IOpenShiftConnection getConnection(final String clientId, final String login, final String password, final String authKey, final String authIV, final String serverUrl, final boolean proxySet, final String proxyHost, final String proxyPort) throws OpenShiftException { Assert.notNull(clientId); Assert.notNull(login); Assert.notNull(password); Assert.notNull(serverUrl); if (proxySet) { System.setProperty("proxyHost", proxyHost); System.setProperty("proxyPort", proxyPort); } try { final IHttpClient httpClient = new UrlConnectionHttpClientBuilder().setCredentials(login, password, authKey, authIV) .client(); final IRestService service = new RestService(serverUrl, clientId, httpClient); return getConnection(service, login, password); } catch (FileNotFoundException e) { throw new OpenShiftException(e, "Failed to establish connection for user ''{0}}''", login); } catch (IOException e) { throw new OpenShiftException(e, "Failed to establish connection for user ''{0}}''", login); } } {code} -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira