Return-Path: X-Original-To: apmail-cxf-commits-archive@www.apache.org Delivered-To: apmail-cxf-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 7514F18D67 for ; Wed, 15 Jul 2015 09:01:50 +0000 (UTC) Received: (qmail 69460 invoked by uid 500); 15 Jul 2015 09:01:47 -0000 Delivered-To: apmail-cxf-commits-archive@cxf.apache.org Received: (qmail 69345 invoked by uid 500); 15 Jul 2015 09:01:47 -0000 Mailing-List: contact commits-help@cxf.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cxf.apache.org Delivered-To: mailing list commits@cxf.apache.org Received: (qmail 69308 invoked by uid 99); 15 Jul 2015 09:01:47 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 15 Jul 2015 09:01:47 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 69CCAE0F7B; Wed, 15 Jul 2015 09:01:47 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: sergeyb@apache.org To: commits@cxf.apache.org Message-Id: X-Mailer: ASF-Git Admin Mailer Subject: cxf git commit: [CXF-6498] Updating 2.0 ClientImpl to accept CXF features Date: Wed, 15 Jul 2015 09:01:47 +0000 (UTC) Repository: cxf Updated Branches: refs/heads/3.0.x-fixes 5dbd6cb35 -> 90de43b2c [CXF-6498] Updating 2.0 ClientImpl to accept CXF features Project: http://git-wip-us.apache.org/repos/asf/cxf/repo Commit: http://git-wip-us.apache.org/repos/asf/cxf/commit/90de43b2 Tree: http://git-wip-us.apache.org/repos/asf/cxf/tree/90de43b2 Diff: http://git-wip-us.apache.org/repos/asf/cxf/diff/90de43b2 Branch: refs/heads/3.0.x-fixes Commit: 90de43b2c01375ac8493c61bb00723fe2b17eaa9 Parents: 5dbd6cb Author: Sergey Beryozkin Authored: Wed Jul 15 11:59:52 2015 +0300 Committer: Sergey Beryozkin Committed: Wed Jul 15 12:01:26 2015 +0300 ---------------------------------------------------------------------- .../org/apache/cxf/jaxrs/client/spec/ClientImpl.java | 13 ++++++++++--- .../systest/jaxrs/security/JAXRS20HttpsBookTest.java | 2 ++ 2 files changed, 12 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/cxf/blob/90de43b2/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/spec/ClientImpl.java ---------------------------------------------------------------------- diff --git a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/spec/ClientImpl.java b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/spec/ClientImpl.java index fa68058..b15d730 100644 --- a/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/spec/ClientImpl.java +++ b/rt/rs/client/src/main/java/org/apache/cxf/jaxrs/client/spec/ClientImpl.java @@ -248,9 +248,13 @@ public class ClientImpl implements Client { ClientProviderFactory pf = ClientProviderFactory.getInstance(WebClient.getConfig(targetClient).getEndpoint()); List providers = new LinkedList(); + List cxfFeatures = + new LinkedList(); Configuration cfg = configImpl.getConfiguration(); for (Object p : cfg.getInstances()) { - if (!(p instanceof Feature)) { + if (p instanceof org.apache.cxf.feature.Feature) { + cxfFeatures.add((org.apache.cxf.feature.Feature)p); + } else if (!(p instanceof Feature)) { Map, Integer> contracts = cfg.getContracts(p.getClass()); if (contracts == null || contracts.isEmpty()) { providers.add(p); @@ -277,8 +281,11 @@ public class ClientImpl implements Client { } setConnectionProperties(configProps, clientCfg); - - // start building the invocation + // CXF Features + for (org.apache.cxf.feature.Feature cxfFeature : cxfFeatures) { + cxfFeature.initialize(clientCfg, clientCfg.getBus()); + } + // Start building the invocation return new InvocationBuilderImpl(WebClient.fromClient(targetClient)); } private void setConnectionProperties(Map configProps, ClientConfiguration clientCfg) { http://git-wip-us.apache.org/repos/asf/cxf/blob/90de43b2/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/JAXRS20HttpsBookTest.java ---------------------------------------------------------------------- diff --git a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/JAXRS20HttpsBookTest.java b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/JAXRS20HttpsBookTest.java index 87fdd62..30fee8b 100644 --- a/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/JAXRS20HttpsBookTest.java +++ b/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/security/JAXRS20HttpsBookTest.java @@ -31,6 +31,7 @@ import javax.ws.rs.client.WebTarget; import javax.ws.rs.core.MediaType; import org.apache.cxf.configuration.jsse.TLSClientParameters; +import org.apache.cxf.feature.LoggingFeature; import org.apache.cxf.systest.jaxrs.Book; import org.apache.cxf.testutil.common.AbstractBusClientServerTestBase; import org.apache.cxf.transport.https.CertificateHostnameVerifier; @@ -65,6 +66,7 @@ public class JAXRS20HttpsBookTest extends AbstractBusClientServerTestBase { builder.keyStore(keyStore, "password"); Client client = builder.build(); + client.register(new LoggingFeature()); WebTarget target = client.target("https://localhost:" + PORT + "/bookstore/securebooks/123"); Book b = target.request().accept(MediaType.APPLICATION_XML_TYPE).get(Book.class);