[GitHub] [apisix] masahills opened a new issue #2165: request help: multi-tenancy/multi-workspace feature

2020-09-03 Thread GitBox
masahills opened a new issue #2165: URL: https://github.com/apache/apisix/issues/2165 ### Issue description Hi, Do you have any plan to introduce multi-tenancy feature into APISIX? The motivation for this feature is that we want to let multiple internal organizations/teams to

[GitHub] [apisix] gy09535 commented on issue #2138: test case: log-rotate.t is unstable

2020-09-03 Thread GitBox
gy09535 commented on issue #2138: URL: https://github.com/apache/apisix/issues/2138#issuecomment-686897638 I think if the log dir has files named "*_access.log" and "*_error.log" ,the log rotate is success. This is an

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

2020-09-03 Thread GitBox
Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r483375985 ## File path: apisix/core/config_etcd.lua ## @@ -453,7 +476,7 @@ function _M.new(key, opts) values = nil, need_reload = true, Review

[GitHub] [apisix] gy09535 opened a new pull request #2164: fix: log-rotate.t is unstable

2020-09-03 Thread GitBox
gy09535 opened a new pull request #2164: URL: https://github.com/apache/apisix/pull/2164 ### What this PR does / why we need it: fix:https://github.com/apache/apisix/issues/2138 ### Pre-submission checklist: * [x ] Did you explain what problem does this PR solve? Or what

[GitHub] [apisix] membphis commented on pull request #2158: bugfix: it is unsafe to get the length of array if it contains any `n…

2020-09-03 Thread GitBox
membphis commented on pull request #2158: URL: https://github.com/apache/apisix/pull/2158#issuecomment-686875823 we'll get the wrong route list in APISIX by this case: ```shell ETCDCTL_API=2 etcdctl rm -r /apisix && make init make run curl -i

[GitHub] [apisix] tokers commented on pull request #2163: improve: cache parsed certs and pkeys to LRU cache

2020-09-03 Thread GitBox
tokers commented on pull request #2163: URL: https://github.com/apache/apisix/pull/2163#issuecomment-686863695 Related issue: https://github.com/apache/apisix/issues/2144 This is an automated message from the Apache Git

[GitHub] [apisix] cwtree commented on issue #487: Who is using APISIX?(欢迎使用 APISIX 的个人和公司在此留言)

2020-09-03 Thread GitBox
cwtree commented on issue #487: URL: https://github.com/apache/apisix/issues/487#issuecomment-686857810 组织:中国移动杭研 位置:中国 杭州 联系方式:chi...@cmhi.chinamobile.com 场景:WAF业务,多网站路由分组 This is an automated message from the

