kwin commented on code in PR #54:
URL: 
https://github.com/apache/sling-org-apache-sling-api/pull/54#discussion_r1775122797


##########
src/main/java/org/apache/sling/api/request/builder/Builders.java:
##########
@@ -92,4 +92,16 @@ private Builders() {}
     public static @NotNull RequestParameter newRequestParameter(String name, 
String value, Charset encoding) {
         return new RequestParameterImpl(name, value, encoding);
     }
+
+    /**
+     * Creates a new binary request parameter
+     *
+     * @param name the parameter name
+     * @param value the parameter value
+     * @return a request parameter
+     * @since 1.3 (Sling API Bundle 2.28.0)
+     */
+    public static @NotNull RequestParameter newRequestParameter(String name, 
byte[] value) {

Review Comment:
   Probably allowing to parameterize the content type would be beneficial here



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@sling.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to