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