iishitahere commented on issue #10002:
URL: https://github.com/apache/cloudstack/issues/10002#issuecomment-2507780380

   > > > 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!
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to