Add Apache HttpClient integration #1455

zhaoyuguang · 2020-05-06T02:19:38Z

resolve #1436 not include AsyncHttpClient

sczyh30 · 2020-05-15T03:07:15Z

Any progress?

zhaoyuguang · 2020-05-15T03:29:55Z

Any progress?

Follow-up, may be next week

codecov-commenter · 2020-05-29T07:31:59Z

Codecov Report

Merging #1455 into master will increase coverage by 0.02%.
The diff coverage is 58.33%.

@@             Coverage Diff              @@
##             master    #1455      +/-   ##
============================================
+ Coverage     44.03%   44.05%   +0.02%     
- Complexity     1728     1736       +8     
============================================
  Files           376      380       +4     
  Lines         10682    10718      +36     
  Branches       1425     1427       +2     
============================================
+ Hits           4704     4722      +18     
- Misses         5409     5424      +15     
- Partials        569      572       +3

Impacted Files	Coverage Δ	Complexity Δ
...pclient/config/SentinelApacheHttpClientConfig.java	`50.00% <50.00%> (ø)`	`4.00 <4.00> (?)`
...he/httpclient/SentinelApacheHttpClientBuilder.java	`56.25% <56.25%> (ø)`	`2.00 <2.00> (?)`
...ctor/DefaultApacheHttpClientResourceExtractor.java	`100.00% <100.00%> (ø)`	`2.00 <2.00> (?)`
...ient/fallback/DefaultApacheHttpClientFallback.java	`100.00% <100.00%> (ø)`	`2.00 <2.00> (?)`
...a/csp/sentinel/slots/statistic/base/LeapArray.java	`67.32% <0.00%> (-2.98%)`	`33.00% <0.00%> (-1.00%)`

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 6b86721...4501ae8. Read the comment docs.

zhaoyuguang · 2020-05-29T10:24:46Z

@sczyh30 Friend ping ready for review

jasonjoo2010 · 2020-06-11T02:06:25Z

sentinel-adapter/sentinel-apache-httpclient-adapter/README.md

+
+### extractor (resource extractor)
+
+We can define `ApacheHttpClientResourceExtractor` to custom resource extractor replace `DefaultApacheHttpClientResourceExtractor`, for example: httpclient:GET:/httpclient/back/1 ==> httpclient:GET:/httpclient/back/{id}


custom should be customize

sentinel-adapter/sentinel-apache-httpclient-adapter/README.md

jasonjoo2010 · 2020-06-11T02:12:38Z

sentinel-adapter/sentinel-apache-httpclient-adapter/README.md

