From f69b0fe7639c0003bcadfed7893e5f06f221a2ac Mon Sep 17 00:00:00 2001 From: Simone Bordet Date: Mon, 17 Jul 2023 16:12:19 +0200 Subject: [PATCH] Fixes #10086 - Revisiting ProxyConfiguration.getProxies() Removed "for removal" deprecation, added clarifying javadocs. Signed-off-by: Simone Bordet --- .../org/eclipse/jetty/client/ProxyConfiguration.java | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/jetty-client/src/main/java/org/eclipse/jetty/client/ProxyConfiguration.java b/jetty-client/src/main/java/org/eclipse/jetty/client/ProxyConfiguration.java index b80d6265725d..ca66deec62e8 100644 --- a/jetty-client/src/main/java/org/eclipse/jetty/client/ProxyConfiguration.java +++ b/jetty-client/src/main/java/org/eclipse/jetty/client/ProxyConfiguration.java @@ -41,10 +41,18 @@ public class ProxyConfiguration private final List proxies = new BlockingArrayQueue<>(); /** + *

Returns the list of proxies added to this configuration.

+ *

This method is deprecated because it has historically been + * used to add/remove proxies directly to/from the returned list, + * but this has now been replaced by {@link #addProxy(Proxy)} and + * {@link #removeProxy(Proxy)} instead.

+ *

In the future, this method will return an unmodifiable + * list of proxies just for querying purposes.

+ * * @deprecated use {@link #addProxy(Proxy)} and {@link #removeProxy(Proxy)} instead * @return the forward proxies to use */ - @Deprecated(forRemoval = true) + @Deprecated public List getProxies() { return proxies;