[GitHub] [apisix] spacewander closed issue #7144: help request: How to add filter conditions for plugins' usage

2022-07-19 Thread GitBox
spacewander closed issue #7144: help request: How to add filter conditions for plugins' usage URL: https://github.com/apache/apisix/issues/7144 -- 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 sp

[apisix] branch master updated: feat: supports dynamic control of whether plugins are running (#7453)

2022-07-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 f5f402f44 feat: supports dynamic control of

[GitHub] [apisix] spacewander merged pull request #7453: feat: supports dynamic control of whether plugins are running

2022-07-19 Thread GitBox
spacewander merged PR #7453: URL: https://github.com/apache/apisix/pull/7453 -- 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] tao12345666333 commented on issue #1169: request help: apisix-ingress-controller has some warning logs when it starts

2022-07-19 Thread GitBox
tao12345666333 commented on issue #1169: URL: https://github.com/apache/apisix-ingress-controller/issues/1169#issuecomment-1189877122 you can enable `debug` level log to see more details -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix] tzssangglass commented on pull request #7498: chore: tweak conf_server unstable test cases

2022-07-19 Thread GitBox
tzssangglass commented on PR #7498: URL: https://github.com/apache/apisix/pull/7498#issuecomment-1189875692 > Better to write down the `why` in the commit message update in PR Description -- This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [apisix-ingress-controller] AlinsRan commented on a diff in pull request #1152: fix: trigger ApisixRoute event when service is created

2022-07-19 Thread GitBox
AlinsRan commented on code in PR #1152: URL: https://github.com/apache/apisix-ingress-controller/pull/1152#discussion_r925204695 ## pkg/ingress/apisix_route.go: ## @@ -82,9 +95,114 @@ func (c *apisixRouteController) runWorker(ctx context.Context) { if quit {

[GitHub] [apisix-ingress-controller] AlinsRan commented on a diff in pull request #1152: fix: trigger ApisixRoute event when service is created

2022-07-19 Thread GitBox
AlinsRan commented on code in PR #1152: URL: https://github.com/apache/apisix-ingress-controller/pull/1152#discussion_r925205372 ## pkg/ingress/apisix_route.go: ## @@ -41,13 +42,19 @@ type apisixRouteController struct { controller *Controller workqueue workqueu

[GitHub] [apisix] spacewander opened a new pull request, #7502: test: test in TEST_NGINX_USE_HUP mode can't run after the standalone mode

2022-07-19 Thread GitBox
spacewander opened a new pull request, #7502: URL: https://github.com/apache/apisix/pull/7502 Otherwise, there will be `failed to fetch /apisix/t/servroot/conf/apisix.yaml attributes: cannot obtain information from file '/apisix/t/servroot/conf/apisix.yaml': No such file or directory`

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1152: fix: trigger ApisixRoute event when service is created

2022-07-19 Thread GitBox
tao12345666333 commented on code in PR #1152: URL: https://github.com/apache/apisix-ingress-controller/pull/1152#discussion_r925200405 ## pkg/kube/apisix_route.go: ## @@ -78,6 +84,18 @@ type apisixRoute struct { v2 *configv2.ApisixRoute } +func (l *apisixRo

[GitHub] [apisix-ingress-controller] juzhiyuan commented on issue #1167: request help: use ingress annotation to support traffic split feature

2022-07-19 Thread GitBox
juzhiyuan commented on issue #1167: URL: https://github.com/apache/apisix-ingress-controller/issues/1167#issuecomment-1189854111 Hi @tao12345666333, please check https://github.com/apache/apisix-website/issues/201 and https://apisix.apache.org/docs/general/documentation-style-guide/#format

[GitHub] [apisix-docker] lilien1010 commented on issue #334: error code 404 but sometimes 200

2022-07-19 Thread GitBox
lilien1010 commented on issue #334: URL: https://github.com/apache/apisix-docker/issues/334#issuecomment-1189844715 @rizkyarmanto you need elaborate more details to allow people to help you solve the issue -- This is an automated message from the Apache Git Service. To respond to the mes

[apisix-website] branch asf-site updated (104a440ce43 -> 5094b32e9e2)

2022-07-19 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/apisix-website.git discard 104a440ce43 Add files via upload discard 5bc05bf5e02 deploy: 3b1c02eb3ec71f4c9637dc27c51ffbf223f3a6e6

[GitHub] [apisix-dashboard] codecov-commenter commented on pull request #2531: test: move upstream unit cases to e2e

2022-07-19 Thread GitBox
codecov-commenter commented on PR #2531: URL: https://github.com/apache/apisix-dashboard/pull/2531#issuecomment-1189818941 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2531?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_

[GitHub] [apisix-dashboard] bzp2010 opened a new pull request, #2531: test: move upstream unit cases to e2e

2022-07-19 Thread GitBox
bzp2010 opened a new pull request, #2531: URL: https://github.com/apache/apisix-dashboard/pull/2531 Please answer these questions before submitting a pull request, **or your PR will get closed**. **Why submit this pull request?** - [ ] Bugfix - [ ] New feature provided -

[apisix-website] branch asf-site updated: Add files via upload

2022-07-19 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 104a440ce43 Add files via upload

[GitHub] [apisix-ingress-controller] lingsamuel commented on a diff in pull request #1152: fix: trigger ApisixRoute event when service is created

2022-07-19 Thread GitBox
lingsamuel commented on code in PR #1152: URL: https://github.com/apache/apisix-ingress-controller/pull/1152#discussion_r925155282 ## pkg/ingress/apisix_route.go: ## @@ -41,13 +43,20 @@ type apisixRouteController struct { controller *Controller workqueue workqu

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1152: fix: trigger ApisixRoute event when service is created

2022-07-19 Thread GitBox
tao12345666333 commented on code in PR #1152: URL: https://github.com/apache/apisix-ingress-controller/pull/1152#discussion_r925127074 ## pkg/ingress/apisix_route.go: ## @@ -41,13 +43,20 @@ type apisixRouteController struct { controller *Controller workqueue wo

[GitHub] [apisix] tokers opened a new issue, #7501: docs: `default_weight` field is missing from `config-default.yaml` and document

2022-07-19 Thread GitBox
tokers opened a new issue, #7501: URL: https://github.com/apache/apisix/issues/7501 ### Current State The Kubernetes Service Discovery module supports a field `default_weight` as the weight for Apache APISIX upstream nodes, but it misses from both the `config-default.yaml` and the do

[GitHub] [apisix] tzssangglass opened a new pull request, #7500: feat(jwt-auth): support setting clock skew for verifying

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

[GitHub] [apisix-ingress-controller] GhangZh commented on issue #1169: request help: apisix-ingress-controller has some warning logs when it starts

2022-07-19 Thread GitBox
GhangZh commented on issue #1169: URL: https://github.com/apache/apisix-ingress-controller/issues/1169#issuecomment-1189773632 > These logs are some expected output, but it also means that you are currently facing some situation. > > 1. Same as 👆 you have some routes not created by A

[GitHub] [apisix] 1renwuruiming closed issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
1renwuruiming closed issue #7495: help request: how to ban cache? URL: https://github.com/apache/apisix/issues/7495 -- 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 unsubscri

[GitHub] [apisix-ingress-controller] wolgod opened a new issue, #1171: request help: How many settings resync_interval are appropriate?

2022-07-19 Thread GitBox
wolgod opened a new issue, #1171: URL: https://github.com/apache/apisix-ingress-controller/issues/1171 ### Issue description resync_interval default is 6h,There are about 3000 apisixroutes in my environment. If the default value of 6h is used, if some events are lost, the routing may

[GitHub] [apisix] spacewander opened a new pull request, #7499: docs(control-api): fix introduced versions

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

[GitHub] [apisix] tzssangglass opened a new pull request, #7498: chore: tweak conf_server unstable test cases

2022-07-19 Thread GitBox
tzssangglass opened a new pull request, #7498: URL: https://github.com/apache/apisix/pull/7498 ### Description Fixes #7476 ### Checklist - [x] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new feat

[GitHub] [apisix] zhixiongdu027 commented on issue #7489: bug: kubernetes discovery failed without recover

2022-07-19 Thread GitBox
zhixiongdu027 commented on issue #7489: URL: https://github.com/apache/apisix/issues/7489#issuecomment-1189721045 > A given Kubernetes server will only preserve a historical record of changes for a limited time. Clusters using etcd 3 preserve changes in the last 5 minutes by default. When t

[GitHub] [apisix] tzssangglass commented on pull request #7497: fix(wolf-rbac): Return 403 error code when the user does not have per…

2022-07-19 Thread GitBox
tzssangglass commented on PR #7497: URL: https://github.com/apache/apisix/pull/7497#issuecomment-1189719577 401 is no authentication information or wrong authentication information, then the client can modify the authentication information to retry; 403 is the client with the correct

[GitHub] [apisix] tzssangglass commented on issue #7496: help request: how to pick up username/password from consumer when accessing API?

2022-07-19 Thread GitBox
tzssangglass commented on issue #7496: URL: https://github.com/apache/apisix/issues/7496#issuecomment-1189716533 If you are talking about passing consumer name upstream, see: https://github.com/apache/apisix/issues/4078#issuecomment-822231019 and `basic-auth` plugin has `hide_credenti

[GitHub] [apisix] crazyMonkey1995 commented on issue #7489: bug: kubernetes discovery failed without recover

2022-07-19 Thread GitBox
crazyMonkey1995 commented on issue #7489: URL: https://github.com/apache/apisix/issues/7489#issuecomment-1189714961 > @crazyMonkey1995 @tokers Sorry, I didn't really expect a "Gone" error on "List" I will try to verify and repair However, this problem is difficult to reproduce. You ca

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1167: request help: use ingress annotation to support traffic split feature

2022-07-19 Thread GitBox
tao12345666333 commented on issue #1167: URL: https://github.com/apache/apisix-ingress-controller/issues/1167#issuecomment-1189713080 @juzhiyuan Can you help point out how to add versioned documentation? I didn't notice it -- This is an automated message from the Apache Git Service. To r

[GitHub] [apisix] tzssangglass commented on issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
tzssangglass commented on issue #7495: URL: https://github.com/apache/apisix/issues/7495#issuecomment-1189712244 > nginx使用了no-cache配置来禁用缓存,那么在apisix上如何对路由层面实现这样的操作呢 we can use `proxy-cache` plugin: https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/proxy-cache.md --

[GitHub] [apisix] zhixiongdu027 commented on issue #7489: bug: kubernetes discovery failed without recover

2022-07-19 Thread GitBox
zhixiongdu027 commented on issue #7489: URL: https://github.com/apache/apisix/issues/7489#issuecomment-1189711684 @crazyMonkey1995 @tokers Sorry, I didn't really expect a "Gone" error on "List" I will try to verify and repair -- This is an automated message from the Apache Git Servi

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on issue #167: RpcCallHandler is dispatched on the IO thread

2022-07-19 Thread GitBox
tzssangglass commented on issue #167: URL: https://github.com/apache/apisix-java-plugin-runner/issues/167#issuecomment-1189711072 We can discuss potential ways to optimize the performance of the java plugin runner. -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [apisix] 1renwuruiming commented on issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
1renwuruiming commented on issue #7495: URL: https://github.com/apache/apisix/issues/7495#issuecomment-1189710531 nginx使用了no-cache配置来禁用缓存,那么在apisix上如何对路由层面实现这样的操作呢 -- 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] zhixiongdu027 commented on issue #7473: bug: use kubernetes dicovery start failed

2022-07-19 Thread GitBox
zhixiongdu027 commented on issue #7473: URL: https://github.com/apache/apisix/issues/7473#issuecomment-1189709671 @myoss looks like you are trying to use Kubernetes Discovery in the Stream Module. Right ? The reason for your problem is that "ngx.process" does not support Stre

[GitHub] [apisix] qihaiyan closed issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan closed issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml URL: https://github.com/apache/apisix/issues/7491 -- 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 t

[GitHub] [apisix] tzssangglass commented on issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
tzssangglass commented on issue #7495: URL: https://github.com/apache/apisix/issues/7495#issuecomment-1189709340 > nginx层面已禁用缓存 You can comb through the disabled cache operations done on Nginx and implement the corresponding configuration on APISIX. -- This is an automated message

[GitHub] [apisix-ingress-controller] cai200631 commented on issue #1167: request help: use ingress annotation to support traffic split feature

2022-07-19 Thread GitBox
cai200631 commented on issue #1167: URL: https://github.com/apache/apisix-ingress-controller/issues/1167#issuecomment-1189707253 > You need to use latest code from master branch or wait for v1.5 released I finally figure out the trick, you guys are too naughty. Why the documentation

[GitHub] [apisix] qihaiyan commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1189707024 > > > try with: https://github.com/apache/apisix/blob/master/docs/en/latest/customize-nginx-configuration.md > > > > > > Because ngx_tpl.lua doesn't have a `client_body_buf

[GitHub] [apisix] spacewander commented on issue #7211: bug: prometheus metrics cause high cpu usage

2022-07-19 Thread GitBox
spacewander commented on issue #7211: URL: https://github.com/apache/apisix/issues/7211#issuecomment-1189706469 Let's discuss it at https://github.com/apache/apisix/issues/7211. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [apisix] spacewander closed issue #7211: bug: prometheus metrics cause high cpu usage

2022-07-19 Thread GitBox
spacewander closed issue #7211: bug: prometheus metrics cause high cpu usage URL: https://github.com/apache/apisix/issues/7211 -- 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. T

[GitHub] [apisix] spacewander commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
spacewander commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1189700564 > > try with: https://github.com/apache/apisix/blob/master/docs/en/latest/customize-nginx-configuration.md > > Because ngx_tpl.lua doesn't have a `client_body_buffer_size`,

[GitHub] [apisix] 1renwuruiming commented on issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
1renwuruiming commented on issue #7495: URL: https://github.com/apache/apisix/issues/7495#issuecomment-1189699854 我部署了一个vue项目,通过apisix代理到nginx来访问,nginx层面已禁用缓存,通过apisix访问时发现页面的数据未进行更新,实际接口已调用过,并且数据正常返回了 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix] spacewander commented on issue #7482: Remove deprecated configuration in the V3

2022-07-19 Thread GitBox
spacewander commented on issue #7482: URL: https://github.com/apache/apisix/issues/7482#issuecomment-1189698373 > Maybe we can remove old configuration items like `port_admin` from the configuration file as well? 🤔 LGTM -- This is an automated message from the Apache Git Service. T

[GitHub] [apisix] spacewander commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
spacewander commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1189697207 raw openresty doesn't support client-control. I would suggest you add client_body_buffer_size via the custom snippet. See https://github.com/apache/apisix/blob/master/docs/e

[GitHub] [apisix-ingress-controller] AlinsRan commented on pull request #1056: feat: add mqtt-proxy plugin in ApisixRoute (#966)

2022-07-19 Thread GitBox
AlinsRan commented on PR #1056: URL: https://github.com/apache/apisix-ingress-controller/pull/1056#issuecomment-1189694529 @stillfox-lee Please replace `suite-plugins/mqtt-proxy.go` with `suite-plugins/suite-plugins-other/mqtt-proxy.go`. Because the secondary directory will not be started

[GitHub] [apisix] spacewander commented on issue #6996: feat: As a user, I want to see `grpc-status-details-bin` in response body in case an error, so that I avoid parsing header to read error respon

2022-07-19 Thread GitBox
spacewander commented on issue #6996: URL: https://github.com/apache/apisix/issues/6996#issuecomment-1189693634 Yes -- 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 unsub

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #2530: test: move stream route unit cases to e2e

2022-07-19 Thread GitBox
juzhiyuan merged PR #2530: URL: https://github.com/apache/apisix-dashboard/pull/2530 -- 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-unsub

[apisix-dashboard] branch next updated (aea02bbb -> d01b53d1)

2022-07-19 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch next in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git from aea02bbb test: move ssl unit cases to e2e (#2528) add d01b53d1 test: move stream route unit cases to e

[GitHub] [apisix-java-plugin-runner] oridag commented on issue #167: RpcCallHandler is dispatched on the IO thread

2022-07-19 Thread GitBox
oridag commented on issue #167: URL: https://github.com/apache/apisix-java-plugin-runner/issues/167#issuecomment-1189542126 Hmm, I may be wrong here. It's possible this will only block that specific connection's thread, in which case it's ok -- This is an automated message from the Apach

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1169: request help: apisix-ingress-controller has some warning logs when it starts

2022-07-19 Thread GitBox
tao12345666333 commented on issue #1169: URL: https://github.com/apache/apisix-ingress-controller/issues/1169#issuecomment-1189311409 These logs are some expected output, but it also means that you are currently facing some situation. 1. Same as 👆 you have some routes not created by

[GitHub] [apisix] vyvyan2010 commented on issue #6996: feat: As a user, I want to see `grpc-status-details-bin` in response body in case an error, so that I avoid parsing header to read error respons

2022-07-19 Thread GitBox
vyvyan2010 commented on issue #6996: URL: https://github.com/apache/apisix/issues/6996#issuecomment-1189310998 `show_status_in_body`, optional, when enabled, will show decoded `grpc-status-details-bin` in response body `status_detail_type`, optional, the message type of the `grpc-status

[GitHub] [apisix] iGeeky opened a new pull request, #7497: fix(wolf-rbac): Return 403 error code when the user does not have per…

2022-07-19 Thread GitBox
iGeeky opened a new pull request, #7497: URL: https://github.com/apache/apisix/pull/7497 …mission. ### Description Return 403 error code when the user does not have permission. It is more reasonable to return 401 error code when user does not have permission ### Check

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1167: request help: use ingress annotation to support traffic split feature

2022-07-19 Thread GitBox
tao12345666333 commented on issue #1167: URL: https://github.com/apache/apisix-ingress-controller/issues/1167#issuecomment-1189300208 You need to use latest code from master branch or wait for v1.5 released -- This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [apisix] tzssangglass commented on issue #7476: help request: get `etcdserver: requested lease not found` error when running t/deployment/conf_server.t

2022-07-19 Thread GitBox
tzssangglass commented on issue #7476: URL: https://github.com/apache/apisix/issues/7476#issuecomment-1189225508 Because two separate etcd instances are used in these two test cases, one of them will be selected in the balancer phase of the conf_server. This error occurs if grant lease and

[GitHub] [apisix] zhongpengrui opened a new issue, #7496: help request:

2022-07-19 Thread GitBox
zhongpengrui opened a new issue, #7496: URL: https://github.com/apache/apisix/issues/7496 ### Description 大佬好,我有这样一个需求:我使用了basic-auth插件作权限过滤,consumer中注册了用户,然后在路由中需要去访问其他的服务接口,需要将basic-auth插件中的username和password取出来,作为访问后面的路由的参数传递过去,即要对参数进行一次包装,请问这需要怎么去处理? ### Environment

[GitHub] [apisix] soulbird commented on pull request #7412: feat(promethues): add ngx.shared.DICT status

2022-07-19 Thread GitBox
soulbird commented on PR #7412: URL: https://github.com/apache/apisix/pull/7412#issuecomment-1189151264 > > @ccxhwmy could you update the [apisix-grafana-dashboard](https://github.com/apache/apisix/blob/master/docs/assets/other/json/apisix-grafana-dashboard.json) meta for shared dict? >

[GitHub] [apisix] ccxhwmy commented on pull request #7412: feat(promethues): add ngx.shared.DICT status

2022-07-19 Thread GitBox
ccxhwmy commented on PR #7412: URL: https://github.com/apache/apisix/pull/7412#issuecomment-1189148865 > @ccxhwmy could you update the [apisix-grafana-dashboard](https://github.com/apache/apisix/blob/master/docs/assets/other/json/apisix-grafana-dashboard.json) meta for shared dict?

[GitHub] [apisix] soulbird commented on pull request #7412: feat(promethues): add ngx.shared.DICT status

2022-07-19 Thread GitBox
soulbird commented on PR #7412: URL: https://github.com/apache/apisix/pull/7412#issuecomment-1189130191 @ccxhwmy could you update the [apisix-grafana-dashboard](https://github.com/apache/apisix/blob/master/docs/assets/other/json/apisix-grafana-dashboard.json) meta for shared dict? -- Th

[GitHub] [apisix] qihaiyan commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188978968 > > > > I've tried client-control plugin, but it doesn't work. And using the config file is simpler than adding a plugin to all routes. @tokers > > > > > > > > > >

[GitHub] [apisix] tokers commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
tokers commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188977345 > > > I've tried client-control plugin, but it doesn't work. And using the config file is simpler than adding a plugin to all routes. @tokers > > > > > > Something you ne

[GitHub] [apisix] qihaiyan commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188976848 > try with: https://github.com/apache/apisix/blob/master/docs/en/latest/customize-nginx-configuration.md Because ngx_tpl.lua doesn't have a `client_body_buffer_size`, so it can

[GitHub] [apisix] qihaiyan commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188975902 > > I've tried client-control plugin, but it doesn't work. And using the config file is simpler than adding a plugin to all routes. @tokers > > Something you need to confirm:

[GitHub] [apisix-ingress-controller] cai200631 commented on issue #1167: request help: use ingress annotation to support traffic split feature

2022-07-19 Thread GitBox
cai200631 commented on issue #1167: URL: https://github.com/apache/apisix-ingress-controller/issues/1167#issuecomment-1188975148 > Yes! We can use the PluginConfig annotation on the Ingress resource to bind any APISIX plugin to the Ingress resource. > > you can refer to https://gith

[apisix] branch master updated: feat(promethues): add ngx.shared.DICT status (#7412)

2022-07-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 4620ceabc feat(promethues): add ngx.shared.

[GitHub] [apisix] spacewander merged pull request #7412: feat(promethues): add ngx.shared.DICT status

2022-07-19 Thread GitBox
spacewander merged PR #7412: URL: https://github.com/apache/apisix/pull/7412 -- 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 #5837: request: Expect monitoring metrics to add monitoring of nginx shared memory usage

2022-07-19 Thread GitBox
spacewander closed issue #5837: request: Expect monitoring metrics to add monitoring of nginx shared memory usage URL: https://github.com/apache/apisix/issues/5837 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL a

[apisix-docker] branch master updated: chore: sync grafana meta from apisix (#335)

2022-07-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-docker.git The following commit(s) were added to refs/heads/master by this push: new ad3717a chore: sync grafana meta fr

[GitHub] [apisix-docker] spacewander merged pull request #335: chore: sync grafana meta from apisix

2022-07-19 Thread GitBox
spacewander merged PR #335: URL: https://github.com/apache/apisix-docker/pull/335 -- 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

[apisix] branch master updated: chore: remove stale todo at plugin redirect (#7487)

2022-07-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 a8069046f chore: remove stale todo at plugi

[GitHub] [apisix] spacewander merged pull request #7487: chore: remove stale todo at plugin redirect

2022-07-19 Thread GitBox
spacewander merged PR #7487: URL: https://github.com/apache/apisix/pull/7487 -- 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] tao12345666333 commented on issue #7473: bug: use kubernetes dicovery start failed

2022-07-19 Thread GitBox
tao12345666333 commented on issue #7473: URL: https://github.com/apache/apisix/issues/7473#issuecomment-1188897979 this project https://github.com/apache/apisix-ingress-controller -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [apisix] tokers commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
tokers commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188897322 > I've tried client-control plugin, but it doesn't work. And using the config file is simpler than adding a plugin to all routes. @tokers Something you need to confirm:

[GitHub] [apisix] tao12345666333 commented on issue #7473: bug: use kubernetes dicovery start failed

2022-07-19 Thread GitBox
tao12345666333 commented on issue #7473: URL: https://github.com/apache/apisix/issues/7473#issuecomment-1188897313 If you really want to use APISIX in a Kubernetes environment, unless you have a specific reason, I recommend you to use the APISIX Ingress controller instead of this module.

[GitHub] [apisix] tao12345666333 commented on issue #7473: bug: use kubernetes dicovery start failed

2022-07-19 Thread GitBox
tao12345666333 commented on issue #7473: URL: https://github.com/apache/apisix/issues/7473#issuecomment-1188895223 @myoss what's your kubernetes version and Helm chart version? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [apisix] tzssangglass commented on issue #7473: bug: use kubernetes dicovery start failed

2022-07-19 Thread GitBox
tzssangglass commented on issue #7473: URL: https://github.com/apache/apisix/issues/7473#issuecomment-117943 @tao12345666333 Can you do a quick verification for this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] tzssangglass commented on issue #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
tzssangglass commented on issue #7495: URL: https://github.com/apache/apisix/issues/7495#issuecomment-114736 what does` ban cache` means? The route would cache what? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] tzssangglass commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
tzssangglass commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-113808 try with: https://github.com/apache/apisix/blob/master/docs/en/latest/customize-nginx-configuration.md -- This is an automated message from the Apache Git Service. To respond t

[GitHub] [apisix] tzssangglass commented on issue #7485: feat: As a user, I want to logger suppert traceId format, so that logger and trace will be linked.

2022-07-19 Thread GitBox
tzssangglass commented on issue #7485: URL: https://github.com/apache/apisix/issues/7485#issuecomment-111046 We can specify log_format to record the trace_id. ref: https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/kafka-logger.md#metadata here is example:

[GitHub] [apisix-java-plugin-runner] tzssangglass merged pull request #158: feat: support hot reload of plugin filters

2022-07-19 Thread GitBox
tzssangglass merged PR #158: URL: https://github.com/apache/apisix-java-plugin-runner/pull/158 -- 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: notificat

[apisix-java-plugin-runner] branch main updated: feat: support hot reload of plugin filters (#158)

2022-07-19 Thread tzssangglass
This is an automated email from the ASF dual-hosted git repository. tzssangglass pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/apisix-java-plugin-runner.git The following commit(s) were added to refs/heads/main by this push: new 4194525 feat: support hot

[GitHub] [apisix-java-plugin-runner] tzssangglass closed issue #129: request help: Filter for hot loading mechanism in pre or post step plugin

2022-07-19 Thread GitBox
tzssangglass closed issue #129: request help: Filter for hot loading mechanism in pre or post step plugin URL: https://github.com/apache/apisix-java-plugin-runner/issues/129 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] tzssangglass commented on a diff in pull request #7453: feat: supports dynamic control of whether plugins are running

2022-07-19 Thread GitBox
tzssangglass commented on code in PR #7453: URL: https://github.com/apache/apisix/pull/7453#discussion_r924322577 ## apisix/schema_def.lua: ## @@ -956,6 +956,11 @@ _M.plugin_injected_schema = { description = "priority of plugins by customized order",

[GitHub] [apisix] tzssangglass commented on issue #7492: help request: How to build a new docker image with new version without running errors.

2022-07-19 Thread GitBox
tzssangglass commented on issue #7492: URL: https://github.com/apache/apisix/issues/7492#issuecomment-1188867139 > alpine-dev alpine-dev is no longer supported, debian-dev will be supported. -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [apisix] tzssangglass commented on issue #7492: help request: How to build a new docker image with new version without running errors.

2022-07-19 Thread GitBox
tzssangglass commented on issue #7492: URL: https://github.com/apache/apisix/issues/7492#issuecomment-1188862208 try with: https://hub.docker.com/r/api7/apisix-base/tags?page=1&name=dev -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix-docker] tzssangglass closed issue #333: question: use local code how to build

2022-07-19 Thread GitBox
tzssangglass closed issue #333: question: use local code how to build URL: https://github.com/apache/apisix-docker/issues/333 -- 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

[GitHub] [apisix-docker] tzssangglass commented on issue #333: question: use local code how to build

2022-07-19 Thread GitBox
tzssangglass commented on issue #333: URL: https://github.com/apache/apisix-docker/issues/333#issuecomment-1188858559 If you are making images, you can write your own Dockerfle. Not relevant to this project, so close it. -- This is an automated message from the Apache Git Service. T

[GitHub] [apisix] qihaiyan commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
qihaiyan commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188852616 I've tried client-control plugin, but it doesn't work. And using the config file is simpler than adding a plugin to all routes. @tokers -- This is an automated message from the Ap

[GitHub] [apisix] tzssangglass commented on issue #7490: bug: 使用openid-connect插件-public-key一直报错

2022-07-19 Thread GitBox
tzssangglass commented on issue #7490: URL: https://github.com/apache/apisix/issues/7490#issuecomment-1188848884 > 对public-key进行了各种设置,都无法离线验证jwt token > 第一种直接设置公钥,不行 > 第二种在公钥前后增加 -BEGIN PUBLIC KEY- ,还是不行 > 第三种对公钥做base64转换还是不行 see: https://github.com/apache/apisix/blob

[GitHub] [apisix-ingress-controller] codecov-commenter commented on pull request #1170: Refactor: annotation

2022-07-19 Thread GitBox
codecov-commenter commented on PR #1170: URL: https://github.com/apache/apisix-ingress-controller/pull/1170#issuecomment-1188846389 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1170?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaig

[GitHub] [apisix] tzssangglass commented on issue #7488: Whether consumer-restriction can be matched with a re or prefix

2022-07-19 Thread GitBox
tzssangglass commented on issue #7488: URL: https://github.com/apache/apisix/issues/7488#issuecomment-1188844301 I have a question, what is this scene? How did you bind the consumer in APISIX to the concept of what entity in your scenario. -- This is an automated message from the Apache G

[GitHub] [apisix] 1renwuruiming opened a new issue, #7495: help request: how to ban cache?

2022-07-19 Thread GitBox
1renwuruiming opened a new issue, #7495: URL: https://github.com/apache/apisix/issues/7495 ### Description 如何对指定的路由禁用缓存 ### Environment - APISIX version (run `apisix version`): - Operating system (run `uname -a`): - OpenResty / Nginx version (run `openresty -V` or `

[GitHub] [apisix] tokers commented on issue #7488: Whether consumer-restriction can be matched with a re or prefix

2022-07-19 Thread GitBox
tokers commented on issue #7488: URL: https://github.com/apache/apisix/issues/7488#issuecomment-1188828741 So we can add a new field `group` in the `consumer-restriction` plugin and it will apply to all consumers in this group. In such a case, we have two options to support the `group

[GitHub] [apisix] tzssangglass closed issue #7493: 作为一个新手,请教一下配置自定义响应的问题

2022-07-19 Thread GitBox
tzssangglass closed issue #7493: 作为一个新手,请教一下配置自定义响应的问题 URL: https://github.com/apache/apisix/issues/7493 -- 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:

[GitHub] [apisix] tokers commented on issue #7491: feat: As a user, I want to config client_body_buffer_size in config.yaml

2022-07-19 Thread GitBox
tokers commented on issue #7491: URL: https://github.com/apache/apisix/issues/7491#issuecomment-1188823774 @qihaiyan Try to use the `client-control` plugin? -- 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-ingress-controller] AlinsRan opened a new pull request, #1170: Refactor/annotation

2022-07-19 Thread GitBox
AlinsRan opened a new pull request, #1170: URL: https://github.com/apache/apisix-ingress-controller/pull/1170 ### Type of change: - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches ### What this PR does / why we need

[GitHub] [apisix-java-plugin-runner] oridag opened a new issue, #168: The `PluginFilter` API is synchronous and blocks a thread

2022-07-19 Thread GitBox
oridag opened a new issue, #168: URL: https://github.com/apache/apisix-java-plugin-runner/issues/168 This is an issue when the filter performs potentially long running IO because it holds the thread while waiting for a response from the external resource. It would be nice to have an asyn

[GitHub] [apisix-java-plugin-runner] oridag opened a new issue, #167: RpcCallHandler is dispatched on the IO thread

2022-07-19 Thread GitBox
oridag opened a new issue, #167: URL: https://github.com/apache/apisix-java-plugin-runner/issues/167 The `RpcCallHandler` is not assigned a thread pool, which means only a single request can be handled concurrently. This is a show stopper for high throughput systems. To process the r

[GitHub] [apisix-ingress-controller] wolgod commented on issue #1169: request help: apisix-ingress-controller has some warning logs when it starts

2022-07-19 Thread GitBox
wolgod commented on issue #1169: URL: https://github.com/apache/apisix-ingress-controller/issues/1169#issuecomment-1188761219 Are there some routes in apisix that are not created through the apisix ingress controller? -- This is an automated message from the Apache Git Service. T

[GitHub] [apisix] xuminwlt commented on issue #7492: help request: How to build a new docker image with new version without running errors.

2022-07-19 Thread GitBox
xuminwlt commented on issue #7492: URL: https://github.com/apache/apisix/issues/7492#issuecomment-1188760482 works fine in https://github.com/apache/apisix-docker/blob/master/debian-dev/Dockerfile, but alpin-dev not well -- This is an automated message from the Apache Git Service. To res

[GitHub] [apisix-dashboard] codecov-commenter commented on pull request #2530: test: move stream route unit cases to e2e

2022-07-19 Thread GitBox
codecov-commenter commented on PR #2530: URL: https://github.com/apache/apisix-dashboard/pull/2530#issuecomment-1188726727 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2530?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_

  1   2   >