Correctly report test status when running through pipe (#4955)

Author: adamw

.github/workflows/ci.yml

@@ -88,11 +88,13 @@ jobs:
       if: matrix.target-platform != 'JS'
       uses: nick-fields/retry@ce71cc2ab81d554ebbe88c79ab5975992d79ba08 # v3, specifically v3.0.2
       with:
-        timeout_minutes: 8
-        max_attempts: 4
+        timeout_minutes: 10
+        max_attempts: 2
         # the netty server interpreter tests include some which run a large number of requests, to detect potential leaks
         # storing the output to verify it in the next step
-        command: sbt $SBT_JAVA_OPTS -v "testScoped ${{ matrix.scala-version }} ${{ matrix.target-platform }}" 2>&1 | tee output.log
+        command: |
+          sbt $SBT_JAVA_OPTS -v "testScoped ${{ matrix.scala-version }} ${{ matrix.target-platform }}" 2>&1 | tee output.log
+          exit ${PIPESTATUS[0]}
     - name: Check for reported leaks
       if: matrix.target-platform != 'JS'
       run: |

server/zio-http-server/src/main/scala/sttp/tapir/server/ziohttp/ZioHttpServerRequest.scala

@@ -25,7 +25,12 @@ case class ZioHttpServerRequest(req: Request, attributes: AttributeMap = Attribu
   override lazy val method: SttpMethod = SttpMethod(req.method.name.toUpperCase)
   override lazy val showShort: String = s"$method ${encodeQueryParams(req.url.path.encode, req.url.queryParams.normalize, Charsets.Http)}"
 
-  override lazy val uri: Uri = Uri.unsafeParse(req.url.encode)
+  override lazy val uri: Uri = {
+    // work-around fo zio-http not decoding path segments properly
+    Uri.unsafeParse(
+      s"/${req.url.path.segments.toList.mkString("/")}${if (req.url.path.hasTrailingSlash) "/" else ""}${req.url.fragment.map(f => s"#$f").getOrElse("")}${req.url.queryParams.encode}"
+    )
+  }
   override lazy val headers: Seq[SttpHeader] = req.headers.toList.map { h => SttpHeader(h.headerName, h.renderedValue) }
   override def attribute[T](k: AttributeKey[T]): Option[T] = attributes.get(k)
   override def attribute[T](k: AttributeKey[T], v: T): ZioHttpServerRequest = copy(attributes = attributes.put(k, v))