[GitHub] [apisix-dashboard] Baoyuantop commented on issue #2019: [health_check] validate form item "checks.active.port"

[GitBox <gi...@apache.org>]

Baoyuantop commented on issue #2019:
URL: https://github.com/apache/apisix-dashboard/issues/2019#issuecomment-889656514


   Thank you @uriddle .
   
   When create or update Upstream, the port option in Health Check is optional.
   
   Dashboard will provide a default value 80, but when the user changes it to empty, dashboard will send `null` to api, this will cause this error.
   
   The possible solution is to not pass the `port` option to the api when the user sets it to `null`, but how will we save the port status as null?
   
   what do you think about it ? @uriddle @juzhiyuan 


-- 
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: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org