[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794274332 ## File path: docs/en/latest/plugins/prometheus.md ## @@ -46,17 +46,16 @@ plugin_attr: Assume environment variable `INTRANET_IP` is `172.1.1.1`, now API

[GitHub] [apisix] Belyenochi edited a comment on issue #3479: authz-keycloak: support configurating token refresh/expire time

2022-01-27 Thread GitBox
Belyenochi edited a comment on issue #3479: URL: https://github.com/apache/apisix/issues/3479#issuecomment-1023969968 I would like to fix this if no one else. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [apisix] Belyenochi commented on issue #3479: authz-keycloak: support configurating token refresh/expire time

2022-01-27 Thread GitBox
Belyenochi commented on issue #3479: URL: https://github.com/apache/apisix/issues/3479#issuecomment-1023969968 I want to fix this if no one else claims 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

[GitHub] [apisix-helm-chart] gxthrj commented on issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
gxthrj commented on issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225#issuecomment-1023967190 > We could use hostnetwork, which would have better performance. We can use `hostnetwork` with `deployment`, too. And set `dnsPolicy` to `ClusterFirstWithHostNet`.

[GitHub] [apisix] bisakhmondal commented on issue #6220: bug: Setting head-name in "request-id" plugin has no effect

2022-01-27 Thread GitBox
bisakhmondal commented on issue #6220: URL: https://github.com/apache/apisix/issues/6220#issuecomment-1023964028 Hi @1657486787, could you please share your apisix route configuration? I was not able to reproduce the bug that you are stating. I have checked it with the header_name you pass

[GitHub] [apisix-helm-chart] gardun0 opened a new issue #226: Chart apisix-ingress-controller 0.9.0 version was not updated on apisix 0.8.0 dependencies

2022-01-27 Thread GitBox
gardun0 opened a new issue #226: URL: https://github.com/apache/apisix-helm-chart/issues/226 The latest version 0.8.0 of apisix's chart did not upgrade the new version of ingress-controller chart 0.9.0. -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [apisix] Chever-John edited a comment on issue #6225: request help: install failed via source on Mac(2.12.0)

2022-01-27 Thread GitBox
Chever-John edited a comment on issue #6225: URL: https://github.com/apache/apisix/issues/6225#issuecomment-1023960355 Could it be that you don't have operation permissions for the file( /usr/share/lua)? Try "ls -" :) -- This is an automated message from the Apache Git Service. To res

[GitHub] [apisix] Chever-John commented on issue #6225: request help: install failed via source on Mac(2.12.0)

2022-01-27 Thread GitBox
Chever-John commented on issue #6225: URL: https://github.com/apache/apisix/issues/6225#issuecomment-1023960355 Could it be that you don't have operation permissions for the file? Try "ls -" :) -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [apisix] juzhiyuan closed issue #6185: request help: Why proxy-rewrite.host must be configured when starting the proxy-rewrite plugin, otherwise it will be forwarded directly to 127.0.0.1 by

2022-01-27 Thread GitBox
juzhiyuan closed issue #6185: URL: https://github.com/apache/apisix/issues/6185 -- 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-unsu

[GitHub] [apisix] juzhiyuan commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
juzhiyuan commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794263012 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#descriptio

[GitHub] [apisix] juzhiyuan commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
juzhiyuan commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794262894 ## File path: docs/en/latest/plugins/prometheus.md ## @@ -46,17 +46,16 @@ plugin_attr: Assume environment variable `INTRANET_IP` is `172.1.1.1`, now A

[GitHub] [apisix] juzhiyuan commented on issue #6068: request help: building failed on Mac(2.11 ldap.h)

2022-01-27 Thread GitBox
juzhiyuan commented on issue #6068: URL: https://github.com/apache/apisix/issues/6068#issuecomment-1023958848 > I got this trouble too, `brew info openldap` show installed dir `/usr/local/opt/openldap`, setted `LDAP_DIR` then `make deps` successfully. > > So, suggest to post this pre

[GitHub] [apisix] kwanhur opened a new pull request #6227: docs: solution for lualdap when install failed with luarocks

2022-01-27 Thread GitBox
kwanhur opened a new pull request #6227: URL: https://github.com/apache/apisix/pull/6227 ### What this PR does / why we need it: enhance user guide to install apisix from source ### Pre-submission checklist: * [x] Did you explain what problem does this PR sol

[apisix] branch master updated (3f7aaff -> 7200d0c)

2022-01-27 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git. from 3f7aaff docs: optimizing api-breaker documentation (#6219) add 7200d0c test(limit-count): fix title & sim

[GitHub] [apisix] spacewander commented on issue #5225: request help: support `make uninstall`

2022-01-27 Thread GitBox
spacewander commented on issue #5225: URL: https://github.com/apache/apisix/issues/5225#issuecomment-1023947616 @kwanhur Would you like to solve this problem? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[GitHub] [apisix] spacewander closed pull request #5229: feat: support `make uninstall`

2022-01-27 Thread GitBox
spacewander closed pull request #5229: URL: https://github.com/apache/apisix/pull/5229 -- 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: notificatio

[GitHub] [apisix] rapiz1 opened a new pull request #5229: feat: support `make uninstall`

2022-01-27 Thread GitBox
rapiz1 opened a new pull request #5229: URL: https://github.com/apache/apisix/pull/5229 Support `make uninstall` by simplying removing installed files and directories. Currently the installation can be simply reversed in this way as well as not introducing too much duplicated code.

[GitHub] [apisix] spacewander closed issue #6226: request help: make uninstall failed on Mac

2022-01-27 Thread GitBox
spacewander closed issue #6226: URL: https://github.com/apache/apisix/issues/6226 -- 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-un

[GitHub] [apisix] spacewander commented on issue #6226: request help: make uninstall failed on Mac

2022-01-27 Thread GitBox
spacewander commented on issue #6226: URL: https://github.com/apache/apisix/issues/6226#issuecomment-1023946029 Duplicate of https://github.com/apache/apisix/issues/5225 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] spacewander merged pull request #6216: test(limit-count): fix title & simpify the cases

2022-01-27 Thread GitBox
spacewander merged pull request #6216: URL: https://github.com/apache/apisix/pull/6216 -- 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: notificatio

[GitHub] [apisix] kwanhur opened a new issue #6226: request help: make uninstall failed on Mac

2022-01-27 Thread GitBox
kwanhur opened a new issue #6226: URL: https://github.com/apache/apisix/issues/6226 ### Issue description After `make install` partial successful steps, no easy way to uninstall, include main dir `/usr/local/apisix` and all the lua libs through `make deps`. Hopefully support

[GitHub] [apisix] jagerzhang closed issue #4760: request help: limit-count could set different limit count/time_window for the different client

2022-01-27 Thread GitBox
jagerzhang closed issue #4760: URL: https://github.com/apache/apisix/issues/4760 -- 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-uns

[GitHub] [apisix] kwanhur opened a new issue #6225: request help: install failed via source on Mac(2.12.0)

2022-01-27 Thread GitBox
kwanhur opened a new issue #6225: URL: https://github.com/apache/apisix/issues/6225 ### Issue description after `make deps` successfully, but `make install` or `sudo make install` failed. it seems no authority to mkdir dir `/usr/share/lua` . Installed lua with brew, so thi

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794230745 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,179 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] bisakhmondal commented on a change in pull request #5940: feat: mocking plugin

2022-01-27 Thread GitBox
bisakhmondal commented on a change in pull request #5940: URL: https://github.com/apache/apisix/pull/5940#discussion_r794227851 ## File path: t/plugin/mocking.t ## @@ -0,0 +1,402 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license

[GitHub] [apisix-helm-chart] Gallardot edited a comment on issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
Gallardot edited a comment on issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225#issuecomment-1023915806 @gxthrj > It is a good idea, btw I would like to ask if the scenario of this proposal is to use Apache APISIX to handle east-west traffic inside Kubernetes?

[GitHub] [apisix-helm-chart] Gallardot commented on issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
Gallardot commented on issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225#issuecomment-1023915806 > It is a good idea, btw I would like to ask if the scenario of this proposal is to use Apache APISIX to handle east-west traffic inside Kubernetes? Yes.

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794220090 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,179 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794220090 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,179 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] kwanhur commented on issue #6068: request help: building failed on Mac(2.11 ldap.h)

