Forward Slash in block names about ipam HOT 4 OPEN

from ipam.

Hi @stumueller, after looking into this I believe the best thing to do is to rename the Blocks. My suggestion would be to use an underscore instead of a slash like so:

10.224.32.0_19

The reason behind this is it appears OpenAPI doesn't really have a way to handle paths as an HTTP path parameter very well and this can also lead to some difficult situations from a testing & stability perspective.

This is documented here: https://fastapi.tiangolo.com/tutorial/path-params/#path-parameters-containing-paths

OpenAPI doesn't support a way to declare a path parameter to contain a path inside, as that could lead to scenarios that are difficult to test and define.

I'll be removing the "/" as a supported character for Block names, however I'm open to any feedback you may have on my above observations.

Thanks for reaching out!

from ipam.

Good catch, I'll make sure that's all addressed accordingly.

Thanks so much!

from ipam.