Correct

Author: dstepanov

core-processor/src/main/java/io/micronaut/context/visitor/ExecutableVisitor.java

@@ -18,6 +18,7 @@
 import io.micronaut.context.annotation.Executable;
 import io.micronaut.core.annotation.Internal;
 import io.micronaut.core.annotation.NonNull;
+import io.micronaut.inject.ast.ClassElement;
 import io.micronaut.inject.ast.KotlinParameterElement;
 import io.micronaut.inject.ast.MethodElement;
 import io.micronaut.inject.ast.ParameterElement;
@@ -48,7 +49,8 @@ public TypeElementQuery query() {
     @Override
     public void visitMethod(MethodElement element, VisitorContext context) {
         for (ParameterElement parameter : element.getParameters()) {
-            if (parameter.getType().isPrimitive() && parameter.isNullable()
+            ClassElement type = parameter.getType();
+            if (type.isPrimitive() && !type.isArray() && parameter.isNullable()
                 && !(parameter instanceof KotlinParameterElement kotlinParameterElement && kotlinParameterElement.hasDefault())) {
                 context.warn("@Nullable on primitive types will allow the method to be executed at runtime with null values, causing an exception", parameter);
             }

http/src/main/java/io/micronaut/http/bind/DefaultRequestBinderRegistry.java

@@ -15,6 +15,8 @@
  */
 package io.micronaut.http.bind;
 
+import io.micronaut.core.annotation.AnnotationMetadata;
+import io.micronaut.core.annotation.AnnotationUtil;
 import io.micronaut.core.annotation.NonNull;
 import io.micronaut.core.bind.ArgumentBinder;
 import io.micronaut.core.bind.annotation.Bindable;
@@ -33,8 +35,8 @@
 import io.micronaut.http.annotation.Body;
 import io.micronaut.http.bind.binders.AnnotatedRequestArgumentBinder;
 import io.micronaut.http.bind.binders.ContinuationArgumentBinder;
-import io.micronaut.http.bind.binders.CookieObjectArgumentBinder;
 import io.micronaut.http.bind.binders.CookieAnnotationBinder;
+import io.micronaut.http.bind.binders.CookieObjectArgumentBinder;
 import io.micronaut.http.bind.binders.DefaultBodyAnnotationBinder;
 import io.micronaut.http.bind.binders.DefaultUnmatchedRequestArgumentBinder;
 import io.micronaut.http.bind.binders.HeaderAnnotationBinder;
@@ -48,6 +50,8 @@
 import io.micronaut.http.bind.binders.TypedRequestArgumentBinder;
 import io.micronaut.http.cookie.Cookie;
 import io.micronaut.http.cookie.Cookies;
+import io.micronaut.inject.annotation.AnnotationMetadataHierarchy;
+import io.micronaut.inject.annotation.MutableAnnotationMetadata;
 import jakarta.inject.Inject;
 import jakarta.inject.Singleton;
 
@@ -71,7 +75,7 @@
 public class DefaultRequestBinderRegistry implements RequestBinderRegistry {
 
     private static final long CACHE_MAX_SIZE = 30;
-
+    private static final AnnotationMetadata NULLABLE_ANNOTATION_METADATA;
     private final Map<Class<? extends Annotation>, RequestArgumentBinder> byAnnotation = new LinkedHashMap<>();
     private final Map<TypeAndAnnotation, RequestArgumentBinder> byTypeAndAnnotation = new LinkedHashMap<>();
     private final Map<Integer, RequestArgumentBinder> byType = new LinkedHashMap<>();
@@ -81,6 +85,12 @@ public class DefaultRequestBinderRegistry implements RequestBinderRegistry {
     private final List<RequestArgumentBinder<Object>> unmatchedBinders = new ArrayList<>();
     private final DefaultUnmatchedRequestArgumentBinder defaultUnmatchedRequestArgumentBinder;
 
+    static {
+        MutableAnnotationMetadata nullable = new MutableAnnotationMetadata();
+        nullable.addAnnotation(AnnotationUtil.NULLABLE, Map.of());
+        NULLABLE_ANNOTATION_METADATA = nullable;
+    }
+
     /**
      * @param conversionService The conversion service
      * @param binders           The request argument binders
@@ -268,7 +278,14 @@ private static ArgumentBinder.BindingResult<? extends HttpRequest<?>> convertBod
                 .filter(arg -> arg.getType() != Void.class);
             if (typeVariable.isPresent()) {
                 @SuppressWarnings("unchecked")
-                ArgumentConversionContext<Object> unwrappedConversionContext = ConversionContext.of((Argument<Object>) typeVariable.get());
+                Argument<Object> argument = (Argument<Object>) typeVariable.get();
+                argument = argument.withAnnotationMetadata(
+                    new AnnotationMetadataHierarchy(
+                        argument.getAnnotationMetadata(),
+                        NULLABLE_ANNOTATION_METADATA // HttpRequest's body can be null
+                    )
+                );
+                ArgumentConversionContext<Object> unwrappedConversionContext = ConversionContext.of(argument);
                 ArgumentBinder.BindingResult<Object> bodyBound = bodyAnnotationBinder.bindFullBody(unwrappedConversionContext, source);
                 // can't use flatMap here because we return a present optional even when the body conversion failed
                 return new PendingRequestBindingResult<>() {
@@ -286,14 +303,14 @@ public List<ConversionError> getConversionErrors() {
                     public Optional<HttpRequest<?>> getValue() {
                         Optional<Object> body = bodyBound.getValue();
                         if (pushCapable) {
-                            return Optional.of(new PushCapableRequestWrapper<Object>((HttpRequest<Object>) source, (PushCapableHttpRequest<?>) source) {
+                            return Optional.of(new PushCapableRequestWrapper<>((HttpRequest<Object>) source, (PushCapableHttpRequest<?>) source) {
                                 @Override
                                 public Optional<Object> getBody() {
                                     return body;
                                 }
                             });
                         } else {
-                            return Optional.of(new HttpRequestWrapper<Object>((HttpRequest<Object>) source) {
+                            return Optional.of(new HttpRequestWrapper<>((HttpRequest<Object>) source) {
                                 @Override
                                 public Optional<Object> getBody() {
                                     return body;

http/src/main/java/io/micronaut/http/body/InputStreamBodyReader.java

@@ -34,10 +34,15 @@
 @Prototype
 @BootstrapContextCompatible
 @Internal
-final class InputStreamBodyReader implements MessageBodyReader<InputStream> {
+final class InputStreamBodyReader implements TypedMessageBodyReader<InputStream> {
 
     @Override
     public InputStream read(Argument<InputStream> type, MediaType mediaType, Headers httpHeaders, InputStream inputStream) throws CodecException {
         return inputStream;
     }
+
+    @Override
+    public Argument<InputStream> getType() {
+        return Argument.of(InputStream.class);
+    }
 }