liuxiran opened a new pull request #1447:
URL: https://github.com/apache/apisix-dashboard/pull/1447


   Please answer these questions before submitting a pull request
   
   - Why submit this pull request?
   - [ ] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   - [ ] Backport patches
   
   - Related issues
   
   ___
   ### Bugfix
   - Description
   
   > This part of the regular expression may cause exponential backtracking on 
strings starting with '0' and containing many repetitions of '0'.
   
   This issue originated from 
https://github.com/apache/apisix-dashboard/pull/1417#issuecomment-772585704, 
which caused The performance on the UI : when type `000` to debug url, it will 
passed the reg test
   
   
![mmexport1612678593730](https://user-images.githubusercontent.com/2561857/107138608-648a0880-6950-11eb-9d57-e150dc06fc95.png)
   
   
   - How to fix?
   
   use [url-regex-safe](https://github.com/niftylettuce/url-regex-safe) instead 
of manually debug url regexp to test the debug url
   
   ___
   ### Please add the corresponding test cases if necessary.
   
   add test cases to test `000` and empty url 
   
   


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

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


Reply via email to