+
+    @Override
+    public String extractor(String method, String uri, HttpRequestWrapper request) {
+        String regex = "/httpclient/back/";


It's not a regular expression but a PREFIX. Better to fix the naming and replace contains with startsWith.

jasonjoo2010 · 2020-06-11T02:33:09Z

...baba/csp/sentinel/adapter/apache/httpclient/extractor/ApacheHttpClientResourceExtractor.java

+ */
+public interface ApacheHttpClientResourceExtractor {
+
+    String extractor(String method, String uri, HttpRequestWrapper request);


In my opinion both parameter #1 and #2 has been wrapped in parameter #3 they are unnecessary.

The first & two parameters are removed

jasonjoo2010 · 2020-06-11T02:44:11Z

...om/alibaba/csp/sentinel/adapter/apache/httpclient/config/SentinelApacheHttpClientConfig.java

+/**
+ * @author zhaoyuguang
+ */
+public final class SentinelApacheHttpClientConfig {


The design for additional configuration is quite straightforward. We can't have different configurations between different clients in this design.

Adapters should follow the ways what they are adapted to follow. HttpClient way is that we can do kinds of decoration, configurations of sockets / interpreters / connection manager / etc. , before we generate the client instance while the client will focus on executing requests. So we should follow this design instead of global shared configuration. It should be extended as new methods of SentinelApacheHttpClientBuilder.

Converted to an internal property of the Builder

jasonjoo2010 · 2020-06-11T02:45:05Z

...om/alibaba/csp/sentinel/adapter/apache/httpclient/config/SentinelApacheHttpClientConfig.java

+
+    private static volatile String prefix = "httpclient:";
+    private static volatile ApacheHttpClientResourceExtractor extractor = new DefaultApacheHttpClientResourceExtractor();
+    private static volatile ApacheHttpClientFallback fallback = new DefaultApacheHttpClientFallback();


And surely we will not need the default ones if we implement it as per instance configuration.

Converted to an internal property of the Builder

sentinel-adapter/sentinel-apache-httpclient-adapter/README.md

jasonjoo2010 · 2020-06-11T02:50:16Z

...p/sentinel/adapter/apache/httpclient/extractor/DefaultApacheHttpClientResourceExtractor.java

+
+    @Override
+    public String extractor(String method, String uri, HttpRequestWrapper request) {
+        return method + ":" + uri;


As the default implementation i can't quite agree with including METHOD in resource name.
Please go for advices from other reviewers, too.

method has remove

jasonjoo2010 · 2020-06-11T02:52:43Z

...alibaba/csp/sentinel/adapter/apache/httpclient/fallback/DefaultApacheHttpClientFallback.java

+    @Override
+    public CloseableHttpResponse handle(HttpRequestWrapper request, BlockException e) {
+        // Just wrap and throw the exception.
+        throw new SentinelRpcException(e);


I suggest that Exception propagation can be implemented in the customized chain directly.

CLAassistant · 2020-06-12T07:21:16Z

All committers have signed the CLA.

zhaoyuguang · 2020-07-08T07:52:53Z

I will push forward and complete this PR as soon as possible

jasonjoo2010

Nice work. LGTM.

Remained some small pieces to be optimized in future like blocking dealing, fine unit tests especially when failed.

jasonjoo2010 · 2020-07-09T06:52:09Z

Thanks for your contribution!

sczyh30 · 2020-07-09T06:54:23Z

👍 Nice work. We may find ways to deal with AsyncHttpClient later :)

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

@deprecated

* calculate process cpu usage to support application running in container environment * Enhance reliability and performance of InMemoryMetricsRepository (alibaba#1319) * Fix InMemoryMetricsRepository can't keep the last five minutes metrics data problem and Improve read-write performance * Use TimeUtil.currentTimeMillis() replace System.currentTimeMillis() for better performance * dashboard: Support setting value pattern for client IP and host in gateway flow rule dialog (alibaba#1325) * doc: Update JDK requirement of the dashboard in README.md (alibaba#1316) * Following discussions in alibaba#1315 * Add "web-context-unify" config in Spring WebMVC adapter to support "chain" relation flow strategy (alibaba#1328) * Fix the parsing issue in large post request for sentinel-transport-simple-http (alibaba#1255) * Add gateway adapter for Zuul 2.x (alibaba#1138) - also add demo for Zuul 2.x adapter * Polish code and demo of Sentinel Zuul 2.x adapter Signed-off-by: Eric Zhao <[email protected]> * Polish code of transport command centers and heartbeat senders Signed-off-by: Eric Zhao <[email protected]> * Polish logging SPI related code and add general JUL adapter for Logger SPI (alibaba#1338) * Move the legacy JUL methods from LogBase to BaseJulLogger. * Add a JavaLoggingAdapter as the general JUL adapter for the Logger SPI, which makes it convenient to use (as the default logger). * Add LoggerSpiProvider to resolve Logger SPI. * Polish the logback demo. Signed-off-by: Eric Zhao <[email protected]> * Move CommandCenterLog to sentinel-transport-common and polish related code (alibaba#1341) Signed-off-by: Eric Zhao <[email protected]> * Polish placeholders in logging content to slf4j convention (alibaba#1342) * Polish placeholders in logging content to "{}" Signed-off-by: Eric Zhao <[email protected]> * Fix timezone problem of sentinel-block.log Signed-off-by: Eric Zhao <[email protected]> * dashboard: Fix NoNodeException problem of FlowRuleZookeeperProvider example (alibaba#1352) * Introduce logging extension: slf4j (alibaba#1344) * Regroup packages related to logging into a separate module and polish document (alibaba#1355) * Fix CI failure in JDK 11 environment (alibaba#1360) * Supplement missing javax.annotation-api to sentinel-cluster-server-envoy-rls and sentinel-demo-zuul2-gateway * Upgrade mockito-core to 2.28.2 (up to date) * Fix the bug that context was not released when blocked in Spring Web adapter (alibaba#1353) * Improve standard output message in LogBase (alibaba#1357) * Complete the unit tests for sentinel-logging-slf4j module (alibaba#1358) * refactor: Make the ProcessorSlot itself as SPI and deprecate legacy slot chain builder (alibaba#411) * Make slots loaded by SPI, mark all slots with @SpiOrder from -10000 to -1000, improve comment * Reserve gateway and param slot chain builder (just extends DefaultSlotChainBuilder) and mark them as @deprecated * Force modifyRule command handler to fail if an incompatible old fastjson found (alibaba#1377) * Note that this is only a temporary solution. * Set default log level of JDK logging to INFO and polish code of SpiLoader (alibaba#1365) * Improve log info in SpiLoader, improve comment and test case * Use error level in catch block, init ArrayList with capacity and improve add item to list * doc: Polish README.md of sentinel-cluster-server-envoy-rls module Signed-off-by: Eric Zhao <[email protected]> * dashboard: Hide advanced options in flow rule dialog when cluster mode is enabled (alibaba#1367) * doc: Update README.md Signed-off-by: Eric Zhao <[email protected]> * Update fastjson to 1.2.68 Signed-off-by: Eric Zhao <[email protected]> * Bump version to 1.7.2 Signed-off-by: Eric Zhao <[email protected]> * Fix the bug of extracting request cookie in Spring Cloud Gateway adapter (alibaba#1400) * Bump version to 1.8.0-SNAPSHOT Signed-off-by: Eric Zhao <[email protected]> * Add JAX-RS adapter (alibaba#1396) * Fix the bug of misplaced locks in ContextUtil and ClusterNode (alibaba#1429) - which may lead to IllegalMonitorStateException in unlock() when unchecked error occurs during lock() * fix: Tracer does not trace exception to DefaultNode (alibaba#1068) * Support setting project.name via the properties file and deprecate legacy config path (alibaba#1412) * Update resolving logic of project name and polish SentinelConfig (alibaba#1437) Signed-off-by: Eric Zhao <[email protected]> * Refactor the mechanism of recording error in Entry and StatisticSlot * Also polish related complete callbacks Signed-off-by: Eric Zhao <[email protected]> * Polish Tracer with entry.setError(ex) mechanism Signed-off-by: Eric Zhao <[email protected]> * dashboard: Remove duplicate code in MetricEntity (alibaba#1441) * dashboard: Fix the bug that cookie may have conflict with web applications under the same domain (alibaba#1443) * Improve deprecated ParameterMetric purge mechanism (alibaba#1372) * Clear useless data in ParameterMetric for all removed rules * Polish code comments of the fundamental Sph/SphO/SphU class Signed-off-by: Eric Zhao <[email protected]> * Add OkHttp integration (alibaba#1456) * dashboard: Fix historical version compatibility problem for auth check via localStorage (alibaba#1473) * Add exceptionPredicate in Tracer for customized exception filtering logic (alibaba#1496) * test: Add test cases for Tuple2 (alibaba#1501) Signed-off-by: yunfeiyanggzq <[email protected]> * Add support for extracting param from complex object (alibaba#1491) * This could enable specified parameter flow control for customized objects. * Support setting class-level defaultFallback for annotation extension (alibaba#1493) * Add unit test for logging/TokenBucket (alibaba#1504) Signed-off-by: yunfeiyanggzq <[email protected]> * Fix sentinel-apache-dubbo-adapter full GC bug (alibaba#1431) * Polish RocketMQ PullConsumerDemo to make code clear (alibaba#1528) * Add unit test for cluster/FlowResponseDataDecoder (alibaba#1514) Signed-off-by: yunfeiyanggzq <[email protected]> * Improve consumer filter of Dubbo 2.6.x and 2.7.x adapter (alibaba#1532) * entry and exit with params in consumer filter * Polish sentinel-opensource-eco-landscape-en.png Signed-off-by: Eric Zhao <[email protected]> * Add annotation extension for Java EE CDI (alibaba#1541) * Add Sentinel annotation and JAX-RS plugins for Quarkus (alibaba#1542) * Add sentinel-quarkus-adapter module, which provides sentinel-annotation-quarkus-adapter and sentinel-jax-rs-quarkus-adapter to adapt sentinel-annotation-cdi-interceptor and sentinel-jax-rs-adapter for Quarkus. It also provides sentinel-native-image-quarkus-adapter to support running Sentinel with Quarkus in native image mode. * Polish document and code of Sentinel annotation CDI extension Signed-off-by: Eric Zhao <[email protected]> * Upgrade fastjson to 1.2.71 (alibaba#1545) * Add file.encoding JVM args in maven-surefire-plugin to avoid charset problem (alibaba#1550) * Add annotation CDI demo and Quarkus adapter demo (alibaba#1543) * Polish document and rearrange package for Quarkus and JAX-RS adapter Signed-off-by: Eric Zhao <[email protected]> * Support customized origin parser in legacy Dubbo 2.6.x adapter (alibaba#1555) * Add Eureka data-source extension (alibaba#1502) * Upgrade nacos-client version to 1.3.0 in sentinel-datasource-nacos (alibaba#1576) * demo: Update slot chain SPI demo (alibaba#1581) Signed-off-by: yunfeiyanggzq <[email protected]> * Add explicit null checking for charset in SimpleHttpClient#encodeRequestParams (alibaba#1589) * Adapter: Support Apache HttpClient (alibaba#1455) Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`. * doc: Fix mistakes in README.md of sentinel-zuul-adapter (alibaba#1593) * Fix incorrect protocol description in FlowRequestData writer/decoder (alibaba#1607) Signed-off-by: yunfeiyanggzq <[email protected]> * Refactor config mechanism for OkHttp adapter and polish related code - One config per interceptor instead of the global config - Polish document and demo Signed-off-by: Eric Zhao <[email protected]> * test: Add unit test for sentinel-cluster-server and polish code (alibaba#1529) Signed-off-by: yunfeiyanggzq <[email protected]> * Refactor degrade hierarchy with new circuit breaker mechanism and improve strategy * Add `CircuitBreaker` abstraction (with half-open state) and add circuit breaker state change event observer support. * Improve circuit breaking strategy (avg RT → slow request ratio) and make statistics of each rule dependent (to support arbitrary statistic interval). * Add simple "trial" mechanism (aka. half-open). * Refactor mechanism of metric recording and state change handling for circuit breakers: record RT and error when requests have completed (i.e. `onExit`, based on alibaba#1420). Signed-off-by: Eric Zhao <[email protected]> * Update test cases for circuit breaking Signed-off-by: Eric Zhao <[email protected]> * Update demo for circuit breaking (DegradeRule) Signed-off-by: Eric Zhao <[email protected]> * test: Update test cases with new degrade mechanism in sentinel-demo-quarkus Signed-off-by: Eric Zhao <[email protected]> * Remove deprecated passCheck() in Rule and polish interface Signed-off-by: Eric Zhao <[email protected]> * Polish cluster flow control demo: add port in Nacos address (alibaba#1655) Signed-off-by: yunfeiyanggzq <[email protected]> * Polish boolean checking in test cases and dashboard (alibaba#1664) * Remove unused code in TokenServerHandler#channelActive (alibaba#1667) Signed-off-by: cj <[email protected]> * Optimize the order of slots in ProcessorSlot SPI config (alibaba#1649) * Fix the bug of circuit breaker half-open state transformation when request is blocked by upcoming rules (alibaba#1645) * Refactor the workflow to fix the bug that circuit breaker may remain half-open state forever when the request is blocked by upcoming rules: revert the state change in exit handler (as a temporary workaround) * Add exit handler in Entry as a per-invocation hook. * Polish CircuitBreaker interface and update comments - Only carry context in tryPass/onComplete method (this might be generic in upcoming versions) Signed-off-by: Eric Zhao <[email protected]> * Refactor exit handler mechanism of Entry - Rename: whenComplete -> whenTerminate - Execute the exit handler directly after the onExit hook of all slots Signed-off-by: Eric Zhao <[email protected]> * Add extended interface for metric extension hook to support distinguishing traffic type (alibaba#1665) - Add EntryType args to all hook methods * dashboard: Refactor degrade service/controller and adapt to new features Signed-off-by: Eric Zhao <[email protected]> * Polish Dubbo 2.6.x adapter and unify callback registry into DubboAdapterGlobalConfig (alibaba#1572) * Unify Dubbo callback registry (for fallback and origin parser) into DubboAdapterGlobalConfig * Polish default fallback implementation (wrap exception with RpcResult rather than directly throw it out) Signed-off-by: Eric Zhao <[email protected]> * Refactor extended MetricExtension interface (matching events in Sentinel) - Unify the extended interface as a few event handlers: onPass, onBlocked, onComplete and onError - Polish related code Signed-off-by: Eric Zhao <[email protected]> * Support customized origin parser in Apache Dubbo 2.7.x adapter and unify config (alibaba#1617) * Support customized origin parser in Apache Dubbo 2.7.x adapter * Unify Dubbo callback registry (for fallback and origin parser) into DubboAdapterGlobalConfig * Polish default fallback implementation (wrap exception with RpcResult rather than directly throw it out) * Polish code and README.md of sentinel-datasource-eureka Signed-off-by: Eric Zhao <[email protected]> * webmvc-adapter: improve to avoid ErrorEntryFreeException (alibaba#1533) If entry already exists in request just skip creation. * adapter: Add test cases for Spring WebFlux HandlerFunction (alibaba#1678) * Add RuntimeException converting method in BlockException and polish logic for validation Signed-off-by: Eric Zhao <[email protected]> * Fix NPE bug and improve default fallback in Dubbo 2.7.x adapter - Fix NPE bug in consumer filter (when non-biz error occurred) - Improve default fallback in Dubbo 2.7.x adapter: convert the BlockException to a simple RuntimeException (with necessary message) - Polish code and comments Signed-off-by: Eric Zhao <[email protected]> * Improve compatibility for dispatched servlet request in Spring Web adapter (alibaba#1681) * Bump version to 1.8.0 Signed-off-by: Eric Zhao <[email protected]> * Update README.md Signed-off-by: Eric Zhao <[email protected]> * Bump version to 1.8.1-SNAPSHOT Signed-off-by: Eric Zhao <[email protected]> * Fix typo in CircuitBreakingIntegrationTest (alibaba#1688) Signed-off-by: yunfeiyanggzq <[email protected]> * Pre-calculate intervalInSecond in LeapArray to reduce redundant calculation (alibaba#1700) * doc: Fix typo in code comments (alibaba#1721) * Solve the URI fetching bug in sentinel-zuul-adapter alibaba#1109 (alibaba#1605) Use `getRequestURI` instead of `getServletPath` to get URI of current request(Both in prefix and regex matching). * Fix NPE bug when updating gateway flow rules before the route/API has been requested once (alibaba#1729) * Make NettyTransportClient.getCurrentId() thread safe (alibaba#1707) Fix issue alibaba#1705. - Use CAS to make it thread safe and limited in the declared range. Signed-off-by: cj <[email protected]> * Add attributes of cluster concurrency limiting in ClusterFlowConfig Signed-off-by: yunfeiyanggzq <[email protected]> * Add concurrency token request/release operation in TokenService Signed-off-by: yunfeiyanggzq <[email protected]> * Add basic cluster concurrency limiting impl in token server module Signed-off-by: yunfeiyanggzq <[email protected]> * Add unit tests for cluster concurrent limiting checker Signed-off-by: yunfeiyanggzq <[email protected]> * doc: Fix content in README.md of sentinel-dashboard (alibaba#1737) * Fix the dependency conflict issue * Optimize logging statements using placeholder (alibaba#1736) * Optimize logging statements using placeholder to avoid unnecessary concatenation (issue alibaba#1735) * Polish document and name of parameter (alibaba#1738) - doc: Fix a typo in description of booting options for sentinel-dashboard - Fix mismatched name of parameter to its comment for VersionUtils.parseVersion() * Fix potential concurrency issue when updating flow rules (alibaba#1783) * test: Fix overrunning test `FlowRuleManagerTest.testLoadAndGetRules` (alibaba#1823) Signed-off-by: Jason Joo <[email protected]> * Improve default block fallback logic in Dubbo 2.6.x adapter to avoid serialization problem (alibaba#1794) - convert BlockException to a simple RuntimeException (with necessary message) * Fix the problem that requests will never be blocked when slowRatioThreshold = 100% (alibaba#1779) * CI: Polish dependencies for ARM64 and add ARM64 job to Travis CI (alibaba#1765) 1. Added ARM64 architecture in .travis.yml 2. Updated 'embedded-consul' version to 2.2.0, 'consul-api' version to 1.4.5 for ARM64 support. 3. Updated grpc.version for 'io.grpc:protoc-gen-grpc-java' to 1.30.2, for ARM64 support. Signed-off-by: odidev <[email protected]> * [feat alibaba#1839]: Make dashboard support deploying under subpath (alibaba#1851) * dashboard: Add statIntervalMs field in DegradeRule dialog (alibaba#1781) Co-authored-by: tianhao <[email protected]> Co-authored-by: jy2156121 <[email protected]> Co-authored-by: Olof <[email protected]> Co-authored-by: cdfive <[email protected]> Co-authored-by: Jason Joo <[email protected]> Co-authored-by: tao.zhang <[email protected]> Co-authored-by: Eric Zhao <[email protected]> Co-authored-by: WongTheo <[email protected]> Co-authored-by: 于玉桔 <[email protected]> Co-authored-by: Zhiguo.Chen <[email protected]> Co-authored-by: seasidesky <[email protected]> Co-authored-by: haifeng <[email protected]> Co-authored-by: johnli <[email protected]> Co-authored-by: zhenxianyimeng <[email protected]> Co-authored-by: pleasecheckhere2016 <[email protected]> Co-authored-by: ZhiQiang Gu <[email protected]> Co-authored-by: zechao zheng <[email protected]> Co-authored-by: yangy <[email protected]> Co-authored-by: xiby <[email protected]> Co-authored-by: iron_city <[email protected]> Co-authored-by: Bo <[email protected]> Co-authored-by: HupJ <[email protected]> Co-authored-by: Peine <[email protected]> Co-authored-by: cj <[email protected]> Co-authored-by: Bill Yip <[email protected]> Co-authored-by: liqiangz <[email protected]> Co-authored-by: mikawudi <[email protected]> Co-authored-by: dani3lWong <[email protected]> Co-authored-by: cj <[email protected]> Co-authored-by: yunfeiyanggzq <[email protected]> Co-authored-by: Luke <[email protected]> Co-authored-by: HelloCoCooo <[email protected]> Co-authored-by: nickChenyx <[email protected]> Co-authored-by: Weihua <[email protected]> Co-authored-by: 王振广 <[email protected]> Co-authored-by: Lynx <[email protected]> Co-authored-by: odidev <[email protected]> Co-authored-by: Brent <[email protected]>

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

wip

1fa15da

sczyh30 added the area/integrations Issues or PRs related to integrations with open-source components label May 6, 2020

cdfive added the to-review To review label May 6, 2020

Merge branch 'zhaoyuguang_24' into zhaoyuguang_019

1022fbe

fix

acb80f7

zhaoyuguang marked this pull request as ready for review May 29, 2020 10:15

add md

4501ae8

sczyh30 requested review from sczyh30, cdfive, jasonjoo2010 and a team June 5, 2020 05:38

jasonjoo2010 requested changes Jun 11, 2020

View reviewed changes

zhaoyuguang added 2 commits July 9, 2020 12:05

fix

068dcd8

fix md

24f09bf

jasonjoo2010 approved these changes Jul 9, 2020

View reviewed changes

jasonjoo2010 merged commit 38911c5 into alibaba:master Jul 9, 2020

sczyh30 removed the to-review To review label Jul 9, 2020

sczyh30 added this to the 1.8.0 milestone Jul 9, 2020

mastertiller pushed a commit to mastertiller/Sentinel that referenced this pull request Aug 20, 2020

Adapter: Support Apache HttpClient (alibaba#1455)

ee0a5fb

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

mastertiller pushed a commit to mastertiller/Sentinel that referenced this pull request Aug 20, 2020

Adapter: Support Apache HttpClient (alibaba#1455)

55914dd

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

mastertiller pushed a commit to mastertiller/Sentinel that referenced this pull request Aug 20, 2020

Adapter: Support Apache HttpClient (alibaba#1455)

ec6cb01

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

hughpearse pushed a commit to hughpearse/Sentinel that referenced this pull request Jun 2, 2021

Adapter: Support Apache HttpClient (alibaba#1455)

eab9c7b

Introduce support through a customized client builder `SentinelApacheHttpClientBuilder`.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add Apache HttpClient integration #1455

Add Apache HttpClient integration #1455

zhaoyuguang commented May 6, 2020 •

edited

Loading

sczyh30 commented May 15, 2020

zhaoyuguang commented May 15, 2020

codecov-commenter commented May 29, 2020 •

edited

Loading

zhaoyuguang commented May 29, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

zhaoyuguang Jul 9, 2020

jasonjoo2010 Jun 11, 2020

CLAassistant commented Jun 12, 2020 •

edited

Loading

zhaoyuguang commented Jul 8, 2020

jasonjoo2010 left a comment

jasonjoo2010 commented Jul 9, 2020

sczyh30 commented Jul 9, 2020


		### extractor (resource extractor)

		We can define `ApacheHttpClientResourceExtractor` to custom resource extractor replace `DefaultApacheHttpClientResourceExtractor`, for example: httpclient:GET:/httpclient/back/1 ==> httpclient:GET:/httpclient/back/{id}

Add Apache HttpClient integration #1455

Add Apache HttpClient integration #1455

Conversation

zhaoyuguang commented May 6, 2020 • edited Loading

sczyh30 commented May 15, 2020

zhaoyuguang commented May 15, 2020

codecov-commenter commented May 29, 2020 • edited Loading

Codecov Report

zhaoyuguang commented May 29, 2020

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

CLAassistant commented Jun 12, 2020 • edited Loading

zhaoyuguang commented Jul 8, 2020

jasonjoo2010 left a comment

Choose a reason for hiding this comment

jasonjoo2010 commented Jul 9, 2020

sczyh30 commented Jul 9, 2020

zhaoyuguang commented May 6, 2020 •

edited

Loading

codecov-commenter commented May 29, 2020 •

edited

Loading

CLAassistant commented Jun 12, 2020 •

edited

Loading