[GitHub] [apisix-dashboard] bzp2010 commented on pull request #2460: refactor: OpenAPI 3 parse and convert

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

bzp2010 commented on PR #2460:
URL: https://github.com/apache/apisix-dashboard/pull/2460#issuecomment-1152153563

   ### Update
   
   - Route IDs are no longer used (now generated by Dashboard itself)
   - The route name generation algorithm has been updated and it now uses the following rule
   
   **`TaskName_consumer/{cid}[_GET/POST]`**
   
   ```text
   taskName: test
   
   methodMerge: true
   Method + URI:
   GET|POST /consumer/{cid} => test_consumer/{cid}
   
   methodMerge: false
   Method + URI:
   GET|POST /consumer/{cid} => test_consumer/{cid}_GET  +  test_consumer/{cid}_POST
   ```
   
   ### Request for Comments
   
   In response to this, I would like to seek more suggestions and I believe there are the following needs met:
   
   - Total length will not be long
   - Try not to have symbols other than `_`, such as `/{}`, etc.
   - Can contain URIs and methods
   - No dependency on specific fields of OpenAPI (as none of them may exist)
   
   


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