Merge pull request #101 from nextcloud/feat/scopes/limit-tag-names

feat(scopes): Limit tag names
nextcloud · Mar 14, 2024 · 09030f8 · 09030f8
2 parents e15283d + a943aab
commit 09030f8
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/src/Helpers.php b/src/Helpers.php
@@ -240,7 +240,12 @@ public static function getOpenAPIAttributeTagsByScope(ClassMethod|Class_|Node $n
 
 						foreach ($arg->value->items as $item) {
 							if ($item?->value instanceof String_) {
-								$foundTags[] = $item->value->value;
+								$tag = $item->value->value;
+								$pattern = "/^[0-9a-zA-Z_-]+$/";
+								if (!preg_match($pattern, $tag)) {
+									Logger::error($routeName, 'Tag "' . $tag . '" has to match pattern "' . $pattern . '"');
+								}
+								$foundTags[] = $tag;
 							}
 						}
 					}