2022-01-27 Thread GitBox
kwanhur commented on issue #6068: URL: https://github.com/apache/apisix/issues/6068#issuecomment-1023911655 I got this trouble too, `brew info openldap` show installed dir `/usr/local/opt/openldap`, setted `LDAP_DIR` then `make deps` successfully. So, suggest to post this prerequisit

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794220090 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,179 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix-ingress-controller] gavinhot commented on issue #840: request help: panic happen in my cluster

2022-01-27 Thread GitBox
gavinhot commented on issue #840: URL: https://github.com/apache/apisix-ingress-controller/issues/840#issuecomment-1023907990 > I'm still on the road to discovery and I'm using ApisixUpstream only. But I found that my app was not registered with Apisix。 I found through debug that the svcL

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794218333 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] 1657486787 commented on issue #6221: bug: Setting rejected_msg in "limit-req" plugin does not save

2022-01-27 Thread GitBox
1657486787 commented on issue #6221: URL: https://github.com/apache/apisix/issues/6221#issuecomment-1023891781 > Hello there, Please provide you environment info in the description. ok ,my environment info : - apisix version (cmd: `apisix version`):2.12 - OS (cmd: `uname -a`):Li

[GitHub] [apisix] tzssangglass commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794193481 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#descrip

[GitHub] [apisix] tzssangglass commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794192690 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#descrip

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794188724 ## File path: docs/zh/latest/plugins/node-status.md ## @@ -40,7 +40,7 @@ title: node-status ## 插件接口 -插件增加接口 `/apisix/status`,可通过 [interceptors](../plu

