Fix boundary (#3)

* Update README.md (zio#3120) Co-authored-by: ZIO Assistant <zio-assistant[bot]@users.noreply.github.com> * Fix: Ensure multipart boundary is RFC 2046 compliant by sanitizing UUID * fmt * Update zio-http/shared/src/main/scala/zio/http/Boundary.scala Co-authored-by: asr2003 <162500856+asr2003@users.noreply.github.com> --------- Co-authored-by: zio-assistant[bot] <130037499+zio-assistant[bot]@users.noreply.github.com> Co-authored-by: ZIO Assistant <zio-assistant[bot]@users.noreply.github.com> Co-authored-by: asr2003 <162500856+asr2003@users.noreply.github.com>
varshith257 · Sep 11, 2024 · a011484 · a011484
1 parent 5dde44d
commit a011484
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -43,7 +43,7 @@ Some of the key features of ZIO HTTP are:
 Setup via `build.sbt`:
 
 ```scala
-libraryDependencies += "dev.zio" %% "zio-http" % "3.0.0-RC10"
+libraryDependencies += "dev.zio" %% "zio-http" % "3.0.0"
 ```
 
 **NOTES ON VERSIONING:**

diff --git a/zio-http/jvm/src/test/scala/zio/http/BoundarySpec.scala b/zio-http/jvm/src/test/scala/zio/http/BoundarySpec.scala
@@ -59,6 +59,17 @@ object BoundarySpec extends ZIOHttpSpec {
         boundary4.isEmpty,
       )
     },
+    suite("randomUUID")(
+      test("generated boundary is RFC 2046 compliant") {
+        for {
+          boundary <- Boundary.randomUUID
+        } yield assertTrue(
+          boundary.id.matches("^[a-zA-Z0-9-]+$"),
+          boundary.id.startsWith("----"),
+          boundary.id.endsWith("----"),
+        )
+      },
+    ),
   )
 
   val spec = suite("BoundarySpec")(

diff --git a/zio-http/shared/src/main/scala/zio/http/Boundary.scala b/zio-http/shared/src/main/scala/zio/http/Boundary.scala
@@ -75,6 +75,6 @@ object Boundary {
 
   def randomUUID(implicit trace: Trace): zio.UIO[Boundary] =
     zio.Random.nextUUID.map { id =>
-      Boundary(s"(((${id.toString()})))")
+      Boundary(s"----${id.toString}----")
     }
 }