SONAR-16230 Add PCI DSS standard support in Plugin API

2 files changed, 40 insertions, 2 deletions

diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
index 6d33f2dc0ee..e3a88118fbb 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/RulesDefinition.java
@@ -273,6 +273,26 @@ public interface RulesDefinition {
     A1, A2, A3, A4, A5, A6, A7, A8, A9, A10
   }
 
+  enum PciDssVersion {
+    V3_2("3.2", "pciDss-3.2"), V4_0("4.0", "pciDss-4.0");
+
+    private final String label;
+    private final String prefix;
+
+    PciDssVersion(String label, String prefix) {
+      this.label = label;
+      this.prefix = prefix;
+    }
+
+    public String label() {
+      return label;
+    }
+
+    public String prefix() {
+      return prefix;
+    }
+  }
+
   interface ExtendedRepository {
     String key();
 
@@ -465,6 +485,11 @@ public interface RulesDefinition {
     public abstract NewRule addOwaspTop10(OwaspTop10Version version, OwaspTop10... standards);
 
     /**
+     * @since 9.5
+     */
+    public abstract NewRule addPciDss(PciDssVersion version, String... requirements);
+
+    /**
      * @since 7.3
      */
     public abstract NewRule addCwe(int... nums);
diff --git a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/internal/DefaultNewRule.java b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/internal/DefaultNewRule.java
index 9077055fe4c..0bae29224f4 100644
--- a/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/internal/DefaultNewRule.java
+++ b/sonar-plugin-api/src/main/java/org/sonar/api/server/rule/internal/DefaultNewRule.java
@@ -25,7 +25,6 @@ import java.util.Collection;
 import java.util.HashMap;
 import java.util.Locale;
 import java.util.Map;
-import java.util.Objects;
 import java.util.Set;
 import java.util.TreeSet;
 import javax.annotation.CheckForNull;
@@ -41,9 +40,11 @@ import org.sonar.api.server.rule.RuleTagFormat;
 import org.sonar.api.server.rule.RulesDefinition;
 import org.sonar.api.server.rule.RulesDefinition.OwaspTop10;
 import org.sonar.api.server.rule.RulesDefinition.OwaspTop10Version;
+import org.sonar.api.server.rule.RulesDefinition.PciDssVersion;
 
 import static java.lang.String.format;
 import static java.nio.charset.StandardCharsets.UTF_8;
+import static java.util.Objects.requireNonNull;
 import static org.apache.commons.lang.StringUtils.isEmpty;
 import static org.apache.commons.lang.StringUtils.trimToNull;
 import static org.sonar.api.utils.Preconditions.checkArgument;
@@ -234,7 +235,7 @@ class DefaultNewRule extends RulesDefinition.NewRule {
 
   @Override
   public DefaultNewRule addOwaspTop10(OwaspTop10Version owaspTop10Version, OwaspTop10... standards) {
-    Objects.requireNonNull(owaspTop10Version, "Owasp version must not be null");
+    requireNonNull(owaspTop10Version, "Owasp version must not be null");
 
     for (OwaspTop10 owaspTop10 : standards) {
       String standard = owaspTop10Version.prefix() + ":" + owaspTop10.name().toLowerCase(Locale.ENGLISH);
@@ -244,6 +245,18 @@ class DefaultNewRule extends RulesDefinition.NewRule {
   }
 
   @Override
+  public DefaultNewRule addPciDss(PciDssVersion pciDssVersion, String... requirements) {
+    requireNonNull(pciDssVersion, "PCI DSS version must not be null");
+    requireNonNull(requirements, "Requirements for PCI DSS standard must not be null");
+
+    for (String requirement : requirements) {
+      String standard = pciDssVersion.prefix() + ":" + requirement;
+      securityStandards.add(standard);
+    }
+    return this;
+  }
+
+  @Override
   public DefaultNewRule addCwe(int... nums) {
     for (int num : nums) {
       String standard = "cwe:" + num;