[GitHub] [apisix-helm-chart] gxthrj commented on issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
gxthrj commented on issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225#issuecomment-1023866977 And I would like to ask why not choose to use `deployment` + `HPA` to deploy Apache APISIX, is there any special advantage of using `daemonset`? -- This is an automated me

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794188149 ## File path: docs/zh/latest/plugins/wolf-rbac.md ## @@ -51,7 +51,7 @@ rbac 功能由 [wolf](https://github.com/iGeeky/wolf) 提供, 有关 `wolf` 的 * /apisix/plugin/w

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794188111 ## File path: docs/zh/latest/plugin-develop.md ## @@ -326,8 +326,7 @@ function _M.api() end ``` -注意注册的接口会暴露到外网。 -你可能需要使用 [interceptors](plugin-intercep

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794187936 ## File path: docs/zh/latest/plugins/prometheus.md ## @@ -46,17 +46,16 @@ plugin_attr: 假设环境变量 `INTRANET_IP` 是 `172.1.1.1`,现在 APISIX 会在 `172.1.1.1:9092`

[GitHub] [apisix-helm-chart] gxthrj commented on issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
gxthrj commented on issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225#issuecomment-1023865452 It is a good idea, btw I would like to ask if the scenario of this proposal is to use Apache APISIX to handle east-west traffic inside Kubernetes? -- This is an automated

[GitHub] [apisix-dashboard] Chaunceyan commented on pull request #2298: fix: retries field to support zero value

2022-01-27 Thread GitBox
Chaunceyan commented on pull request #2298: URL: https://github.com/apache/apisix-dashboard/pull/2298#issuecomment-1023863619 > > > > You can just create a upstream with zero value, and check it return values. Sure, added in the new commit -- This is an automated message

[GitHub] [apisix] leslie-tsang commented on issue #6221: bug: Setting rejected_msg in "limit-req" plugin does not save

2022-01-27 Thread GitBox
leslie-tsang commented on issue #6221: URL: https://github.com/apache/apisix/issues/6221#issuecomment-1023862897 Hello there, Please provide you environment info in the description. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[GitHub] [apisix] tzssangglass commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794183325 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,179 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] leslie-tsang opened a new pull request #6222: fix(deps): upgrade jsonschema to `0.9.7` to ensure character % will be handled correctly