[apisix-dashboard] branch master updated: feat: update README (#439)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/master by this push: new f9db078 feat: update README

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #439: feat: update README

2020-09-03 Thread GitBox
juzhiyuan merged pull request #439: URL: https://github.com/apache/apisix-dashboard/pull/439 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

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

2020-09-03 Thread GitBox
Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r483344517 ## File path: apisix/core/config_etcd.lua ## @@ -453,7 +476,7 @@ function _M.new(key, opts) values = nil, need_reload = true, Review

[GitHub] [apisix] Yiyiyimu removed a comment on pull request #2036: feature: support etcd v3, by mocking v2 API

2020-09-03 Thread GitBox
Yiyiyimu removed a comment on pull request #2036: URL: https://github.com/apache/apisix/pull/2036#issuecomment-686846176 Hi @membphis I noticed there are still errrors in your latest commit like https://github.com/apache/apisix/runs/1046997001#step:6:423, but since there is no

[GitHub] [apisix] Yiyiyimu commented on pull request #2036: feature: support etcd v3, by mocking v2 API

2020-09-03 Thread GitBox
Yiyiyimu commented on pull request #2036: URL: https://github.com/apache/apisix/pull/2036#issuecomment-686846176 Hi @membphis I noticed there are still errrors in your latest commit like https://github.com/apache/apisix/runs/1046997001#step:6:423, but since there is no `no_error_log` add

[GitHub] [apisix-dashboard] nic-chen commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
nic-chen commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r483036011 ## File path: api/route/route.go ## @@ -209,6 +223,15 @@ func updateRoute(c *gin.Context) { } }()

[GitHub] [apisix] moonming commented on pull request #2153: Change: worker and cpu optimization

2020-09-03 Thread GitBox
moonming commented on pull request #2153: URL: https://github.com/apache/apisix/pull/2153#issuecomment-686533007 OK Vinci Xu 于 2020年9月3日周四 下午9:02写道: > *@ShiningRush* commented on this pull request. > -- > > In bin/apisix >

[GitHub] [apisix-dashboard] nic-chen commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
nic-chen commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686529095 @ShiningRush We could refactor the current code first, and then implement other features step by step.

[GitHub] [apisix-dashboard] ShiningRush commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
ShiningRush commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686488717 Fine, I will make a simple design image about watching ETCD in this week. BTW: I have read the code of manager API, but I do not see any code about RBAC, is the

[GitHub] [apisix] ShiningRush commented on a change in pull request #2153: Change: worker and cpu optimization

2020-09-03 Thread GitBox
ShiningRush commented on a change in pull request #2153: URL: https://github.com/apache/apisix/pull/2153#discussion_r482959730 ## File path: bin/apisix ## @@ -821,6 +821,11 @@ local function init() sys_conf["dns_resolver"] = dns_addrs end +local

[GitHub] [apisix] tokers opened a new pull request #2163: improve: cache parsed certs and pkeys to LRU cache

2020-09-03 Thread GitBox
tokers opened a new pull request #2163: URL: https://github.com/apache/apisix/pull/2163 ### What this PR does / why we need it: The processes of `ngx_ssl.parse_pem_cert` and `ngx_ssl.parse_pem_priv_key` are relatively expensive, and keypair is inclined to be constant (for a long

[apisix-dashboard] branch juzhiyuan-patch-1 updated (25875b6 -> a099e1e)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from 25875b6 Update README.zh-CN.md add a099e1e Update multiple-node-build.yml No new

[apisix-dashboard] branch juzhiyuan-patch-1 updated (5a24147 -> 25875b6)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from 5a24147 Update README.md add 25875b6 Update README.zh-CN.md No new revisions were

[apisix-dashboard] branch juzhiyuan-patch-1 updated (f10f70e -> 5a24147)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from f10f70e feat: test node.js 8.x add 5a24147 Update README.md No new revisions were

[GitHub] [apisix-dashboard] juzhiyuan closed issue #422: Node.js Version test

2020-09-03 Thread GitBox
juzhiyuan closed issue #422: URL: https://github.com/apache/apisix-dashboard/issues/422 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

[apisix-dashboard] 01/01: feat: test node.js 8.x

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit f10f70ea2a500f9704b7998c4c960e9e490185e4 Author: 琚致远 AuthorDate: Thu Sep 3 18:55:11 2020 +0800

[GitHub] [apisix-dashboard] juzhiyuan opened a new pull request #439: feat: test node.js 8.x

2020-09-03 Thread GitBox
juzhiyuan opened a new pull request #439: URL: https://github.com/apache/apisix-dashboard/pull/439 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bug fix - [ ] New feature provided - [x] Improve performance ___ ###

[apisix-dashboard] branch juzhiyuan-patch-1 created (now f10f70e)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at f10f70e feat: test node.js 8.x This branch includes the following new commits: new

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #438: feat: added multiple node builds action

2020-09-03 Thread GitBox
juzhiyuan merged pull request #438: URL: https://github.com/apache/apisix-dashboard/pull/438 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

[apisix-dashboard] branch master updated: feat: added multiple node builds action (#438)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/master by this push: new b34d5f5 feat: added multiple node

[GitHub] [apisix-dashboard] juzhiyuan opened a new pull request #438: feat: added multiple node builds action

2020-09-03 Thread GitBox
juzhiyuan opened a new pull request #438: URL: https://github.com/apache/apisix-dashboard/pull/438 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bug fix - [ ] New feature provided - [x] Improve performance ### New

[apisix-dashboard] 01/01: feat: added multiple node builds action

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit f8483427dc603bac4605c01b23014282b0bf906e Author: 琚致远 AuthorDate: Thu Sep 3 18:38:24 2020 +0800

[apisix-dashboard] branch juzhiyuan-patch-1 created (now f848342)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at f848342 feat: added multiple node builds action This branch includes the following new

[GitHub] [apisix] y934317258 commented on issue #2157: request help: apisix 1.3 upgrade apisix 1.5,admin api occur 403

2020-09-03 Thread GitBox
y934317258 commented on issue #2157: URL: https://github.com/apache/apisix/issues/2157#issuecomment-686400297 After verification, 1.5 and above, when allow_admin is empty, all access is forbidden. Is this a bug? This is an

[GitHub] [apisix] idbeta opened a new issue #2162: bug: after used luarocks to install APISIX in centos7.6(ARM64), an error was occurred

2020-09-03 Thread GitBox
idbeta opened a new issue #2162: URL: https://github.com/apache/apisix/issues/2162 ### Issue description It looks like APISIX was ​​not installed correctly. [root@ecs-9148 /]# cat /usr/local/bin/apisix #!/bin/sh LUAROCKS_SYSCONFDIR='/usr/local/etc/luarocks' exec

[GitHub] [apisix-dashboard] juzhiyuan edited a comment on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
juzhiyuan edited a comment on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686396088 cc @bzp2010 @gxthrj @moonming @membphis @ShiningRush @tiannian @LiteSun This is an automated

[GitHub] [apisix] idbeta opened a new issue #2161: bug: After the 1.5Rpm package was installed in centos7.6(ARM64), an error was occurred

2020-09-03 Thread GitBox
idbeta opened a new issue #2161: URL: https://github.com/apache/apisix/issues/2161 ### Issue description After the 1.5Rpm package was installed in centos7.6(ARM64), an error was occurred [root@ecs-9148 ~]# apisix start nginx: [error] init_by_lua error: error loading module 'lfs'

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #437: Feature:route publish management

2020-09-03 Thread GitBox
juzhiyuan commented on issue #437: URL: https://github.com/apache/apisix-dashboard/issues/437#issuecomment-686396700 cc @moonming @nic-chen This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
juzhiyuan commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686396088 cc @bzp2010 @gxthrj @moonming @membphis @ShiningRush @liuxiran @tiannian @LiteSun This is an automated

[GitHub] [apisix] moonming commented on a change in pull request #2158: bugfix: it is unsafe to get the length of array if it contains any `n…

2020-09-03 Thread GitBox
moonming commented on a change in pull request #2158: URL: https://github.com/apache/apisix/pull/2158#discussion_r482866183 ## File path: apisix/core/config_etcd.lua ## @@ -172,8 +171,8 @@ local function sync_data(self) self.values_hash = nil end -

[GitHub] [apisix] idbeta opened a new issue #2160: bug: grpc-transcode plugin crash in ARM64 environment

2020-09-03 Thread GitBox
idbeta opened a new issue #2160: URL: https://github.com/apache/apisix/issues/2160 ### Issue description [error] 11324#11324: init_worker_by_lua error: ...isix/apache-apisix-1.5/apisix/plugins/grpc-transcode.lua:94: attempt to index upvalue 'proto' (a userdata value) stack

[GitHub] [apisix] idbeta opened a new issue #2159: bug:prometheus plugin crash in ARM64 environment

2020-09-03 Thread GitBox
idbeta opened a new issue #2159: URL: https://github.com/apache/apisix/issues/2159 ### Issue description [error] 11324#11324: *1 [lua] plugin.lua:66: load_plugin(): failed to load plugin [prometheus] err: bad light userdata pointer, context: init_worker_by_lua* ###

[GitHub] [apisix] y934317258 commented on issue #2157: request help: apisix 1.3 upgrade apisix 1.5,admin api occur 403

2020-09-03 Thread GitBox
y934317258 commented on issue #2157: URL: https://github.com/apache/apisix/issues/2157#issuecomment-686388222 ![image](https://user-images.githubusercontent.com/68832358/92101575-e7da8780-ee0f-11ea-914e-12d1c246f488.png) change api six version from 1.3 to 1.5 only

[GitHub] [apisix-dashboard] liuxiran opened a new issue #437: Feature:route publish management

2020-09-03 Thread GitBox
liuxiran opened a new issue #437: URL: https://github.com/apache/apisix-dashboard/issues/437 Please answer these questions before submitting your issue. - Why do you submit this issue? - [ ] Question or discussion - [ ] Bug - [ ] Requirements - [ ] Feature or performance

[GitHub] [apisix] membphis opened a new pull request #2158: bugfix: it is unsafe to get the length of array if it contains any `n…

2020-09-03 Thread GitBox
membphis opened a new pull request #2158: URL: https://github.com/apache/apisix/pull/2158 …il` object, and we may get wrong length if it contain any `nil`. https://stackoverflow.com/questions/25903170/operator-nil-values-at-end-of-array ### What this PR does / why

[GitHub] [apisix] moonming commented on pull request #2148: fix: serverless plugin not work in header_filter phase

2020-09-03 Thread GitBox
moonming commented on pull request #2148: URL: https://github.com/apache/apisix/pull/2148#issuecomment-686370229 No, just commit in the same PR. I will squash commits before merge. This is an automated message from the

[GitHub] [apisix] mddmzl commented on pull request #2148: fix: serverless plugin not work in header_filter phase

2020-09-03 Thread GitBox
mddmzl commented on pull request #2148: URL: https://github.com/apache/apisix/pull/2148#issuecomment-686369379 @moonming should i squash the two commits into one and create a new pull request? This is an automated message

[GitHub] [apisix-dashboard] nic-chen commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
nic-chen commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686365420 I have updated https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686153897 Note that we need to implement the features using by relational database

[GitHub] [apisix] y934317258 opened a new issue #2157: request help: apisix 1.3 upgrade apisix 1.5,admin api occur 403

2020-09-03 Thread GitBox
y934317258 opened a new issue #2157: URL: https://github.com/apache/apisix/issues/2157 ### Issue description apisix 1.3 upgrade apisix 1.5,admin api occur 403

[GitHub] [apisix-dashboard] juzhiyuan edited a comment on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
juzhiyuan edited a comment on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686153897 # APIs ## Route ### Create POST `/routes` ### Delete DELETE `/routes/:id ` ### Update PUT `/routes/:id` ### Fetch List GET

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
juzhiyuan commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686348210 cc @gxthrj @nic-chen This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix-dashboard] liuxiran commented on issue #434: Requirements for refactoring the Dashboard with Manager API

2020-09-03 Thread GitBox
liuxiran commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-686339062 if routeGroup merged, there are some **routeGroup** apis here: @juzhiyuan @nic-chen Create POST `/routegroups` Delete DELETE `/routegroups/:gid`

[GitHub] [apisix] membphis commented on a change in pull request #2139: bugfix: grpc-transcode plugin converts http/json parameters abnormally

2020-09-03 Thread GitBox
membphis commented on a change in pull request #2139: URL: https://github.com/apache/apisix/pull/2139#discussion_r482798553 ## File path: t/plugin/grpc-transcode.t ## @@ -572,10 +572,10 @@ passed === TEST 20: hit route --- request POST /grpctest -{"message":"Hello world",

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r482730584 ## File path: api/route/route_group.go ## @@ -0,0 +1,195 @@ +package route + +import ( + "encoding/json" + "fmt" + "net/http" +

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r482788436 ## File path: api/route/route.go ## @@ -101,11 +101,15 @@ func listRoute(c *gin.Context) { db = db.Where("upstream_nodes like ?

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r482788323 ## File path: api/route/route_group.go ## @@ -0,0 +1,195 @@ +package route + +import ( + "encoding/json" + "fmt" + "net/http" +

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r481901228 ## File path: api/service/route_group.go ## @@ -0,0 +1,126 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [apisix] wangyusheng1985 commented on issue #487: Who is using APISIX?(欢迎使用 APISIX 的个人和公司在此留言)

2020-09-03 Thread GitBox
wangyusheng1985 commented on issue #487: URL: https://github.com/apache/apisix/issues/487#issuecomment-686327469 Organization: MiaobiAI .Inc Location: Beijing, China Contact: wangyush...@aimiaobi.com Scenario: to manage dynamic proxy and pass proxy api 组织:MiaobiAI 地址:中国

[GitHub] [apisix] DHB-liuhong opened a new issue #2156: bug: can not run apisix on ARM

2020-09-03 Thread GitBox
DHB-liuhong opened a new issue #2156: URL: https://github.com/apache/apisix/issues/2156 ### Issue description ### Environment * apisix version (cmd: `apisix version`): 1.4 * OS: arm linux 7.6 ### Minimal test code / Steps to reproduce the issue 1. apisix start

[GitHub] [apisix] ChelinTsien commented on pull request #1960: add AK/SK auth plugin

2020-09-03 Thread GitBox
ChelinTsien commented on pull request #1960: URL: https://github.com/apache/apisix/pull/1960#issuecomment-686294354 OK, I thought this pr was not adopted, I will find a time to add documentation and test cases This is an

[GitHub] [apisix] moonming merged pull request #2145: fix: remove useless `)` in CODE_STYLE.md

2020-09-03 Thread GitBox
moonming merged pull request #2145: URL: https://github.com/apache/apisix/pull/2145 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

[apisix] branch master updated: doc: remove useless `)` in code-style.(#2145)

2020-09-03 Thread wenming
This is an automated email from the ASF dual-hosted git repository. wenming pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git The following commit(s) were added to refs/heads/master by this push: new ea7b7dd doc: remove useless `)` in

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r482730584 ## File path: api/route/route_group.go ## @@ -0,0 +1,195 @@ +package route + +import ( + "encoding/json" + "fmt" + "net/http" +

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r482730584 ## File path: api/route/route_group.go ## @@ -0,0 +1,195 @@ +package route + +import ( + "encoding/json" + "fmt" + "net/http" +

[GitHub] [apisix] gxthrj opened a new issue #2155: request help: add field `desc` for consumer

2020-09-03 Thread GitBox
gxthrj opened a new issue #2155: URL: https://github.com/apache/apisix/issues/2155 ### Issue description ### Environment * apisix version (cmd: `apisix version`): * OS: This is an automated message

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #421: test: add test case for docker compose

2020-09-03 Thread GitBox
juzhiyuan commented on pull request #421: URL: https://github.com/apache/apisix-dashboard/pull/421#issuecomment-686277464 @nic-chen Ready for review or just in pending? This is an automated message from the Apache Git

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #426: fix: no response when clicking [createNew] button in create route res…

2020-09-03 Thread GitBox
juzhiyuan commented on pull request #426: URL: https://github.com/apache/apisix-dashboard/pull/426#issuecomment-686277272 cc @LiteSun This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix-dashboard] juzhiyuan closed issue #427: bug: there is no response clicking 'createNew' button in create route resultView

2020-09-03 Thread GitBox
juzhiyuan closed issue #427: URL: https://github.com/apache/apisix-dashboard/issues/427 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

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #426: fix: no response when clicking [createNew] button in create route res…

2020-09-03 Thread GitBox
juzhiyuan merged pull request #426: URL: https://github.com/apache/apisix-dashboard/pull/426 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

[apisix-dashboard] branch master updated: fix: no response when clicking [createNew] button in create route resultView (#426)

2020-09-03 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/master by this push: new 476a2cb fix: no response when

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #433: Feat: dashboard support route group

2020-09-03 Thread GitBox
juzhiyuan commented on pull request #433: URL: https://github.com/apache/apisix-dashboard/pull/433#issuecomment-686275843 waiting for #432 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
juzhiyuan commented on pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#issuecomment-686276002 @nic-chen any update? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #436: fix: httpheaderrewrite data lost when view update in create route step2

2020-09-03 Thread GitBox
juzhiyuan commented on pull request #436: URL: https://github.com/apache/apisix-dashboard/pull/436#issuecomment-686275654 @LiteSun Review here This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #382: dashboard api error

2020-09-03 Thread GitBox
juzhiyuan commented on issue #382: URL: https://github.com/apache/apisix-dashboard/issues/382#issuecomment-686274794 > me too! APISIX 1.5 > > tail -f access.log > 10.111.96.23 - - [24/Aug/2020:13:30:48 +0800] 10.111.96.95:9080 "GET /apisix/favicon.ico HTTP/1.1" 404 54 0.000

[GitHub] [apisix-dashboard] juzhiyuan closed issue #382: dashboard api error

2020-09-03 Thread GitBox
juzhiyuan closed issue #382: URL: https://github.com/apache/apisix-dashboard/issues/382 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

[GitHub] [apisix] moonming commented on pull request #2148: fix: serverless plugin not work in header_filter phase

2020-09-03 Thread GitBox
moonming commented on pull request #2148: URL: https://github.com/apache/apisix/pull/2148#issuecomment-686274434 @mddmzl good catch  please fix https://github.com/apache/apisix/blob/master/t/admin/schema.t#L160 and https://github.com/apache/apisix/blob/master/t/admin/schema.t#L200

[GitHub] [apisix] moonming opened a new issue #2154: bug: failed test cases about parse yaml config

2020-09-03 Thread GitBox
moonming opened a new issue #2154: URL: https://github.com/apache/apisix/issues/2154 https://github.com/apache/apisix/pull/2148/checks?check_run_id=1060755809#step:6:448 https://github.com/apache/apisix/pull/2148/checks?check_run_id=1060755809#step:6:465

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #432: Feat: manage api support route group

2020-09-03 Thread GitBox
liuxiran commented on a change in pull request #432: URL: https://github.com/apache/apisix-dashboard/pull/432#discussion_r481901228 ## File path: api/service/route_group.go ## @@ -0,0 +1,126 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *