[GitHub] [apisix] membphis commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirom

2020-09-10 Thread GitBox
membphis commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-690032838 Improving performance mainly depends on the quality of the rules. For runtime selection, if better performance is required, streaming matching such as hyperscan is needed. --

[GitHub] [apisix] membphis commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirom

2020-09-10 Thread GitBox
membphis commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-690034015 you can take a look at this benchmark result: https://rust-leipzig.github.io/regex/2017/03/28/comparison-of-regex-engines/ --

[GitHub] [apisix] membphis commented on issue #2196: feature(kafka-logger): need to read the request body data if it was cached to local file

2020-09-10 Thread GitBox
membphis commented on issue #2196: URL: https://github.com/apache/apisix/issues/2196#issuecomment-690036027 https://github.com/apache/apisix/blob/master/apisix/utils/log-util.lua#L67 ```lua local body_file = ngx.req.get_body_file() if body_file then -- we only readed the

[GitHub] [apisix] membphis commented on issue #2196: feature(kafka-logger): need to read the request body data if it was cached to local file

2020-09-10 Thread GitBox
membphis commented on issue #2196: URL: https://github.com/apache/apisix/issues/2196#issuecomment-690036238 @swayamraina I have assigned this issue to you, many thx This is an automated message from the Apache Git Service. To

[GitHub] [apisix] gy09535 commented on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 commented on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690039140 > I think we can send the log to aliyun sls is an easy way, and that is a stable way. > > use `lua-resty-logger-socket` is a better way. Agree with you and I use cosocket

[GitHub] [apisix] membphis commented on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
membphis commented on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690041009 welcome PR,then the community can share this feature. ^_^ This is an automated message from the Apache Git Service. T

[GitHub] [apisix] membphis commented on pull request #2163: improve: cache parsed SSL certs and pkeys

2020-09-10 Thread GitBox
membphis commented on pull request #2163: URL: https://github.com/apache/apisix/pull/2163#issuecomment-690043885 > @membphis changed. approved, need one more `approved`, then your PR can be merged ^_^ This is an automa

[GitHub] [apisix] linsir commented on issue #2201: request help: use core.log Output log, output to which file

2020-09-10 Thread GitBox
linsir commented on issue #2201: URL: https://github.com/apache/apisix/issues/2201#issuecomment-690048160 no, should change the `error_log_level`. @y934317258

[GitHub] [apisix] moonming merged pull request #2163: improve: cache parsed SSL certs and pkeys

2020-09-10 Thread GitBox
moonming merged pull request #2163: URL: https://github.com/apache/apisix/pull/2163 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: improve: cache parsed certs and pkeys to LRU cache (#2163)

2020-09-10 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 1556cd0 improve: cache parsed certs and pkeys

[GitHub] [apisix-dashboard] membphis commented on a change in pull request #451: feat(route): add publish and offline to route

2020-09-10 Thread GitBox
membphis commented on a change in pull request #451: URL: https://github.com/apache/apisix-dashboard/pull/451#discussion_r486130770 ## File path: src/pages/Route/typing.d.ts ## @@ -113,6 +116,7 @@ declare namespace RouteModule { // Request Body or Response Data for API ty

[apisix] branch juzhiyuan-patch-1 updated (9e5b3b1 -> 77a9fcc)

2020-09-10 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix.git. from 9e5b3b1 Update health-check.md add 1556cd0 improve: cache parsed certs and pkeys to LRU cache (#2

[GitHub] [apisix] maxluo1992 opened a new issue #2206: request help:

2020-09-10 Thread GitBox
maxluo1992 opened a new issue #2206: URL: https://github.com/apache/apisix/issues/2206 ### Issue description I use the docker image to open apisix in the k8s environment, but an error 401 Authorization Required is reported when accessing the resources of the dashboard. But I did not rep

[GitHub] [apisix] MrJLuo opened a new issue #2207: request help: websocket proxy

2020-09-10 Thread GitBox
MrJLuo opened a new issue #2207: URL: https://github.com/apache/apisix/issues/2207 ### Issue description 1.Use APISIX to proxy the websocket protocol. If you need to use the http protocol to forward to the upstream, can you only rewrite the header through the proxy-rewrite plugin? 2.

[GitHub] [apisix] membphis commented on issue #2207: request help: websocket proxy

2020-09-10 Thread GitBox
membphis commented on issue #2207: URL: https://github.com/apache/apisix/issues/2207#issuecomment-69005 By default, apisix transparently transmits user parameters. If you need to rewrite a client request (such as a scheme), you need to rewrite it through the proxy-rewrite plug-in

[GitHub] [apisix] mlick opened a new issue #2208: 没找到类似kong的strip_path这个功能

2020-09-10 Thread GitBox
mlick opened a new issue #2208: URL: https://github.com/apache/apisix/issues/2208 ### Issue description ![image](https://user-images.githubusercontent.com/8747500/92697047-5a51e700-f37d-11ea-9107-0b8687dd068e.png) 类似于nginx这个配置,如果后面有个根目录 / ,则这个 /sugrec 就不会追加到 baidu.com 这个地址后面, 如果没有

[GitHub] [apisix] moonming commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-10 Thread GitBox
moonming commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486136441 ## File path: conf/config.yaml ## @@ -14,10 +14,23 @@ # See the License for the specific language governing permissions and # limitations under the Lice

[GitHub] [apisix] moonming commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-10 Thread GitBox
moonming commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486140004 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204&page=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix] moonming commented on a change in pull request #2092: bug: removed default access token for Admin API.