2022-01-27 Thread GitBox
leslie-tsang opened a new pull request #6222: URL: https://github.com/apache/apisix/pull/6222 ### What this PR does / why we need it: See https://github.com/api7/jsonschema/pull/73 ### Pre-submission checklist: * [x] Did you explain what problem does this

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794183501 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#description)

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794184239 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#description)

[GitHub] [apisix] 1657486787 opened a new issue #6221: bug: Setting rejected_msg in "limit-req" plugin does not save

2022-01-27 Thread GitBox
1657486787 opened a new issue #6221: URL: https://github.com/apache/apisix/issues/6221 ### Issue description Setting rejected_msg in "limit-req" plugin does not save ### Environment - apisix version (cmd: `apisix version`): - OS (cmd: `uname -a`): - OpenResty / Ngi

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794183501 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#description)

[GitHub] [apisix] 1657486787 opened a new issue #6220: bug: Setting head-name in "request-id" plugin has no effect

2022-01-27 Thread GitBox
1657486787 opened a new issue #6220: URL: https://github.com/apache/apisix/issues/6220 ### Issue description Setting head-name in "request-id" plugin has no effect ### Environment - apisix version (cmd: `apisix version`): - OS (cmd: `uname -a`): - OpenResty / Nginx

[GitHub] [apisix] shuaijinchao commented on a change in pull request #6214: fix(admin-token): fixed token is used by default

2022-01-27 Thread GitBox
shuaijinchao commented on a change in pull request #6214: URL: https://github.com/apache/apisix/pull/6214#discussion_r794180151 ## File path: ci/linux_apisix_current_luarocks_runner.sh ## @@ -66,6 +67,10 @@ script() { ulimit -n -S ulimit -n -H +# set default adm

[GitHub] [apisix] leslie-tsang commented on issue #6185: request help: Why proxy-rewrite.host must be configured when starting the proxy-rewrite plugin, otherwise it will be forwarded directly to 127.

2022-01-27 Thread GitBox
leslie-tsang commented on issue #6185: URL: https://github.com/apache/apisix/issues/6185#issuecomment-1023854969 > I don't understand why proxy-rewrite.host must be configured, otherwise it won't work properly. Hello there, `proxy-rewrite.host` is an optional configuration, which mea

[GitHub] [apisix] tzssangglass commented on a change in pull request #6127: docs: update the jwt-auth zh version

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6127: URL: https://github.com/apache/apisix/pull/6127#discussion_r794178594 ## File path: docs/zh/latest/plugins/jwt-auth.md ## @@ -48,6 +53,9 @@ title: jwt-auth | algorithm | string | 可选 | "HS256" | ["HS256", "HS512",

[GitHub] [apisix] tzssangglass commented on a change in pull request #6210: docs: update document directory

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6210: URL: https://github.com/apache/apisix/pull/6210#discussion_r794173824 ## File path: docs/zh/latest/plugin-develop.md ## @@ -25,7 +25,7 @@ title: 插件开发 - [目录](#目录) - [检查外部依赖](#检查外部依赖) -- [插件命名与配置](#插件命名与配置) +- [插件命名,优先

[GitHub] [apisix-helm-chart] Gallardot opened a new issue #225: Proposal: APISIX can run as DaemonSet

2022-01-27 Thread GitBox
Gallardot opened a new issue #225: URL: https://github.com/apache/apisix-helm-chart/issues/225 In the microservices architecture, multiple independent business services may use independent registries (Eureka \ Nacos \Consul). We hope that APISIX can be deployed to the cluster as daemonset.

[apisix] branch master updated (5f6d160 -> 3f7aaff)

