SONAR-13513 Request parameter should not allow NUL character

1 files changed, 11 insertions, 0 deletions

diff --git a/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java b/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
index 2baebb55587..a484ebdd241 100644
--- a/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
+++ b/sonar-plugin-api/src/test/java/org/sonar/api/server/ws/RequestTest.java
@@ -49,8 +49,10 @@ import static com.google.common.base.Strings.repeat;
 import static com.google.common.collect.Lists.newArrayList;
 import static java.lang.String.format;
 import static org.assertj.core.api.Assertions.assertThat;
+import static org.assertj.core.api.Assertions.assertThatThrownBy;
 import static org.assertj.core.api.Assertions.fail;
 import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
 import static org.sonar.api.utils.DateUtils.parseDate;
 import static org.sonar.api.utils.DateUtils.parseDateTime;
 
@@ -217,6 +219,15 @@ public class RequestTest {
   }
 
   @Test
+  public void param_contains_NUL_char_should_throw_exception() {
+    underTest.setParam("a_string", "value\0value");
+
+    assertThatThrownBy(() -> underTest.param("a_string"))
+      .isInstanceOf(IllegalArgumentException.class)
+      .hasMessage("Request parameters are not allowed to contain NUL character");
+  }
+
+  @Test
   public void null_param() {
     assertThat(underTest.param("a_string")).isNull();
     assertThat(underTest.paramAsBoolean("a_boolean")).isNull();