Hi @weizhouapache, I’d like to work on this issue. From my understanding, the Source/Destination CIDR fields should default to 0.0.0.0/0 instead of being empty. Could you confirm if this is the expected behavior? If assigned, I’ll get started right away. Looking forward to contributing!

Hi @weizhouapache, I’d like to work on this issue. From my understanding, the Source/Destination CIDR fields should default to 0.0.0.0/0 instead of being empty. Could you confirm if this is the expected behavior? If assigned, I’ll get started right away. Looking forward to contributing!

@iishitahere , that is correct. When first editting and thanclearing the field, the default of 0.0.0.0/0 should be used again. This can be a UI fix, but as I understand the problem it can also be solved in the API. What do you think @weizhouapache ?

Hi @weizhouapache, I’d like to work on this issue. From my understanding, the Source/Destination CIDR fields should default to 0.0.0.0/0 instead of being empty. Could you confirm if this is the expected behavior? If assigned, I’ll get started right away. Looking forward to contributing!

@iishitahere , that is correct. When first editting and thanclearing the field, the default of 0.0.0.0/0 should be used again. This can be a UI fix, but as I understand the problem it can also be solved in the API. What do you think @weizhouapache ?

@DaanHoogland @iishitahere
from what I know, most APIs with source/destination CIDR are able to process an empty list.
so it looks like a UI issue, UI passes a list with empty string as api parameter. it should not pass the value if the cidr is an empty string or only contains spaces.

i @weizhouapache, I’d like to work on this issue. From my understanding, the Source/Destination CIDR fields should default to 0.0.0.0/0 instead of being empty. Could you confirm if this is the expected behavior? If assigned, I’ll get started right away. Looking forward to contributing!

@iishitahere , that is correct. When first editting and thanclearing the field, the default of 0.0.0.0/0 should be used again. This can be a UI fix, but as I understand the problem it can also be solved in the API. What do you think @weizhouapache ?

@DaanHoogland @iishitahere from what I know, most APIs with source/destination CIDR are able to process an empty list. so it looks like a UI issue, UI passes a list with empty string as api parameter. it should not pass the value if the cidr is an empty string or only contains spaces.

Thank you, @weizhouapache and @DaanHoogland , for the clarification! Based on your input, it seems that addressing this as a UI fix would be the right approach. I’ll focus on ensuring that the UI does not pass an empty string or spaces as API parameters for the CIDR field and defaults to 0.0.0.0/0 when the field is cleared.

I’ll start working on this and keep you updated on my progress. Please let me know if there are any additional details or specific areas you'd like me to focus on. Thanks again for assigning me to this issue!