2022-01-27 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git. from 5f6d160 fix(request-id): if the header_name is already present in the… (#6160) add 3f7aaff docs: optimiz

[GitHub] [apisix] spacewander merged pull request #6219: docs: optimizing api-breaker documentation

2022-01-27 Thread GitBox
spacewander merged pull request #6219: URL: https://github.com/apache/apisix/pull/6219 -- 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: notificatio

[GitHub] [apisix] spacewander closed issue #6175: request help: api-breaker plugin donot work

2022-01-27 Thread GitBox
spacewander closed issue #6175: URL: https://github.com/apache/apisix/issues/6175 -- 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-un

[GitHub] [apisix-go-plugin-runner] spacewander commented on issue #62: Cannot setting response header and proxy to backend at the same time in go plugin

2022-01-27 Thread GitBox
spacewander commented on issue #62: URL: https://github.com/apache/apisix-go-plugin-runner/issues/62#issuecomment-1023841418 Not support yet. -- 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

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794168886 ## File path: conf/config-default.yaml ## @@ -381,6 +381,7 @@ plugins: # plugin list (sorted by priority) - rocketmq-logg

[GitHub] [apisix] tzssangglass commented on issue #5755: [DISCUSS]: Improving the performance of the prometheus plugin

2022-01-27 Thread GitBox
tzssangglass commented on issue #5755: URL: https://github.com/apache/apisix/issues/5755#issuecomment-1023837619 new version has be released: https://github.com/knyar/nginx-lua-prometheus/releases/tag/0.20220127 In the following days I will submit a PR to fix APISIX. -- This is an

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on issue #75: feature: java plugin log path error

2022-01-27 Thread GitBox
tzssangglass commented on issue #75: URL: https://github.com/apache/apisix-java-plugin-runner/issues/75#issuecomment-1023836831 > how do I send logs to the rocketmq-logger plug-in ? ref: https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/rocketmq-logger.md BT

[GitHub] [apisix] tzssangglass commented on issue #6217: request help: does apisix authentication plugin support multiple authentication methods at the same time

2022-01-27 Thread GitBox
tzssangglass commented on issue #6217: URL: https://github.com/apache/apisix/issues/6217#issuecomment-1023836067 In fact, I have doubts about this OR approach, which I now feel adds complexity. We now have the public-API plugin and maybe we can achieve similar functionality with the hel

[GitHub] [apisix] spacewander commented on pull request #5940: feat: mocking plugin

2022-01-27 Thread GitBox
spacewander commented on pull request #5940: URL: https://github.com/apache/apisix/pull/5940#issuecomment-1023835155 Let's merge master to make CI pass? -- 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-java-plugin-runner] laizuan commented on issue #75: feature: java plugin log path error

2022-01-27 Thread GitBox
laizuan commented on issue #75: URL: https://github.com/apache/apisix-java-plugin-runner/issues/75#issuecomment-1023834217 how do I send logs to the rocketmq-logger plug-in ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794164504 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794162338 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] tzssangglass commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
tzssangglass commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794161639 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#descrip

[GitHub] [apisix-ingress-controller] gavinhot edited a comment on issue #840: request help: panic happen in my cluster

2022-01-27 Thread GitBox
gavinhot edited a comment on issue #840: URL: https://github.com/apache/apisix-ingress-controller/issues/840#issuecomment-1023827467 I'm still on the road to discovery and I'm using ApisixUpstream only. But I found that my app was not registered with Apisix。 I found through debug that t

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794160032 ## File path: docs/zh/latest/plugin-develop.md ## @@ -326,8 +326,7 @@ function _M.api() end ``` -注意注册的接口会暴露到外网。 -你可能需要使用 [interceptors](plugin-intercep

[GitHub] [apisix] spacewander commented on a change in pull request #6219: docs: optimizing api-breaker documentation

2022-01-27 Thread GitBox
spacewander commented on a change in pull request #6219: URL: https://github.com/apache/apisix/pull/6219#discussion_r794159919 ## File path: docs/en/latest/plugins/api-breaker.md ## @@ -90,7 +95,7 @@ curl "http://127.0.0.1:9080/apisix/admin/routes/1"; -H 'X-API-KEY: edd1c9f034

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794159838 ## File path: docs/en/latest/plugins/public-api.md ## @@ -0,0 +1,132 @@ +--- +title: public-api +--- + + + +## Summary + +- [**Description**](#description)

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794159753 ## File path: docs/en/latest/plugins/clickhouse-logger.md ## @@ -0,0 +1,157 @@ +--- +title: clickhouse-logger +--- + + + +## Summary + +- [**Name**](#

[GitHub] [apisix-ingress-controller] gavinhot commented on issue #840: request help: panic happen in my cluster

2022-01-27 Thread GitBox
gavinhot commented on issue #840: URL: https://github.com/apache/apisix-ingress-controller/issues/840#issuecomment-1023827467 I'm still on the road to discovery and I'm using ApisixUpstream only. But I found that my app was not registered with Apisix -- This is an automated message from

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794158591 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794158591 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] bzp2010 commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
bzp2010 commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794159168 ## File path: docs/en/latest/plugins/prometheus.md ## @@ -46,17 +46,16 @@ plugin_attr: Assume environment variable `INTRANET_IP` is `172.1.1.1`, now API

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794158591 ## File path: apisix/plugins/clickhouse-logger.lua ## @@ -0,0 +1,231 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] zhendongcmss commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
zhendongcmss commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794158245 ## File path: conf/config-default.yaml ## @@ -381,6 +381,7 @@ plugins: # plugin list (sorted by priority) - rocketmq-logg

