[GitHub] [apisix-dashboard] stevenlee87 commented on issue #2476: bug: ssl upgrade display :"Invalid date"

2022-06-19 Thread GitBox
stevenlee87 commented on issue #2476: URL: https://github.com/apache/apisix-dashboard/issues/2476#issuecomment-1160032190 > Can you check the details of the request in the browser Network panel? This problem has nothing to do with the browser ?? I use apisix-dashboard to upload

[apisix] branch master updated: fix: the argument to usleep should be integer (#7271)

2022-06-19 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 c72c08737 fix: the argument to usleep shoul

[GitHub] [apisix] spacewander merged pull request #7271: fix: the argument to usleep should be integer

2022-06-19 Thread GitBox
spacewander merged PR #7271: URL: https://github.com/apache/apisix/pull/7271 -- 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-unsubscr...@a

[apisix] branch master updated: feat(deployment): support connecting to etcd via https (#7269)

2022-06-19 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 089e8a218 feat(deployment): support connect

[GitHub] [apisix] spacewander merged pull request #7269: feat(deployment): support connecting to etcd via https

2022-06-19 Thread GitBox
spacewander merged PR #7269: URL: https://github.com/apache/apisix/pull/7269 -- 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-unsubscr...@a

[apisix-website] branch master updated: docs: update too long blog title(part2) (#1158)

2022-06-19 Thread sylviasu
This is an automated email from the ASF dual-hosted git repository. sylviasu pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new ced64b95d3a docs: update too long blog

[GitHub] [apisix-website] SylviaBABY merged pull request #1158: docs: update too long blog title(part2)

2022-06-19 Thread GitBox
SylviaBABY merged PR #1158: URL: https://github.com/apache/apisix-website/pull/1158 -- 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-unsubs

[apisix] 02/07: fix(proxy-cache): bypass when method mismatch cache_method (#7111)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 375eb8a150976f0b6150af9d5a839038d4252a76 Author: soulbird AuthorDate: Tue May 24 15:01:27 2022 +0800 fix

[apisix] 06/07: fix: grpc-transcode request support object array (#7231)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 314d13cff555fe0b8110ad0053f6f44bba99d129 Author: 余茂林 <1090568...@qq.com> AuthorDate: Fri Jun 17 11:07:33 2022

[apisix] 05/07: refactor: merge grpc_server_example (#6653)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 7b06f140631dc2122e9201510005aa992527aa52 Author: 罗泽轩 AuthorDate: Mon Mar 21 17:27:49 2022 +0800 refactor

[apisix] 07/07: ci: pass

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 24eea265e46c9ea7fc13be2cbeb01cd1f1276963 Author: spacewander AuthorDate: Mon Jun 20 10:30:28 2022 +0800

[apisix] branch release/2.13 updated (9b0cc7a3b -> 24eea265e)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git from 9b0cc7a3b fix: hide 5xx error message from client (#6982) new 670213149 chore: validate etcd conf stri

[apisix] branch master updated: fix(traffic-split): the default timeout doesn't match the one in Nginx (#7277)

2022-06-19 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 ea7dda050 fix(traffic-split): the default t

[GitHub] [apisix] spacewander merged pull request #7277: fix(traffic-split): the default timeout doesn't match the one in Nginx

2022-06-19 Thread GitBox
spacewander merged PR #7277: URL: https://github.com/apache/apisix/pull/7277 -- 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-unsubscr...@a

[apisix] 03/07: fix: reduce memory usage when abnormal weights are given in chash (#7103)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit be890c38970882290855a49122e7ee4e796e59bc Author: 罗泽轩 AuthorDate: Wed May 25 10:10:24 2022 +0800 fix: red

[apisix] 04/07: fix(proxy-cache): allow nil ctx vars in cache key (#7168)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 82c233e2361c4755fb45ad6a35546d76b212a329 Author: PiteChen <49583093+worldwonde...@users.noreply.github.com> Au

[apisix] 01/07: chore: validate etcd conf strictly (#7245)

2022-06-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch release/2.13 in repository https://gitbox.apache.org/repos/asf/apisix.git commit 67021314906a7d678912134c66ab47e1444eaf71 Author: 罗泽轩 AuthorDate: Tue Jun 14 09:28:37 2022 +0800 chore: v

[GitHub] [apisix] spacewander merged pull request #7278: backport: commits to release/2.13

2022-06-19 Thread GitBox
spacewander merged PR #7278: URL: https://github.com/apache/apisix/pull/7278 -- 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-unsubscr...@a

[GitHub] [apisix-ingress-controller] dickens7 commented on issue #1086: feature request: ingress annotations support ApisixService

2022-06-19 Thread GitBox
dickens7 commented on issue #1086: URL: https://github.com/apache/apisix-ingress-controller/issues/1086#issuecomment-1160007376 I thought about it, what I need more should be `ApisixPluginConfig` instead of `ApisixService` In some simple scenarios that require the use of apisix plugi

[GitHub] [apisix] dickens7 commented on pull request #7252: fix(skywalking-logger, error-log-logger): skywalking service_instance_name $hostname is supported

2022-06-19 Thread GitBox
dickens7 commented on PR #7252: URL: https://github.com/apache/apisix/pull/7252#issuecomment-1159977534 > how about `^[a-zA-Z\-0-9\.]+` `service_instance_name` is user-defined and can be set to any value such as `#xxx`. This is just for testing, it should work. But I think m

[GitHub] [apisix] navendu-pottekkat commented on issue #7274: help request: Is there any documentation that can assist in building a development environment?

2022-06-19 Thread GitBox
navendu-pottekkat commented on issue #7274: URL: https://github.com/apache/apisix/issues/7274#issuecomment-1159975069 The doc added with https://github.com/apache/apisix/pull/7219 is pretty comprehensive and should contain information about how to build and run APISIX locally. And this is m

[GitHub] [apisix] tzssangglass commented on issue #7258: bug: read header may cause an abnormally high CPU

2022-06-19 Thread GitBox
tzssangglass commented on issue #7258: URL: https://github.com/apache/apisix/issues/7258#issuecomment-1159966615 > `core.request.get_header` in the plugin. Here, I think it is enough to get only one of the request headers we need. fair enough -- This is an automated message from th

[GitHub] [apisix] tzssangglass commented on pull request #7252: fix(skywalking-logger, error-log-logger): skywalking service_instance_name $hostname is supported

2022-06-19 Thread GitBox
tzssangglass commented on PR #7252: URL: https://github.com/apache/apisix/pull/7252#issuecomment-1159965047 how about `^[a-zA-Z\-0-9\.]+` -- 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 speci

[GitHub] [apisix] tzssangglass commented on issue #7276: bug: runtime error: /usr/local/openresty/lualib/resty/core/request.lua:116: bad argument #1 to 'lower' (string expected, got nil)

2022-06-19 Thread GitBox
tzssangglass commented on issue #7276: URL: https://github.com/apache/apisix/issues/7276#issuecomment-1159958227 The test cases I tried to reproduce ``` use t::APISIX 'no_plan'; add_block_preprocessor(sub { my ($block) = @_; my $http_config = $block->http_co

[GitHub] [apisix] nic-chen commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
nic-chen commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901241347 ## docs/zh/latest/plugins/prometheus.md: ## @@ -75,70 +102,52 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13 "upstream": {

[GitHub] [apisix] tzssangglass commented on issue #7276: bug: runtime error: /usr/local/openresty/lualib/resty/core/request.lua:116: bad argument #1 to 'lower' (string expected, got nil)

2022-06-19 Thread GitBox
tzssangglass commented on issue #7276: URL: https://github.com/apache/apisix/issues/7276#issuecomment-1159952038 > 2\. lua entry thread aborted: runtime error: /usr/local/openresty/lualib/resty/core/request.lua:116 can you show the full function stacks? -- This is an automated mess

[GitHub] [apisix] tzssangglass commented on issue #7272: limit-count 全局限流功能

2022-06-19 Thread GitBox
tzssangglass commented on issue #7272: URL: https://github.com/apache/apisix/issues/7272#issuecomment-1159950559 any error logs about it? -- 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 speci

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7219: docs: create page for "Building APISIX"

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7219: URL: https://github.com/apache/apisix/pull/7219#discussion_r901238899 ## docs/en/latest/building-apisix.md: ## @@ -0,0 +1,276 @@ +--- +id: building-apisix +title: Building APISIX from source +keywords: + - API gateway + - Apache APIS

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7219: docs: create page for "Building APISIX"

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7219: URL: https://github.com/apache/apisix/pull/7219#discussion_r901237423 ## docs/en/latest/building-apisix.md: ## @@ -0,0 +1,276 @@ +--- +id: building-apisix +title: Building APISIX from source +keywords: + - API gateway + - Apache APIS

[GitHub] [apisix] tzssangglass commented on issue #7274: help request: Is there any documentation that can assist in building a development environment?

2022-06-19 Thread GitBox
tzssangglass commented on issue #7274: URL: https://github.com/apache/apisix/issues/7274#issuecomment-1159943955 #7219 is for building from source code, not for development environments. But you can still refer to the Build APISIX dependencies section of it, they are the same. -- This is

[GitHub] [apisix] soulbird commented on issue #7258: bug: read header may cause an abnormally high CPU

2022-06-19 Thread GitBox
soulbird commented on issue #7258: URL: https://github.com/apache/apisix/issues/7258#issuecomment-1159932779 We can cache headers on non-critical code paths, such as calling `core.request.get_header` in the plugin. Here, I think it is enough to get only one of the request headers we need.

[GitHub] [apisix] tzssangglass commented on issue #7258: bug: read header may cause an abnormally high CPU

2022-06-19 Thread GitBox
tzssangglass commented on issue #7258: URL: https://github.com/apache/apisix/issues/7258#issuecomment-1159926758 > It seems that there isn't necessary to call it here, what about changing it to this? It's a question of balance. case 1: If the request gets only one header here i

[GitHub] [apisix] dickens7 commented on pull request #7252: fix(skywalking-logger, error-log-logger): skywalking service_instance_name $hostname is supported

2022-06-19 Thread GitBox
dickens7 commented on PR #7252: URL: https://github.com/apache/apisix/pull/7252#issuecomment-1159919402 > we can verify `hostname` like: > > https://github.com/apache/apisix/blob/afce02cea506eb4c70f2dd9d5230d18c7ab8b51e/t/plugin/server-info.t#L108 This won't work, because `$hos

[GitHub] [apisix] spacewander merged pull request #7266: feat: export some importent params for kafka-client

2022-06-19 Thread GitBox
spacewander merged PR #7266: URL: https://github.com/apache/apisix/pull/7266 -- 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-unsubscr...@a

[apisix] branch master updated: feat: export some importent params for kafka-client (#7266)

2022-06-19 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 067b2eb38 feat: export some importent param

[GitHub] [apisix] spacewander commented on pull request #7277: fix(traffic-split): the default timeout doesn't match the one in Nginx

2022-06-19 Thread GitBox
spacewander commented on PR #7277: URL: https://github.com/apache/apisix/pull/7277#issuecomment-1159916901 Yes, the default value of proxy_connect_timeout -- 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] tzssangglass commented on pull request #7252: fix(skywalking-logger, error-log-logger): skywalking service_instance_name $hostname is supported

2022-06-19 Thread GitBox
tzssangglass commented on PR #7252: URL: https://github.com/apache/apisix/pull/7252#issuecomment-1159916378 we can verify `hostname` like: https://github.com/apache/apisix/blob/master/t/plugin/server-info.t#L108 -- This is an automated message from the Apache Git Service. To respond to th

[GitHub] [apisix] tzssangglass commented on pull request #7277: fix(traffic-split): the default timeout doesn't match the one in Nginx

2022-06-19 Thread GitBox
tzssangglass commented on PR #7277: URL: https://github.com/apache/apisix/pull/7277#issuecomment-1159912409 here default means `proxy_connect_timeout`? -- 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 g

[GitHub] [apisix] tzssangglass commented on issue #7260: feat: As a user, I want to add header rewriting to traffict_split plugin, so that it simplifies the setting of multiple lanes

2022-06-19 Thread GitBox
tzssangglass commented on issue #7260: URL: https://github.com/apache/apisix/issues/7260#issuecomment-1159911733 > Or put the headers field to the upstream object, so we can use it in a more general way, not just traffic-split. LGTM -- This is an automated message from the Apache G

[GitHub] [apisix] tzssangglass closed issue #7261: bug: http method rewrite not works in proxy-rewrite plugin

2022-06-19 Thread GitBox
tzssangglass closed issue #7261: bug: http method rewrite not works in proxy-rewrite plugin URL: https://github.com/apache/apisix/issues/7261 -- 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 spec

[GitHub] [apisix] tzssangglass commented on issue #7261: bug: http method rewrite not works in proxy-rewrite plugin

2022-06-19 Thread GitBox
tzssangglass commented on issue #7261: URL: https://github.com/apache/apisix/issues/7261#issuecomment-1159905971 considered as solved, feel free to reopen if you need. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901210996 ## docs/zh/latest/plugins/prometheus.md: ## @@ -1,5 +1,11 @@ --- title: prometheus +keywords: + - APISIX + - API Gateway + - Plugin + - Prometheus +description

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901203423 ## docs/en/latest/terminology/plugin.md: ## @@ -95,6 +95,43 @@ the configuration above means customizing the error response from the jwt-auth p | Name | Ty

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901201503 ## apisix/plugin.lua: ## @@ -398,6 +406,51 @@ function _M.filter(ctx, conf, plugins, route_conf) trace_plugins_info_for_debug(ctx, plugins) +if custom_s

[GitHub] [apisix] spacewander opened a new pull request, #7278: backport: commits to release/2.13

2022-06-19 Thread GitBox
spacewander opened a new pull request, #7278: URL: https://github.com/apache/apisix/pull/7278 ### Description Fixes # (issue) ### Checklist - [x] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new f

[GitHub] [apisix] juzhiyuan commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
juzhiyuan commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901199518 ## docs/zh/latest/plugins/prometheus.md: ## @@ -23,50 +29,71 @@ title: prometheus ## 描述 -此插件是提供符合 prometheus 数据格式的监控指标数据。 +`prometheus` 插件以 [Prometheus](https://p

[GitHub] [apisix] juzhiyuan commented on pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
juzhiyuan commented on PR #7275: URL: https://github.com/apache/apisix/pull/7275#issuecomment-1159885011 Thanks for your translation!! -- 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

[GitHub] [apisix] soulbird commented on issue #7227: help request: is there rpm which can be installed on arm ?

2022-06-19 Thread GitBox
soulbird commented on issue #7227: URL: https://github.com/apache/apisix/issues/7227#issuecomment-1159882931 This is already in my plan. -- 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 specif

[GitHub] [apisix] juzhiyuan commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
juzhiyuan commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901197817 ## docs/zh/latest/plugins/prometheus.md: ## @@ -75,70 +102,52 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13 "upstream": {

[GitHub] [apisix] hf400159 commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
hf400159 commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901196044 ## docs/zh/latest/plugins/prometheus.md: ## @@ -75,70 +102,52 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13 "upstream": {

[GitHub] [apisix] hf400159 commented on a diff in pull request #7275: docs: udpate prometheus Chinese doc

2022-06-19 Thread GitBox
hf400159 commented on code in PR #7275: URL: https://github.com/apache/apisix/pull/7275#discussion_r901196044 ## docs/zh/latest/plugins/prometheus.md: ## @@ -75,70 +102,52 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13 "upstream": {

[GitHub] [apisix] spacewander commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
spacewander commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901194744 ## apisix/schema_def.lua: ## @@ -952,6 +952,11 @@ _M.plugin_injected_schema = { { type = "object" }, } }, +

[GitHub] [apisix] spacewander commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
spacewander commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901194054 ## apisix/plugin.lua: ## @@ -398,6 +406,49 @@ function _M.filter(ctx, conf, plugins, route_conf) trace_plugins_info_for_debug(ctx, plugins) + +if custom_

[GitHub] [apisix] spacewander commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
spacewander commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901193594 ## docs/en/latest/terminology/plugin.md: ## @@ -95,6 +95,43 @@ the configuration above means customizing the error response from the jwt-auth p | Name | Typ

[GitHub] [apisix] soulbird commented on issue #7261: bug: http method rewrite not works in proxy-rewrite plugin

2022-06-19 Thread GitBox
soulbird commented on issue #7261: URL: https://github.com/apache/apisix/issues/7261#issuecomment-1159873010 Fixed in https://github.com/apache/apisix/pull/6300. Please upgrade apisix and dashboard. -- This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [apisix] tokers commented on issue #7260: feat: As a user, I want to add header rewriting to traffict_split plugin, so that it simplifies the setting of multiple lanes

2022-06-19 Thread GitBox
tokers commented on issue #7260: URL: https://github.com/apache/apisix/issues/7260#issuecomment-1159871642 > Or put the headers field to the upstream object, so we can use it in a more general way, not just traffic-split. Sounds good. -- This is an automated message from the Apache

[GitHub] [apisix] tokers commented on issue #7261: bug: http method rewrite not works in proxy-rewrite plugin

2022-06-19 Thread GitBox
tokers commented on issue #7261: URL: https://github.com/apache/apisix/issues/7261#issuecomment-1159870589 @mojerro I just checked the changelog, this problem was fixed since `2.13.0`, see https://github.com/apache/apisix/blob/master/CHANGELOG.md#2130. -- This is an automated message from

[GitHub] [apisix] spacewander opened a new pull request, #7277: fix(traffic-split): the default timeout doesn't match the one in Nginx

2022-06-19 Thread GitBox
spacewander opened a new pull request, #7277: URL: https://github.com/apache/apisix/pull/7277 Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [x] I have explained the need for this PR and the problem it solves - [ ] I have

[apisix] branch master updated: feat(cli): display test option when help (#7268)

2022-06-19 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 afce02cea feat(cli): display test option wh

[GitHub] [apisix] spacewander merged pull request #7268: feat(cli): display test option when help

2022-06-19 Thread GitBox
spacewander merged PR #7268: URL: https://github.com/apache/apisix/pull/7268 -- 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-unsubscr...@a

[GitHub] [apisix] spacewander closed issue #7148: feat: As a user, I want to add `apisix test` to `apisix help`, so that people could know this command quickly

2022-06-19 Thread GitBox
spacewander closed issue #7148: feat: As a user, I want to add `apisix test` to `apisix help`, so that people could know this command quickly URL: https://github.com/apache/apisix/issues/7148 -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901139711 ## docs/en/latest/terminology/plugin.md: ## @@ -95,6 +95,43 @@ the configuration above means customizing the error response from the jwt-auth p | Name | Ty

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901138984 ## docs/en/latest/terminology/plugin.md: ## @@ -95,6 +95,43 @@ the configuration above means customizing the error response from the jwt-auth p | Name | Ty

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901138709 ## apisix/plugin.lua: ## @@ -398,6 +406,49 @@ function _M.filter(ctx, conf, plugins, route_conf) trace_plugins_info_for_debug(ctx, plugins) + +if custom

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901138156 ## apisix/plugin.lua: ## @@ -398,6 +406,49 @@ function _M.filter(ctx, conf, plugins, route_conf) trace_plugins_info_for_debug(ctx, plugins) + +if custom

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7273: feat: allows users to specify plugin execution priority

2022-06-19 Thread GitBox
tzssangglass commented on code in PR #7273: URL: https://github.com/apache/apisix/pull/7273#discussion_r901138123 ## docs/en/latest/terminology/plugin.md: ## @@ -95,6 +95,43 @@ the configuration above means customizing the error response from the jwt-auth p | Name | Ty

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1089: 如何在k8s集群中使用apisix搭建公网和内网网关

2022-06-19 Thread GitBox
tao12345666333 commented on issue #1089: URL: https://github.com/apache/apisix-ingress-controller/issues/1089#issuecomment-1159768816 In addition, you can modify the namespace that APISIX Ingress can handle through configuration -- This is an automated message from the Apache Git Service

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1089: 如何在k8s集群中使用apisix搭建公网和内网网关

2022-06-19 Thread GitBox
tao12345666333 commented on issue #1089: URL: https://github.com/apache/apisix-ingress-controller/issues/1089#issuecomment-1159768493 > 还有就是配置Ingress资源如何使用插件功能。 For ingress resources, you can add annotations to use plugins -- This is an automated message from the Apache Git Servic

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1089: 如何在k8s集群中使用apisix搭建公网和内网网关

2022-06-19 Thread GitBox
tao12345666333 commented on issue #1089: URL: https://github.com/apache/apisix-ingress-controller/issues/1089#issuecomment-1159768053 > 如何在ApisixRoute中申明ingress-class来隔离内网和公网网关; Unfortunately, this feature is not implemented yet. Ref #593 -- This is an automated message from the Ap

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1088: request help: DigitalOcean support

2022-06-19 Thread GitBox
tao12345666333 commented on issue #1088: URL: https://github.com/apache/apisix-ingress-controller/issues/1088#issuecomment-1159767494 Back to your question, you can run `kubectl -n APISIX get service/etcd-headless ` -- This is an automated message from the Apache Git Service. To re

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1088: request help: DigitalOcean support

2022-06-19 Thread GitBox
tao12345666333 commented on issue #1088: URL: https://github.com/apache/apisix-ingress-controller/issues/1088#issuecomment-1159766885 If it is your first experience or trying it out, I really recommend that you install directly with Helm, it is the easiest and continuously maintained way t

[GitHub] [apisix] bisakhmondal commented on a diff in pull request #7099: init elasticsearch-logger.lua

2022-06-19 Thread GitBox
bisakhmondal commented on code in PR #7099: URL: https://github.com/apache/apisix/pull/7099#discussion_r901125113 ## t/plugin/elasticsearch-logger.t: ## @@ -0,0 +1,182 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements.

[GitHub] [apisix] bisakhmondal commented on a diff in pull request #7099: init elasticsearch-logger.lua

2022-06-19 Thread GitBox
bisakhmondal commented on code in PR #7099: URL: https://github.com/apache/apisix/pull/7099#discussion_r901123107 ## apisix/plugins/elasticsearch-logger.lua: ## @@ -0,0 +1,181 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor license a

[GitHub] [apisix] bisakhmondal commented on a diff in pull request #7099: init elasticsearch-logger.lua

2022-06-19 Thread GitBox
bisakhmondal commented on code in PR #7099: URL: https://github.com/apache/apisix/pull/7099#discussion_r901123107 ## apisix/plugins/elasticsearch-logger.lua: ## @@ -0,0 +1,181 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor license a

[GitHub] [apisix-ingress-controller] BFergerson commented on issue #1088: request help: DigitalOcean support

2022-06-19 Thread GitBox
BFergerson commented on issue #1088: URL: https://github.com/apache/apisix-ingress-controller/issues/1088#issuecomment-1159756099 Thanks for the recommendation. I'm trying to follow [this guide](https://github.com/apache/apisix-ingress-controller/blob/master/docs/en/latest/practices/the-har

[GitHub] [apisix] mojerro commented on issue #7261: bug: http method rewrite not works in proxy-rewrite plugin

2022-06-19 Thread GitBox
mojerro commented on issue #7261: URL: https://github.com/apache/apisix/issues/7261#issuecomment-1159746735 Can you reproduce this problem or some extra information I need to provide -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [apisix] spacewander commented on issue #7260: feat: As a user, I want to add header rewriting to traffict_split plugin, so that it simplifies the setting of multiple lanes

2022-06-19 Thread GitBox
spacewander commented on issue #7260: URL: https://github.com/apache/apisix/issues/7260#issuecomment-1159708263 Or put the headers field to the upstream object, so we can use it in a more general way, not just traffic-split. -- This is an automated message from the Apache Git Service. To

[GitHub] [apisix] spacewander commented on a diff in pull request #7266: feat: export some importent params for kafka-client

2022-06-19 Thread GitBox
spacewander commented on code in PR #7266: URL: https://github.com/apache/apisix/pull/7266#discussion_r901095840 ## docs/en/latest/plugins/kafka-logger.md: ## @@ -47,6 +47,10 @@ For more info on Batch-Processor in Apache APISIX please refer. | include_resp_body| boolean | opti

[GitHub] [apisix] spacewander commented on issue #7260: feat: As a user, I want to add header rewriting to traffict_split plugin, so that it simplifies the setting of multiple lanes

2022-06-19 Thread GitBox
spacewander commented on issue #7260: URL: https://github.com/apache/apisix/issues/7260#issuecomment-1159705582 LGTM. Although I would like to add upstream-level plugins (proxy-rewrite per upstream). -- This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [apisix] spacewander commented on issue #7227: help request: is there rpm which can be installed on arm ?

2022-06-19 Thread GitBox
spacewander commented on issue #7227: URL: https://github.com/apache/apisix/issues/7227#issuecomment-1159705084 CC @soulbird -- 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.

[GitHub] [apisix-go-plugin-runner] spacewander merged pull request #89: feat: add response filter for plugin

2022-06-19 Thread GitBox
spacewander merged PR #89: URL: https://github.com/apache/apisix-go-plugin-runner/pull/89 -- 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-

[apisix-go-plugin-runner] branch master updated: feat: add response filter for plugin (#89)

2022-06-19 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-go-plugin-runner.git The following commit(s) were added to refs/heads/master by this push: new 2d6b282 feat: add respons

[GitHub] [apisix] spacewander commented on issue #7239: help request: 插件 request-id 与 opentelemetry 结合

2022-06-19 Thread GitBox
spacewander commented on issue #7239: URL: https://github.com/apache/apisix/issues/7239#issuecomment-1159704040 > > opentelemetry is moved to the top so we can report requests which are rejected. If opentelemetry needs to depend on request-id, the priority of request-id should be higher.

[GitHub] [apisix-ingress-controller] guwenjiang85 opened a new issue, #1089: 如何在k8s集群中使用apisix搭建公网和内网网关

2022-06-19 Thread GitBox
guwenjiang85 opened a new issue, #1089: URL: https://github.com/apache/apisix-ingress-controller/issues/1089 ### Issue description 我目前使用apisix都是通过ApisixRoute,没有使用ingress资源; 如何在ApisixRoute中申明ingress-class来隔离内网和公网网关; 还有就是配置Ingress资源如何使用插件功能。 ### Environment - your api

[GitHub] [apisix-dashboard] shuqingzai opened a new issue, #2479: 导入 OpenAPI3.0 文档,数据验证

2022-06-19 Thread GitBox
shuqingzai opened a new issue, #2479: URL: https://github.com/apache/apisix-dashboard/issues/2479 ### Issue description 文档: [Import OpenAPI Guide](https://apisix.apache.org/zh/docs/dashboard/IMPORT_OPENAPI_USER_GUIDE) 使用 [request-validation](https://apisix.apache.org/zh/docs/apisix/

[GitHub] [apisix] tzssangglass commented on issue #7136: help request: upgrade to 2.14.0 after the route matching error, adjust the priority also did not solve, back to 2.13.1 on the normal

2022-06-19 Thread GitBox
tzssangglass commented on issue #7136: URL: https://github.com/apache/apisix/issues/7136#issuecomment-1159659872 > @spacewander @tzssangglass Do you have any idea about this? I've gathered similar information from other channels, but there is no reproducible demo, and I've noticed one