You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2020/11/18 08:01:20 UTC

[GitHub] [apisix-dashboard] LiteSun opened a new issue #823: [Proposal]: Version management of route

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


   Please answer these questions before submitting your issue.
   
   - Why do you submit this issue?
   - [ ] Question or discussion
   - [ ] Bug
   - [ ] Requirements
   - [X] Feature or performance improvement
   - [ ] Other
   
   ___
   ### Question
   - What do you want to know?
   
   ___
   ### Bug
   - Which version of Apache APISIX Dashboard, OS, and Browser?
   
   - What happened?
   If possible, provide a way to reproduce the error.
   
   ___
   ### Requirements or improvement
   - Please describe your requirements or improvement suggestions.
   
   TODO


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] juzhiyuan commented on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
juzhiyuan commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-731560942


   For draft proposal, please add some attractive notes, such as in title [DRAFT]


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] membphis commented on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
membphis commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-747892113


   In the first version, I think it is enough to control the API version by only supporting the header.
   This makes our implementation design easier.
   
   I was confused with `Edit version drawer` and `version table`, need more docs @LiteSun , I can not understand them.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] LiteSun commented on issue #823: [DRAFT]: Version management of route

Posted by GitBox <gi...@apache.org>.
LiteSun commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-747847820


   In order to manage api more finely, I propose to add api version management feature.
   
   ## front-end
    - add version management section in route step1 page
    
   ![image](https://user-images.githubusercontent.com/31329157/102568888-6b9a5480-411f-11eb-93b5-7f3bf69585f8.png)
   
   - when `enable version` turn on,  section `version config` and `version table` will show.
   
   ![image](https://user-images.githubusercontent.com/31329157/102569147-df3c6180-411f-11eb-9aee-31c67c6ddfb6.png)
   
   ![image](https://user-images.githubusercontent.com/31329157/102569417-67226b80-4120-11eb-90e4-fe0e48439d01.png)
   
   - when click `add` or `edit`, `Edit version drawer` will show.
   ![image](https://user-images.githubusercontent.com/31329157/102569807-20814100-4121-11eb-8653-7e7c02db1c80.png)
   
   ## back-end
   - `POST /apisix/admin/routes` and `PUT /apisix/admin/routes` need to  support `version`.
   
   ```json
   {
   ...,
   ...,
   version: {
       enable: true,
       location:'header' | 'url' | 'first part of path',
       strip_versio?n: false,
       name:'string',
       default: 'string',
       version_list:[{
                   value:'v1',
                   override target host:'string',
                   expires: timestamp
       },{}]
   }
   }
   ```
   
   - need a new API to get version list, `get /version`:
   ```json
      version_list:[{
                   value:'v1',
                   override target host:'string',
                   expires: timestamp
       },{}]
   ```
   
   ## apisix
   need to discuss.
   
   cc @juzhiyuan @liuxiran  @moonming @membphis @nic-chen 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] juzhiyuan commented on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
juzhiyuan commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-748007006


   I will take this proposal.


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] LiteSun edited a comment on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
LiteSun edited a comment on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-747847820


   In order to manage api more finely, I propose to add api version management feature.
   
   ## front-end
    - add version management section in route step1 page
    
   ![image](https://user-images.githubusercontent.com/31329157/102568888-6b9a5480-411f-11eb-93b5-7f3bf69585f8.png)
   
   - when `enable version` turn on,  section `version config` and `version table` will show.
   
   ![image](https://user-images.githubusercontent.com/31329157/102569147-df3c6180-411f-11eb-9aee-31c67c6ddfb6.png)
   
   ![image](https://user-images.githubusercontent.com/31329157/102569417-67226b80-4120-11eb-90e4-fe0e48439d01.png)
   
   - when click `add` or `edit`, `Edit version drawer` will show.
   ![image](https://user-images.githubusercontent.com/31329157/102569807-20814100-4121-11eb-8653-7e7c02db1c80.png)
   
   ## back-end
   - `POST /apisix/admin/routes` and `PUT /apisix/admin/routes` need to  support `version`.
   
   ```json
   {
   ...,
   ...,
   version: {
       enable: true,
       location:'header' | 'url' | 'first part of path',
       strip_versio?n: false,
       name:'string',
       default: 'string',
       version_list:[{
                   value:'v1',
                   override target host:'string',
                   expires: timestamp
       },{}]
   }
   }
   ```
   ## apisix
   need to discuss.
   
   cc @juzhiyuan @liuxiran  @moonming @membphis @nic-chen 


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] LiteSun closed issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
LiteSun closed issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823


   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] LiteSun commented on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
LiteSun commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-747917717


   > I was confused with `Edit version drawer` and `version table`, need more docs @LiteSun , I can not understand them.
   
   1. version table is used to show the version list, just like our  `Advanced Routing Matching Conditions`
   ![image](https://user-images.githubusercontent.com/31329157/102585838-09a01600-4144-11eb-8ca1-b1d93dae3508.png)
   
   2.` Edit version drawer` is used to edit or create a version item. will trigger show when you click the `add` button or `edit`  


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] moonming commented on issue #823: [Proposal]: Version management of route

Posted by GitBox <gi...@apache.org>.
moonming commented on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-731559963


   Why empty?


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [apisix-dashboard] LiteSun edited a comment on issue #823: [DRAFT]: Version management of route

Posted by GitBox <gi...@apache.org>.
LiteSun edited a comment on issue #823:
URL: https://github.com/apache/apisix-dashboard/issues/823#issuecomment-747847820


   In order to manage api more finely, I propose to add api version management feature.
   
   ## front-end
    - add version management section in route step1 page
    
   ![image](https://user-images.githubusercontent.com/31329157/102568888-6b9a5480-411f-11eb-93b5-7f3bf69585f8.png)
   
   - when `enable version` turn on,  section `version config` and `version table` will show.
   
   ![image](https://user-images.githubusercontent.com/31329157/102569147-df3c6180-411f-11eb-9aee-31c67c6ddfb6.png)
   
   ![image](https://user-images.githubusercontent.com/31329157/102569417-67226b80-4120-11eb-90e4-fe0e48439d01.png)
   
   - when click `add` or `edit`, `Edit version drawer` will show.
   ![image](https://user-images.githubusercontent.com/31329157/102569807-20814100-4121-11eb-8653-7e7c02db1c80.png)
   
   ## back-end
   - `POST /apisix/admin/routes` and `PUT /apisix/admin/routes` need to  support `version`.
   
   ```json
   {
   ...,
   ...,
   version: {
       enable: true,
       location:'header' | 'url' | 'first part of path',
       strip_versio?n: false,
       name:'string',
       default: 'string',
       version_list:[{
                   value:'v1',
                   override target host:'string',
                   expires: timestamp
       },{}]
   }
   }
   ```
   
   - need a new API to get version list, `get /version`:
   ```json
      version_list:[{
                   value:'v1',
                   override_target_host:'string',
                   expires: timestamp
       },{}]
   ```
   
   ## apisix
   need to discuss.
   
   cc @juzhiyuan @liuxiran  @moonming @membphis @nic-chen 


----------------------------------------------------------------
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:
users@infra.apache.org