[GitHub] [apisix-ingress-controller] gavinhot commented on issue #840: request help: panic happen in my cluster

2022-01-27 Thread GitBox
gavinhot commented on issue #840: URL: https://github.com/apache/apisix-ingress-controller/issues/840#issuecomment-1023824841 I unmount webhook-certs and it's fine. I do this because my Dev environment is fine on my MAC laptop。 -- This is an automated message from the Apache Git Ser

[GitHub] [apisix] spacewander commented on a change in pull request #6215: feat: clickhouse logger

2022-01-27 Thread GitBox
spacewander commented on a change in pull request #6215: URL: https://github.com/apache/apisix/pull/6215#discussion_r794146392 ## File path: docs/en/latest/plugins/clickhouse-logger.md ## @@ -0,0 +1,157 @@ +--- +title: clickhouse-logger +--- + + + +## Summary + +- [**Name**](#n

[apisix] branch master updated (427e926 -> 46834b2)

2022-01-27 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git. from 427e926 change: don't expose public API by default & remove plugin interceptor (#6196) add 46834b2 feat(

[apisix] branch master updated (46834b2 -> 5f6d160)

2022-01-27 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git. from 46834b2 feat(loggly): support http/s bulk sending for batch logs (#6212) add 5f6d160 fix(request-id): if

[GitHub] [apisix] moonming commented on a change in pull request #6214: fix(admin-token): fixed token is used by default

2022-01-27 Thread GitBox
moonming commented on a change in pull request #6214: URL: https://github.com/apache/apisix/pull/6214#discussion_r794148153 ## File path: ci/linux_apisix_current_luarocks_runner.sh ## @@ -66,6 +67,10 @@ script() { ulimit -n -S ulimit -n -H +# set default admin t

[GitHub] [apisix] zlhgo commented on issue #6184: bug: balancer.lua:187: attempt to index local 'up_conf' (a nil value)

2022-01-27 Thread GitBox
zlhgo commented on issue #6184: URL: https://github.com/apache/apisix/issues/6184#issuecomment-1023811916 > > 我们截取了串站时的前后5万行日志,有57M,如果需要这份日志的话,可以提供一个上传地址我传上来。再次感谢。 > > Can you upload it to some cloud drive, email me the link and password? 发你邮件了。 -- This is an automated messa

[GitHub] [apisix] spacewander merged pull request #6160: fix(request-id): if the header_name is already present in the…

2022-01-27 Thread GitBox
spacewander merged pull request #6160: URL: https://github.com/apache/apisix/pull/6160 -- 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: notificatio

[GitHub] [apisix] spacewander closed issue #6066: bug: request-id plugin overwrite X-Request-Id already present in the request

2022-01-27 Thread GitBox
spacewander closed issue #6066: URL: https://github.com/apache/apisix/issues/6066 -- 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-un

[GitHub] [apisix] spacewander merged pull request #6212: feat(loggly): support http/s bulk sending for batch logs

2022-01-27 Thread GitBox
spacewander merged pull request #6212: URL: https://github.com/apache/apisix/pull/6212 -- 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: notificatio

[GitHub] [apisix] moonming commented on a change in pull request #6214: fix(admin-token): fixed token is used by default

2022-01-27 Thread GitBox
moonming commented on a change in pull request #6214: URL: https://github.com/apache/apisix/pull/6214#discussion_r794145536 ## File path: apisix/init.lua ## @@ -86,6 +88,12 @@ function _M.http_init(args) core.log.error("failed to load the configuration: ", err)

[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #864: chore: hi 2022

2022-01-27 Thread GitBox
netlify[bot] edited a comment on pull request #864: URL: https://github.com/apache/apisix-website/pull/864#issuecomment-1023802864 ✔️ Deploy Preview for *apache-apisix* ready! 🔨 Explore the source changes: 37c420487b9eda2cee9079622d0f685b10d5f788 🔍 Inspect the deploy log:

[GitHub] [apisix-website] netlify[bot] commented on pull request #864: chore: hi 2022

2022-01-27 Thread GitBox
netlify[bot] commented on pull request #864: URL: https://github.com/apache/apisix-website/pull/864#issuecomment-1023802864 👷 Deploy Preview for *apache-apisix* processing. 🔨 Explore the source changes: 37c420487b9eda2cee9079622d0f685b10d5f788 🔍 Inspect the deploy log: [ht

[GitHub] [apisix-website] hf400159 opened a new pull request #864: chore: hi 2022

2022-01-27 Thread GitBox
hf400159 opened a new pull request #864: URL: https://github.com/apache/apisix-website/pull/864 Fixes: #[Add issue number here] Changes: Screenshots of the change: -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [apisix-dashboard] zaunist commented on pull request #2298: fix: retries field to support zero value

2022-01-27 Thread GitBox
zaunist commented on pull request #2298: URL: https://github.com/apache/apisix-dashboard/pull/2298#issuecomment-1023802565 > You can just create a upstream with zero value, and check it return values. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] zaunist commented on pull request #2298: fix: retries field to support zero value

2022-01-27 Thread GitBox
zaunist commented on pull request #2298: URL: https://github.com/apache/apisix-dashboard/pull/2298#issuecomment-1023801988 > > Better add some E2E test case to cover this change. > > Hi @zaunist, no problem, I suppose some cases on the route handler will do? I think it's bette

[GitHub] [apisix-dashboard] jwrookie commented on issue #2286: /debug/pprof/ this api un auth

2022-01-27 Thread GitBox
jwrookie commented on issue #2286: URL: https://github.com/apache/apisix-dashboard/issues/2286#issuecomment-1023801274 > but at turn it on, the problem will reproduce -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [apisix-dashboard] Chaunceyan commented on pull request #2298: fix: retries field to support zero value

2022-01-27 Thread GitBox
Chaunceyan commented on pull request #2298: URL: https://github.com/apache/apisix-dashboard/pull/2298#issuecomment-1023799179 > Better add some E2E test case to cover this change. Hi @zaunist, no problem, I suppose some cases on the route handler will do? -- This is an automated m

[GitHub] [apisix-website] yzeng25 commented on pull request #863: Updated Blog list page's style

2022-01-27 Thread GitBox
yzeng25 commented on pull request #863: URL: https://github.com/apache/apisix-website/pull/863#issuecomment-1023798367 Hi @BeeBombshell , Could you check the online preview link provided by netlify comment above? When I click in to see the preview, the website's format is completely

[GitHub] [apisix] tzssangglass commented on issue #6184: bug: balancer.lua:187: attempt to index local 'up_conf' (a nil value)

2022-01-27 Thread GitBox
tzssangglass commented on issue #6184: URL: https://github.com/apache/apisix/issues/6184#issuecomment-1023797703 > 我们截取了串站时的前后5万行日志,有57M,如果需要这份日志的话,可以提供一个上传地址我传上来。再次感谢。 Can you upload it to some cloud drive, email me the link and password? -- This is an automated message from the A

[GitHub] [apisix] shuaijinchao commented on a change in pull request #6218: docs: add public-api related content

2022-01-27 Thread GitBox
shuaijinchao commented on a change in pull request #6218: URL: https://github.com/apache/apisix/pull/6218#discussion_r794135901 ## File path: docs/zh/latest/plugin-develop.md ## @@ -326,8 +326,7 @@ function _M.api() end ``` -注意注册的接口会暴露到外网。 -你可能需要使用 [interceptors](plugin-int

  1   2   >