[GitHub] [apisix-dashboard] Chaunceyan opened a new issue #2297: Route.Upstream.Retries field needs to support zero value

Thu, 27 Jan 2022 01:43:51 -0800 


Chaunceyan opened a new issue #2297:
URL: https://github.com/apache/apisix-dashboard/issues/2297


   ### Issue description
   
   As shown in the doc, the retries field expected zero and null value to 
behave differently. 
   But current entity definition omit zero value to empty, so this 
configuration does not function properly. 
   
![image](https://user-images.githubusercontent.com/8214895/151332575-06c43693-0e75-4be9-9d84-ca373f9b516d.png)
   
   I am raising PR for this.
   
   ### Expected behavior
   
   Zero and Null value should be well distinguished.
   
   ### How to Reproduce
   
   1. Create Upstream with retries being zero.
   2. View Upstream to see the retries being empty.
   
   ### Screenshots
   
   
![image](https://user-images.githubusercontent.com/8214895/151333059-df5d0347-e064-4717-abd2-7f281ff38e68.png)
   
![image](https://user-images.githubusercontent.com/8214895/151333126-3d2cdbd5-4088-43ed-8519-fa6c9cb8cad4.png)
   
![image](https://user-images.githubusercontent.com/8214895/151333308-e1470460-f6ab-47a4-b5e6-24ef4db3606f.png)
   
   
   ### Environment
   
   - apisix version (cmd: `apisix version`):
   - OS (cmd: `uname -a`):
   - OpenResty / Nginx version (cmd: `nginx -V` or `openresty -V`):
   - etcd version, if have (cmd: run `etcd --version`):
   - apisix-dashboard version, if have:
   - Browser version, if have:
   
   
   ### Additional context
   
   _No response_


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