2020-09-10 Thread GitBox
moonming commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r486140991 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204&page=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix] Firstsawyou opened a new pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
Firstsawyou opened a new pull request #2209: URL: https://github.com/apache/apisix/pull/2209 ### What this PR does / why we need it: feature: #2202 ### Pre-submission checklist: * [x] Did you explain what problem does this PR solve? Or what new features have be

[GitHub] [apisix] moonming commented on pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
moonming commented on pull request #2209: URL: https://github.com/apache/apisix/pull/2209#issuecomment-690066620 Is this PR still a draft? This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [apisix-dashboard] liubinqi2007 opened a new issue #463: Custom plug-in profiles and links

2020-09-10 Thread GitBox
liubinqi2007 opened a new issue #463: URL: https://github.com/apache/apisix-dashboard/issues/463  https://user-images.githubusercontent.com/17898385/92698954-971edd80-f37f-11ea-8463-d37a1133f5dc.png";> This is an

[GitHub] [apisix] moonming commented on issue #2208: 没找到类似kong的strip_path这个功能

2020-09-10 Thread GitBox
moonming commented on issue #2208: URL: https://github.com/apache/apisix/issues/2208#issuecomment-690069365 Is this configure? https://github.com/apache/apisix/blob/master/conf/config-default.yaml#L83 This is an automated me

[GitHub] [apisix] moonming commented on issue #2208: 没找到类似kong的strip_path这个功能

2020-09-10 Thread GitBox
moonming commented on issue #2208: URL: https://github.com/apache/apisix/issues/2208#issuecomment-690069517 Please use English in the public channel, thx This is an automated message from the Apache Git Service. To respond to

[GitHub] [apisix] moonming commented on issue #2206: request help:

2020-09-10 Thread GitBox
moonming commented on issue #2206: URL: https://github.com/apache/apisix/issues/2206#issuecomment-690073030 @juzhiyuan please take a look This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [apisix] moonming commented on issue #2204: bug: “prevNode” should not return after setting a route

2020-09-10 Thread GitBox
moonming commented on issue #2204: URL: https://github.com/apache/apisix/issues/2204#issuecomment-690073526 yes, agreed, we can remove the `prevNode` section now. Welcome PR This is an automated message from the Apache Gi

[GitHub] [apisix] moonming commented on issue #2203: Is there a version release plan?

2020-09-10 Thread GitBox
moonming commented on issue #2203: URL: https://github.com/apache/apisix/issues/2203#issuecomment-690073824 please take a look at https://github.com/apache/apisix/milestones This is an automated message from the Apache Git Se

[GitHub] [apisix] moonming commented on issue #2198: request help: How to deploy apisix

2020-09-10 Thread GitBox
moonming commented on issue #2198: URL: https://github.com/apache/apisix/issues/2198#issuecomment-690075073 Please use English in the public channel, thx Your understanding is right, DP and CP are communicate through etcd, here is the technical architecture of Apache APISIX: https://git

[GitHub] [apisix] moonming commented on issue #2199: request help: The parameter meaning of JWT auth plug-in. What are the functions of parameter configuration in consumer and routing

2020-09-10 Thread GitBox
moonming commented on issue #2199: URL: https://github.com/apache/apisix/issues/2199#issuecomment-690075487 @juzhiyuan please take a look. Is this a TODO? This is an automated message from the Apache Git Service. To respond t

[GitHub] [apisix] spacewander merged pull request #2181: chore: add string utility for simplicity and correctness

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

