AlinsRan opened a new pull request, #2815:
URL: https://github.com/apache/apisix-ingress-controller/pull/2815

   ### Type of change:
   
   - [x] Bugfix
   
   ### What this PR does / why we need it:
   
   Fixes #2723
   
   `GetStringsAnnotation` split comma-separated annotation values with 
`strings.Split(value, ",")` and did not trim whitespace. A value like 
`127.0.0.1, 0.0.0.0` produced `["127.0.0.1", " 0.0.0.0"]`; the leading space 
made the IP invalid, which failed validation and stalled sync for every 
resource under that ingressClass.
   
   This trims each split element and drops blanks (so trailing/extra commas 
don't yield empty entries).
   
   ### Pre-submission checklist:
   
   - [x] Did you explain what problem does this PR solve?
   - [x] Have you added corresponding test cases? (`TestGetStringsAnnotation`)
   - [ ] Have you modified the corresponding document? (n/a)
   - [x] Is this PR backward compatible?


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