Further implement http-netty4 package (Azure#44890)

Further implement http-netty4 package

Author: alzimmermsft

.vscode/cspell.json

@@ -203,7 +203,7 @@
     "sdk/devcenter/azure-developer-devcenter/**",
     "sdk/loadtesting/azure-developer-loadtesting/**",
     "sdk/clientcore/core/**",
-    "sdk/clientcore/http-netty/**",
+    "sdk/clientcore/http-netty4/**",
     "sdk/clientcore/http-okhttp3/**",
     "sdk/clientcore/annotation-processor/**",
     "sdk/clientcore/annotation-processor-test/**",

eng/.docsettings.yml

@@ -98,7 +98,7 @@ known_content_issues:
   - ['sdk/core/README.md', '#3113']
   - ['sdk/clientcore/README.md', '#3113']
   - ['sdk/clientcore/core/README.md', '#3113']
-  - ['sdk/clientcore/http-netty/README.md', '#3113']
+  - ['sdk/clientcore/http-netty4/README.md', '#3113']
   - ['sdk/clientcore/http-okhttp3/README.md', '#3113']
   - ['sdk/clientcore/tools/annotation-processor/README.md', '#3113']
   - ['sdk/clientcore/optional-dependency-tests/README.md', '#3113']

eng/versioning/version_client.txt

@@ -481,13 +481,13 @@ com.azure.v2:azure-core-test;2.0.0-beta.1;2.0.0-beta.1
 com.azure.v2:azure-identity;2.0.0-beta.1;2.0.0-beta.1
 io.clientcore:clientcore-parent;1.0.0-beta.3;1.0.0-beta.3
 io.clientcore:core;1.0.0-beta.8;1.0.0-beta.9
+io.clientcore:http-netty4;1.0.0-beta.1;1.0.0-beta.1
 io.clientcore:http-okhttp3;1.0.0-beta.2;1.0.0-beta.3
 io.clientcore:http-stress;1.0.0-beta.1;1.0.0-beta.1
 io.clientcore:optional-dependency-tests;1.0.0-beta.1;1.0.0-beta.1
 io.clientcore:annotation-processor;1.0.0-beta.1;1.0.0-beta.2
 io.clientcore:annotation-processor-test;1.0.0-beta.1;1.0.0-beta.1
 
-
 # Unreleased dependencies: Copy the entry from above, prepend "unreleased_" and remove the current
 # version. Unreleased dependencies are only valid for dependency versions.
 # Format;

sdk/clientcore/annotation-processor/pom.xml

@@ -113,7 +113,7 @@
     <dependency>
       <groupId>io.clientcore</groupId>
       <artifactId>core</artifactId>
-      <version>1.0.0-beta.9</version> <!-- {x-version-update;io.clientcore:core;dependency} -->
+      <version>1.0.0-beta.8</version> <!-- {x-version-update;io.clientcore:core;dependency} -->
     </dependency>
 
     <!-- Unit Test -->

sdk/clientcore/ci.yml

@@ -30,7 +30,11 @@ parameters:
     displayName: 'core'
     type: boolean
     default: true
-  - name: release_clientcorehttpokhttp3
+  - name: release_httpnetty4
+    displayName: 'http-netty4'
+    type: boolean
+    default: false
+  - name: release_httpokhttp3
     displayName: 'http-okhttp3'
     type: boolean
     default: false
@@ -52,10 +56,14 @@ extends:
         releaseInBatch: ${{ parameters.release_core }}
         triggeringPaths:
         - /sdk/parents/clientcore-parent
+      - name: http-netty4
+        groupId: io.clientcore
+        safeName: httpnetty4
+        releaseInBatch: ${{ parameters.release_httpnetty4 }}
       - name: http-okhttp3
         groupId: io.clientcore
         safeName: httpokhttp3
-        releaseInBatch: ${{ parameters.release_clientcorehttpokhttp3 }}
+        releaseInBatch: ${{ parameters.release_httpokhttp3 }}
     AdditionalModules:
       - name: optional-dependency-tests
         groupId: io.clientcore

sdk/clientcore/core/src/main/java/io/clientcore/core/http/models/HttpHeader.java

@@ -23,7 +23,6 @@
  */
 @Metadata(properties = MetadataProperties.FLUENT)
 public final class HttpHeader {
-    private static final String[] EMPTY_HEADER_ARRAY = new String[0];
 
     private final HttpHeaderName name;
 
@@ -114,21 +113,6 @@ public String getValue() {
         return CACHED_STRING_VALUE_UPDATER.get(this);
     }
 
-    /**
-     * Gets the comma separated value as an array. Changes made to this array will not be reflected in the headers.
-     *
-     * @return the values of this {@link HttpHeader} that are separated by a comma
-     */
-    String[] getValuesArray() {
-        if (value != null) {
-            return new String[] { value };
-        } else if (!isNullOrEmpty(values)) {
-            return values.toArray(new String[0]);
-        } else {
-            return EMPTY_HEADER_ARRAY;
-        }
-    }
-
     /**
      * Returns all values associated with this header, represented as an unmodifiable list of strings.
      *

sdk/clientcore/core/src/test/java/io/clientcore/core/http/client/SimpleBasicAuthHttpProxyServer.java

@@ -45,13 +45,11 @@ public ProxyEndpoint start() {
             if (proxyAuthorization == null) {
                 resp.setStatus(407);
                 resp.setHeader("Proxy-Authenticate", "Basic");
-
                 return;
             }
 
             if (!proxyAuthorization.startsWith("Basic")) {
                 resp.setStatus(401);
-
                 return;
             }
 

sdk/clientcore/core/src/test/java/io/clientcore/core/shared/HttpClientTests.java

@@ -80,7 +80,6 @@ public abstract class HttpClientTests {
     private static final String UTF_32BE_BOM_RESPONSE = "utf32BeBomBytes";
     private static final String UTF_32LE_BOM_RESPONSE = "utf32LeBomBytes";
     private static final String BOM_WITH_DIFFERENT_HEADER = "bomBytesWithDifferentHeader";
-    private static final String SSE_RESPONSE = "serversentevent";
 
     protected static final String ECHO_RESPONSE = "echo";