[apisix] branch master updated: chore: add string utility for simplicity and correctness (#2181)

2020-09-10 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander 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 37817e4 chore: add string utility for simp

[GitHub] [apisix] spacewander commented on issue #2194: bug: error when route not found

2020-09-10 Thread GitBox
spacewander commented on issue #2194: URL: https://github.com/apache/apisix/issues/2194#issuecomment-690091100 Please attach your report to #2006 instead. This is an automated message from the Apache Git Service. To respond t

[GitHub] [apisix] spacewander closed issue #2194: bug: error when route not found

2020-09-10 Thread GitBox
spacewander closed issue #2194: URL: https://github.com/apache/apisix/issues/2194 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 th

[GitHub] [apisix] maxluo1992 commented on issue #2206: request help:

2020-09-10 Thread GitBox
maxluo1992 commented on issue #2206: URL: https://github.com/apache/apisix/issues/2206#issuecomment-690093689 Check the nginx error.log, which reads: [lua] init.lua:95: handler(): failed to check token: missing apikey, client: 10.244.3.1, server: , request: "GET /apisix/admin/upstreams H

[GitHub] [apisix] maxluo1992 edited a comment on issue #2206: request help:

2020-09-10 Thread GitBox
maxluo1992 edited a comment on issue #2206: URL: https://github.com/apache/apisix/issues/2206#issuecomment-690093689 Check the nginx error.log, which reads: [lua] init.lua:95: handler(): failed to check token: missing apikey, client: 10.244.3.1, server: , request: "GET /apisix/admin/upst

[apisix] branch juzhiyuan-patch-1 updated (77a9fcc -> 4af66a2)

2020-09-10 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix.git. from 77a9fcc Merge branch 'master' into juzhiyuan-patch-1 add 37817e4 chore: add string utility for si

[GitHub] [apisix] membphis commented on pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
membphis commented on pull request #2209: URL: https://github.com/apache/apisix/pull/2209#issuecomment-690114593 @Firstsawyou this PR is missing test case. if it is still in process, you should choose `draft` when you create new PR. pls take a look at the output of CI: https

[GitHub] [apisix] Firstsawyou commented on pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
Firstsawyou commented on pull request #2209: URL: https://github.com/apache/apisix/pull/2209#issuecomment-690118556 > Is this PR still a draft? yes. This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix] Firstsawyou commented on pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
Firstsawyou commented on pull request #2209: URL: https://github.com/apache/apisix/pull/2209#issuecomment-690118919 > @Firstsawyou this PR is missing test case. if it is still in process, you should choose `draft` when you create new PR. > > pls take a look at the output of CI: >

[GitHub] [apisix] moonming commented on pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

2020-09-10 Thread GitBox
moonming commented on pull request #2209: URL: https://github.com/apache/apisix/pull/2209#issuecomment-690119215 please add the WIP or draft title. Thanks, Ming Wen Twitter: _WenMing Firstsawyou 于2020年9月10日周四 下午5:44写道: > Is this PR still a draft? > > yes

[GitHub] [apisix] moonming commented on pull request #2193: fixed #2191

2020-09-10 Thread GitBox
moonming commented on pull request #2193: URL: https://github.com/apache/apisix/pull/2193#issuecomment-690119857 thx, it will be better if you can add test cases. This is an automated message from the Apache Git Service. To r

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486203895 ## File path: apisix/plugins/hmac-auth.lua ## @@ -0,0 +1,291 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486208741 ## File path: apisix/plugins/hmac-auth.lua ## @@ -0,0 +1,291 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486218030 ## File path: t/APISIX.pm ## @@ -96,6 +96,18 @@ etcd: _EOC_ } +my $custom_hmac_auth = $ENV{"CUSTOM_HMAC_AUTH"} || "false"; Review comment: I g

[GitHub] [apisix] nic-chen merged pull request #2139: bugfix: grpc-transcode plugin converts http/json parameters abnormally

2020-09-10 Thread GitBox
nic-chen merged pull request #2139: URL: https://github.com/apache/apisix/pull/2139 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: bugfix: grpc-transcode plugin converts http/json parameters abnormally (#2139)

2020-09-10 Thread chenjunxu
This is an automated email from the ASF dual-hosted git repository. chenjunxu 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 087f87a bugfix: grpc-transcode plugin conver

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486218352 ## File path: t/APISIX.pm ## @@ -96,6 +96,18 @@ etcd: _EOC_ } +my $custom_hmac_auth = $ENV{"CUSTOM_HMAC_AUTH"} || "false"; +if ($custom_hmac_auth eq "

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486218352 ## File path: t/APISIX.pm ## @@ -96,6 +96,18 @@ etcd: _EOC_ } +my $custom_hmac_auth = $ENV{"CUSTOM_HMAC_AUTH"} || "false"; +if ($custom_hmac_auth eq "

[apisix] branch juzhiyuan-patch-1 updated (4af66a2 -> 5c038e5)

2020-09-10 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch juzhiyuan-patch-1 in repository https://gitbox.apache.org/repos/asf/apisix.git. from 4af66a2 Merge branch 'master' into juzhiyuan-patch-1 add 087f87a bugfix: grpc-transcode plugin co

[apisix] branch master updated: doc: fixed typo in `health-check.md` (#2200)

2020-09-10 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis 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 98da870 doc: fixed typo in `health-check.md`

[GitHub] [apisix] membphis merged pull request #2200: fix typo Update health-check.md

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

[GitHub] [apisix] tokers commented on issue #2144: optimization: cache the parsed certs and pkeys into LRU queues to reduce the parsing overheads

2020-09-10 Thread GitBox
tokers commented on issue #2144: URL: https://github.com/apache/apisix/issues/2144#issuecomment-690234104 The related PR was merged, i'm closing this issue. This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix] tokers closed issue #2144: optimization: cache the parsed certs and pkeys into LRU queues to reduce the parsing overheads

2020-09-10 Thread GitBox
tokers closed issue #2144: URL: https://github.com/apache/apisix/issues/2144 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 spe

[GitHub] [apisix] spacewander commented on a change in pull request #2174: optimize set nginx conf env

2020-09-10 Thread GitBox
spacewander commented on a change in pull request #2174: URL: https://github.com/apache/apisix/pull/2174#discussion_r486286638 ## File path: conf/config-default.yaml ## @@ -115,6 +115,8 @@ nginx_config: # config for render the template to genarate n work

[GitHub] [apisix] nic-chen commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
nic-chen commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486312169 ## File path: apisix/plugins/hmac-auth.lua ## @@ -0,0 +1,291 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor

[GitHub] [apisix] EnableAsync commented on issue #2125: request help: move etcd cluster in docker for CI

2020-09-10 Thread GitBox
EnableAsync commented on issue #2125: URL: https://github.com/apache/apisix/issues/2125#issuecomment-690267645 I tried to deploy etcd with docker and found that the latest version of etcd did not pass all the test cases. https://github.com/EnableAsync/apisix/actions/runs/247908169

[GitHub] [apisix] nic-chen commented on a change in pull request #2192: feat: add AK/SK auth plugin

2020-09-10 Thread GitBox
nic-chen commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r486314631 ## File path: apisix/plugins/hmac-auth.lua ## @@ -0,0 +1,291 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor

[GitHub] [apisix] agile6v opened a new pull request #2210: doc: fix format

2020-09-10 Thread GitBox
agile6v opened a new pull request #2210: URL: https://github.com/apache/apisix/pull/2210 ### What this PR does / why we need it: fix doc format ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new features have bee

[GitHub] [apisix] juzhiyuan commented on issue #2206: request help:

2020-09-10 Thread GitBox
juzhiyuan commented on issue #2206: URL: https://github.com/apache/apisix/issues/2206#issuecomment-690333590 > http://192.168.63.184:31191/apisix/dashboard May I know which version of the dashboard are you using? v1.0?

[GitHub] [apisix-dashboard] maxluo1992 opened a new issue #464: request help:

2020-09-10 Thread GitBox
maxluo1992 opened a new issue #464: URL: https://github.com/apache/apisix-dashboard/issues/464 ### Issue description I use the docker image to open apisix in the k8s environment, but an error 401 Authorization Required is reported when accessing the resources of the dashboard. But I did

[GitHub] [apisix] juzhiyuan commented on issue #2199: request help: The parameter meaning of JWT auth plug-in. What are the functions of parameter configuration in consumer and routing

2020-09-10 Thread GitBox
juzhiyuan commented on issue #2199: URL: https://github.com/apache/apisix/issues/2199#issuecomment-690335645 https://github.com/apache/apisix/issues/2082#issue-681772868 We have fixed a couple of plugins' schema, I will add this one to the TODO list. ---

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #462: chore: remove unnecessary license file

2020-09-10 Thread GitBox
juzhiyuan merged pull request #462: URL: https://github.com/apache/apisix-dashboard/pull/462 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 (54f7a98 -> 1eb1524)

2020-09-10 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from 54f7a98 add tools to check ASF headers in code files. (#454) add 1eb1524 chore: remove unnecessar

[GitHub] [apisix-dashboard] juzhiyuan commented on pull request #460: feat: support ungroup in manager api

2020-09-10 Thread GitBox
juzhiyuan commented on pull request #460: URL: https://github.com/apache/apisix-dashboard/pull/460#issuecomment-690338834 @liuxiran Please add test cases, so we could move forward this PR. This is an automated message from th

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #463: Custom plug-in profiles and links

2020-09-10 Thread GitBox
juzhiyuan commented on issue #463: URL: https://github.com/apache/apisix-dashboard/issues/463#issuecomment-690350570 Sure, the description would be added once I have implemented the PluginModule. This is an automated messag

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #461: Increase the checking of .conf and .toml files. And delete the redundant configuration of the ASF-Release.cfg file.

2020-09-10 Thread GitBox
juzhiyuan commented on a change in pull request #461: URL: https://github.com/apache/apisix-dashboard/pull/461#discussion_r486418283 ## File path: .github/workflows/api_ci.yml ## @@ -54,6 +54,10 @@ jobs: run: | curl 127.0.0.1:9080 +- name: run Makefile Re

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #461: Increase the checking of .conf and .toml files. And delete the redundant configuration of the ASF-Release.cfg file.

2020-09-10 Thread GitBox
juzhiyuan commented on a change in pull request #461: URL: https://github.com/apache/apisix-dashboard/pull/461#discussion_r486418283 ## File path: .github/workflows/api_ci.yml ## @@ -54,6 +54,10 @@ jobs: run: | curl 127.0.0.1:9080 +- name: run Makefile Re

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #461: Increase the checking of .conf and .toml files. And delete the redundant configuration of the ASF-Release.cfg file.

2020-09-10 Thread GitBox
juzhiyuan merged pull request #461: URL: https://github.com/apache/apisix-dashboard/pull/461 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-dashboard] juzhiyuan merged pull request #455: ci: Add lint

2020-09-10 Thread GitBox
juzhiyuan merged pull request #455: URL: https://github.com/apache/apisix-dashboard/pull/455 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: ci: Add lint (#455)

2020-09-10 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 d6ba6cc ci: Add lint (#455) d6ba6c

[apisix-dashboard] branch master updated (1eb1524 -> d3678bc)

2020-09-10 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from 1eb1524 chore: remove unnecessary license file (#462) add d3678bc Increase the checking of .conf

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #458: bug: When creating a route, host should not be required

2020-09-10 Thread GitBox
juzhiyuan commented on issue #458: URL: https://github.com/apache/apisix-dashboard/issues/458#issuecomment-690358927 ping @membphis @nic-chen This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [apisix-dashboard] LiteSun commented on issue #458: bug: When creating a route, host should not be required

2020-09-10 Thread GitBox
LiteSun commented on issue #458: URL: https://github.com/apache/apisix-dashboard/issues/458#issuecomment-690387597 any update? This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [apisix] membphis opened a new issue #2212: chore: update the openresty to 1.17.8 for Tengine CI

2020-09-10 Thread GitBox
membphis opened a new issue #2212: URL: https://github.com/apache/apisix/issues/2212 https://github.com/apache/apisix/blob/master/.travis/linux_tengine_runner.sh#L66 This is an automated message from the Apache Git Servi

[GitHub] [apisix] membphis commented on pull request #2211: bugfix: use the route's `has_domain` attribute if the route contains upstream object

2020-09-10 Thread GitBox
membphis commented on pull request #2211: URL: https://github.com/apache/apisix/pull/2211#issuecomment-690463565 > some checks failed fail to access GitHub, it should be a network problem. rerun it again. This is an au

[GitHub] [apisix] gxthrj commented on pull request #2211: bugfix: use the route's `has_domain` attribute if the route contains upstream object

2020-09-10 Thread GitBox
gxthrj commented on pull request #2211: URL: https://github.com/apache/apisix/pull/2211#issuecomment-690432265 some checks failed This is an automated message from the Apache Git Service. To respond to the message, please log

[apisix-dashboard] branch master updated: Feat: dashboard support route group (#433)

2020-09-10 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 e2271b3 Feat: dashboard support ro

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

2020-09-10 Thread GitBox
juzhiyuan merged pull request #433: URL: https://github.com/apache/apisix-dashboard/pull/433 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] membphis opened a new pull request #2211: bugfix: use the route's `has_domain` attribute if the route contains upstream object

2020-09-10 Thread GitBox
membphis opened a new pull request #2211: URL: https://github.com/apache/apisix/pull/2211 ### What this PR does / why we need it: step: 1. create a route with an upstream object, and the upstream node contains domain. 2. create a service, no upstream object (eg: onl

[GitHub] [apisix-dashboard] liuxiran commented on a change in pull request #450: feat: add publish status to route in manager api

2020-09-10 Thread GitBox
liuxiran commented on a change in pull request #450: URL: https://github.com/apache/apisix-dashboard/pull/450#discussion_r486457935 ## File path: api/route/route_test.go ## @@ -0,0 +1,51 @@ +package route + +import ( + "github.com/apisix/manager-api/conf" + "github.

[GitHub] [apisix-dashboard] Firstsawyou opened a new pull request #465: feature:Independent license check operation file.

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

[apisix-dashboard] branch master updated: feature:Independent license check operation file. (#465)

2020-09-10 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-dashboard.git The following commit(s) were added to refs/heads/master by this push: new 885025e feature:Independent license

[GitHub] [apisix-dashboard] moonming merged pull request #465: feature:Independent license check operation file.

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

[GitHub] [apisix-dashboard] maxluo1992 commented on issue #464: request help: 401 when using apisix in k8s

2020-09-10 Thread GitBox
maxluo1992 commented on issue #464: URL: https://github.com/apache/apisix-dashboard/issues/464#issuecomment-690813400 Yes,my dashboard version is v1.0. This is an automated message from the Apache Git Service. To respond to

[GitHub] [apisix] gy09535 commented on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 commented on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to send logs to

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to send l

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix] moonming commented on issue #2125: request help: move etcd cluster in docker for CI

2020-09-10 Thread GitBox
moonming commented on issue #2125: URL: https://github.com/apache/apisix/issues/2125#issuecomment-690828117 @EnableAsync Apache APISIX only supports etcd v2 now, so the latest version of etcd not pass all test cases is expected. @membphis please take a look at tengine failed test cas

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix] gy09535 edited a comment on issue #2169: feature: how use apisix send log to aliyun sls

2020-09-10 Thread GitBox
gy09535 edited a comment on issue #2169: URL: https://github.com/apache/apisix/issues/2169#issuecomment-690824661 > welcome PR,then the community can share this feature. ^_^ I use cosocket and processor to send logs to sls , you can define different routes or services plugins to se

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #464: request help: 401 when using apisix in k8s

2020-09-10 Thread GitBox
juzhiyuan commented on issue #464: URL: https://github.com/apache/apisix-dashboard/issues/464#issuecomment-690831752 Please use the latest (master or 1.5 branches) if possible. This is an automated message from the Apache Gi

[GitHub] [apisix] fankeke commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirome

2020-09-10 Thread GitBox
fankeke commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-690832270 Improving performance mainly depends on the quality of the rules. === thanks for reply In our system, there are already 1+ million block urls, if every request sho

[GitHub] [apisix-dashboard] liuxiran commented on issue #417: Get routeInfo [websocket] field is not match the value enterd

2020-09-10 Thread GitBox
liuxiran commented on issue #417: URL: https://github.com/apache/apisix-dashboard/issues/417#issuecomment-690832706 update master, Api can return the right protocols now. close it. This is an automated message from the Apac

[GitHub] [apisix-dashboard] liuxiran closed issue #417: Get routeInfo [websocket] field is not match the value enterd

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

[GitHub] [apisix] mlick commented on issue #2208: Is Apache APISIX support strip_path in Kong?

2020-09-10 Thread GitBox
mlick commented on issue #2208: URL: https://github.com/apache/apisix/issues/2208#issuecomment-690833024 @moonming got it thanks! This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [apisix-dashboard] liubinqi2007 commented on issue #463: Custom plug-in profiles and links

2020-09-10 Thread GitBox
liubinqi2007 commented on issue #463: URL: https://github.com/apache/apisix-dashboard/issues/463#issuecomment-690835352 Has this functionality been implemented yet? Still need to develop This is an automated message from the

[GitHub] [apisix] powermanlc opened a new issue #2213: can router support uri+params style

2020-09-10 Thread GitBox
powermanlc opened a new issue #2213: URL: https://github.com/apache/apisix/issues/2213 ### Issue description I have a question, normally router matched /a/b/c, can support **/aa/bb/cc?d=1&e=2** this http get method, params connect the uri. ### Environment * apisix v

  1   2   >