René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Validation of Security Groups is wrong","articleBody":"## Invalid validation rule for security group\n\nWe have a validation rule in place that checks for the correct name of a security group. It does not allow consecutive special characters in a name that is actually valid:\n\ncls.__pydantic_validator__.validate_python(\\npydantic_core._pydantic_core.ValidationError: 1 validation error for SecurityGroup\\nname\\n  \nValue error, must validate the regular expression /^[A-Za-z0-9]+((-|_|\\\\s|\\\\.)[A-Za-z0-9]+)*$/ [type=value_error, input_value='Nagios Monitoring - NRPE', input_type=str]\n\nThe problem is, that a space, dash and another space in a row are correct, but maybe two spaces are not. Either the regex needs to be more specific or is should be deleted. \n\n## Steps to reproduce\nThe error is in the class iaas/models/security_group.py\n\n```python\n    @field_validator(\"name\")\n    def name_validate_regular_expression(cls, value):\n        \"\"\"Validates the regular expression\"\"\"\n        if not re.match(r\"^[A-Za-z0-9]+((-|_|\\s|\\.)[A-Za-z0-9]+)*$\", value):\n            raise ValueError(r\"must validate the regular expression /^[A-Za-z0-9]+((-|_|\\s|\\.)[A-Za-z0-9]+)*$/\")\n        return value\n```\n\nthe security groups cannot even be loaded:\n\n```python\n    def __get_security_groups(self, client: DefaultApi, uuid: UUID) -\u003e List[StackitSecurityGroup]:\n        security_groups = client.list_security_groups(uuid)\n```\n\n## Actual behavior\n\nRaises an exception on valid security names\n\n## Expected behavior\n\nRaise only an exception on invalid security group name\n\n## Environment\n - OS: Mac\n - Python version (see `python --version`): `Python 3.9.4`\n - Version of the Python STACKIT SDK:  4.5.0\n\nUnfortunately I could not find the check Openstack is implementing in its code and validation ...\nMaybe it is also not in the responsibility of this class to check weather the name complies with a certain pattern as the Openstack Neutron server will raise a 4xx error when the name is not correct (e.g., too long, already taken, etc) in any case. Maybe we want to rely on what Openstack says to the name (or any other field, or even on any other resource). Maybe we will not be able to keep up with changes on the server side.\n\n\nUps, now I see this bug: https://github.com/stackitcloud/stackit-sdk-python/issues/790\nSame problem, different Openstack resource\nAgain, maybe not the responsibility of this lib to check for the validity of the names, esp. not when reading only the resources","author":{"url":"https://github.com/wuppermann","@type":"Person","name":"wuppermann"},"datePublished":"2025-05-06T15:02:13.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/1054/stackit-sdk-python/issues/1054"}