fix: Various fixes to make http-recorder usable again. (#2983)

As it stands, http-recorder only works with GET requests without a body.
This situation is caused by two issues:
- The hash code of the body is not computed correctly. `Arrays.hashcode`
should be used to compute an hash code over the content of the array as
the `hashcode` method on the array itself only relies on the address of
the object.
- `RedirectToRxEndpoint` only exposes a `GET` endpoint. No other verbs
can pass-through. I don't know if there's a better way to fix it, but
the quick fix I'm proposing works.

airframe-http/.jvm/src/test/scala/wvlet/airframe/http/HttpMessageTest.scala

@@ -14,6 +14,8 @@
 package wvlet.airframe.http
 import wvlet.airspec.AirSpec
 
+import java.nio.charset.StandardCharsets
+
 /**
   */
 class HttpMessageTest extends AirSpec {
@@ -160,4 +162,12 @@ class HttpMessageTest extends AirSpec {
     r.acceptsMsgPack shouldBe true
     r.acceptsJson shouldBe false
   }
+
+  test("hashcode of the body gets computed correctly") {
+    val json = "{\"drink\": \"coffee\"}";
+    val req1 = Http.request("/drinks").withContent(json)
+    val req2 = Http.request("/drinks").withContent(json.getBytes(StandardCharsets.UTF_8))
+
+    req1.message.contentHash shouldBe req2.message.contentHash
+  }
 }

airframe-http/src/main/scala/wvlet/airframe/http/HttpMessage.scala

@@ -18,6 +18,7 @@ import wvlet.airframe.msgpack.spi.MsgPack
 
 import java.nio.charset.StandardCharsets
 import java.time.Instant
+import java.util
 import scala.language.experimental.macros
 
 trait HttpMessage[Raw] extends HttpMessageBase[Raw] {
@@ -141,7 +142,7 @@ object HttpMessage {
     def nonEmpty: Boolean = !isEmpty
     def toContentString: String
     def toContentBytes: Array[Byte]
-    def contentHash: Int = toContentBytes.hashCode()
+    def contentHash: Int = util.Arrays.hashCode(toContentBytes)
   }
 
   object Message {

airframe-http/src/main/scala/wvlet/airframe/http/router/RedirectToRxEndpoint.scala

@@ -13,7 +13,7 @@
  */
 package wvlet.airframe.http.router
 
-import wvlet.airframe.http.{Endpoint, HttpMessage, RPCContext, RxHttpEndpoint}
+import wvlet.airframe.http.{Endpoint, HttpMessage, HttpMethod, RPCContext, RxHttpEndpoint}
 import wvlet.airframe.rx.Rx
 import wvlet.log.LogSupport
 
@@ -22,8 +22,31 @@ import wvlet.log.LogSupport
   * @param endpoint
   */
 class RedirectToRxEndpoint(endpoint: RxHttpEndpoint) extends LogSupport {
-  @Endpoint(path = "/*path")
-  def process(): Rx[HttpMessage.Response] = {
+  @Endpoint(path = "/*path", method = HttpMethod.GET)
+  def get(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.POST)
+  def post(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.PUT)
+  def put(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.HEAD)
+  def head(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.PATCH)
+  def patch(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.DELETE)
+  def delete(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.OPTIONS)
+  def options(): Rx[HttpMessage.Response] = process()
+
+  @Endpoint(path = "/*path", method = HttpMethod.TRACE)
+  def trace(): Rx[HttpMessage.Response] = process()
+
+  private def process(): Rx[HttpMessage.Response] = {
     val req = RPCContext.current.httpRequest
     endpoint.apply(req)
   }

airframe-http/src/test/scala/wvlet/airframe/http/router/CustomEndpointTest.scala

@@ -13,7 +13,7 @@
  */
 package wvlet.airframe.http.router
 
-import wvlet.airframe.http.{Http, HttpMessage, RxHttpEndpoint, RxRouter}
+import wvlet.airframe.http._
 import wvlet.airframe.rx.Rx
 import wvlet.airframe.surface.Surface
 import wvlet.airspec.AirSpec
@@ -31,7 +31,19 @@ class CustomEndpointTest extends AirSpec {
     router shouldMatch { case RxRouter.EndpointNode(controllerSurface, methodSurfaces, Some(ep)) =>
       ep shouldBeTheSameInstanceAs endpoint
       controllerSurface shouldBe Surface.of[RedirectToRxEndpoint]
-      methodSurfaces(0).name shouldBe "process"
+
+      val expectedMethods = Set(
+        HttpMethod.GET.toLowerCase,
+        HttpMethod.POST.toLowerCase,
+        HttpMethod.PATCH.toLowerCase,
+        HttpMethod.DELETE.toLowerCase,
+        HttpMethod.PUT.toLowerCase,
+        HttpMethod.OPTIONS.toLowerCase,
+        HttpMethod.TRACE.toLowerCase,
+        HttpMethod.HEAD.toLowerCase
+      )
+
+      methodSurfaces.map(_.name).toSet == expectedMethods
     }
   }
 }