[GitHub] [apisix-ingress-controller] tao12345666333 opened a new pull request, #1630: docs: using tool auto generate references

2023-01-30 Thread via GitHub
tao12345666333 opened a new pull request, #1630: URL: https://github.com/apache/apisix-ingress-controller/pull/1630 Signed-off-by: Jintao Zhang ### Type of change: - [x] Docs ### What this PR does / why we need it: ### Pre-submission ch

[GitHub] [apisix] mscb402 commented on a diff in pull request #8711: feat: file logger plugin support response body in variable

2023-01-30 Thread via GitHub
mscb402 commented on code in PR #8711: URL: https://github.com/apache/apisix/pull/8711#discussion_r1091550772 ## apisix/core/ctx.lua: ## @@ -213,6 +213,10 @@ do route_name = true, service_id = true, service_name = true, +resp_body = function(ct

[GitHub] [apisix-ingress-controller] fatpa commented on a diff in pull request #1519: feat: Add prefer_name into ApisixClusterConfig

2023-01-30 Thread via GitHub
fatpa commented on code in PR #1519: URL: https://github.com/apache/apisix-ingress-controller/pull/1519#discussion_r1091544834 ## utils/kind-with-registry.sh: ## @@ -67,7 +67,7 @@ nodes: containerdConfigPatches: - |- [plugins."io.containerd.grpc.v1.cri".registry.mirrors."

[GitHub] [apisix] ada012 opened a new issue, #8756: help request: How does upstream support subset like functionality in apisix's kubernetes service discovery?

2023-01-30 Thread via GitHub
ada012 opened a new issue, #8756: URL: https://github.com/apache/apisix/issues/8756 ### Description Hi, I see that in apisix's kubernetes service discovery, it supports "label_selector". https://apisix.apache.org/zh/docs/apisix/discovery/kubernetes/ But how to support "l

[apisix-website] branch master updated: chore: APISIX Ingress controller v1.6 (#1489)

2023-01-30 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao 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 dc62647e262 chore: APISIX Ingress c

[GitHub] [apisix-website] tao12345666333 merged pull request #1489: chore: APISIX Ingress controller v1.6

2023-01-30 Thread via GitHub
tao12345666333 merged PR #1489: URL: https://github.com/apache/apisix-website/pull/1489 -- 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-ingress-controller] tao12345666333 commented on pull request #1628: docs: update the-hard-way doc contoller image version to 1.6.0

2023-01-30 Thread via GitHub
tao12345666333 commented on PR #1628: URL: https://github.com/apache/apisix-ingress-controller/pull/1628#issuecomment-1409882436 Thank you for your contribution. Did you follow the documentation exactly for installation with v1.6? -- This is an automated message from the Apache Git Ser

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1595: feat: support disable status

2023-01-30 Thread via GitHub
tao12345666333 commented on code in PR #1595: URL: https://github.com/apache/apisix-ingress-controller/pull/1595#discussion_r1091532263 ## cmd/ingress/ingress.go: ## @@ -192,6 +192,7 @@ For example, no available LB exists in the bare metal environment.`) cmd.Persistent

[GitHub] [apisix-ingress-controller] tao12345666333 closed pull request #1483: ci:test

2023-01-30 Thread via GitHub
tao12345666333 closed pull request #1483: ci:test URL: https://github.com/apache/apisix-ingress-controller/pull/1483 -- 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 unsubscr

[GitHub] [apisix-ingress-controller] tao12345666333 commented on pull request #1483: ci:test

2023-01-30 Thread via GitHub
tao12345666333 commented on PR #1483: URL: https://github.com/apache/apisix-ingress-controller/pull/1483#issuecomment-1409878972 I think we can close this one safely. -- 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-ingress-controller] AlinsRan commented on a diff in pull request #1595: feat: support disable status

2023-01-30 Thread via GitHub
AlinsRan commented on code in PR #1595: URL: https://github.com/apache/apisix-ingress-controller/pull/1595#discussion_r1091529040 ## cmd/ingress/ingress.go: ## @@ -192,6 +192,7 @@ For example, no available LB exists in the bare metal environment.`) cmd.PersistentFlags(

[GitHub] [apisix-ingress-controller] navendu-pottekkat commented on a diff in pull request #1590: chore: update issue templates

2023-01-30 Thread via GitHub
navendu-pottekkat commented on code in PR #1590: URL: https://github.com/apache/apisix-ingress-controller/pull/1590#discussion_r1091530382 ## .github/ISSUE_TEMPLATE/improve_docs.yml: ## @@ -0,0 +1,33 @@ +name: "Documentation Issue" +description: Issues related to documentation.

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1519: feat: Add prefer_name into ApisixClusterConfig

2023-01-30 Thread via GitHub
tao12345666333 commented on code in PR #1519: URL: https://github.com/apache/apisix-ingress-controller/pull/1519#discussion_r1091529318 ## utils/kind-with-registry.sh: ## @@ -67,7 +67,7 @@ nodes: containerdConfigPatches: - |- [plugins."io.containerd.grpc.v1.cri".registry.

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1590: chore: update issue templates

2023-01-30 Thread via GitHub
tao12345666333 commented on code in PR #1590: URL: https://github.com/apache/apisix-ingress-controller/pull/1590#discussion_r1091527102 ## .github/ISSUE_TEMPLATE/improve_docs.yml: ## @@ -0,0 +1,33 @@ +name: "Documentation Issue" +description: Issues related to documentation. +t

[apisix-ingress-controller] branch master updated: feat: add new ApisixGlobalRule resource to support global rules (#1586)

2023-01-30 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git The following commit(s) were added to refs/heads/master by this push: new fa57ff5e feat: add new A

[GitHub] [apisix-ingress-controller] tao12345666333 closed issue #1104: request help: How to create a global rule for a cluster scoped plugin?

2023-01-30 Thread via GitHub
tao12345666333 closed issue #1104: request help: How to create a global rule for a cluster scoped plugin? URL: https://github.com/apache/apisix-ingress-controller/issues/1104 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #1586: feat: ApisixGlobalRule support global rules

2023-01-30 Thread via GitHub
tao12345666333 merged PR #1586: URL: https://github.com/apache/apisix-ingress-controller/pull/1586 -- 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: notif

[GitHub] [apisix-ingress-controller] tao12345666333 opened a new issue, #1629: chore: Update NOTICE year to 2023

2023-01-30 Thread via GitHub
tao12345666333 opened a new issue, #1629: URL: https://github.com/apache/apisix-ingress-controller/issues/1629 This line should be updated https://github.com/apache/apisix-ingress-controller/blob/4c0535ba929c64ed9f9ac603300156879fe79e26/NOTICE#L2 -- This is an automated message fro

[GitHub] [apisix-website] netlify[bot] commented on pull request #1489: chore: APISIX Ingress controller v1.6

2023-01-30 Thread netlify
netlify[bot] commented on PR #1489: URL: https://github.com/apache/apisix-website/pull/1489#issuecomment-1409858014 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | ddb93f4a

[GitHub] [apisix-website] tao12345666333 opened a new pull request, #1489: chore: APISIX Ingress controller v1.6

2023-01-30 Thread via GitHub
tao12345666333 opened a new pull request, #1489: URL: https://github.com/apache/apisix-website/pull/1489 Signed-off-by: Jintao Zhang Fixes: #[Add issue number here] Changes: APISIX Ingress controller v1.6 Screenshots of the change: -- This is an au

[GitHub] [apisix] spacewander opened a new pull request, #8755: chore: upgrade `casbin` to `1.41.5` (#8744)

2023-01-30 Thread via GitHub
spacewander opened a new pull request, #8755: URL: https://github.com/apache/apisix/pull/8755 Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have

[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1595: feat: support disable status

2023-01-30 Thread via GitHub
tao12345666333 commented on code in PR #1595: URL: https://github.com/apache/apisix-ingress-controller/pull/1595#discussion_r1091476830 ## cmd/ingress/ingress.go: ## @@ -192,6 +192,7 @@ For example, no available LB exists in the bare metal environment.`) cmd.Persistent

[GitHub] [apisix-ingress-controller] stubbornTanzhe opened a new pull request, #1628: update contoller image version to 1.6.0

2023-01-30 Thread via GitHub
stubbornTanzhe opened a new pull request, #1628: URL: https://github.com/apache/apisix-ingress-controller/pull/1628 ### Type of change: - [x] Update docs ### What this PR does / why we need it: if the image verion keeps to 1.4.0, the controller pod will CR

[GitHub] [apisix] spacewander opened a new pull request, #8754: feat: release 2.15.2

2023-01-30 Thread via GitHub
spacewander opened a new pull request, #8754: URL: https://github.com/apache/apisix/pull/8754 Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have

[apisix] branch master updated: chore: hi 2023 (#8748)

2023-01-30 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 acee3a877 chore: hi 2023 (#8748) acee3a877

[GitHub] [apisix] spacewander merged pull request #8748: chore: hi 2023

2023-01-30 Thread via GitHub
spacewander merged PR #8748: URL: https://github.com/apache/apisix/pull/8748 -- 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-dashboard] codecov-commenter commented on pull request #2725: chore(deps): bump docker/build-push-action from 2 to 4

2023-01-30 Thread via GitHub
codecov-commenter commented on PR #2725: URL: https://github.com/apache/apisix-dashboard/pull/2725#issuecomment-1409823251 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2725?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_

[GitHub] [apisix-dashboard] dependabot[bot] closed pull request #2447: chore(deps): bump docker/build-push-action from 2 to 3

2023-01-30 Thread dependabot
dependabot[bot] closed pull request #2447: chore(deps): bump docker/build-push-action from 2 to 3 URL: https://github.com/apache/apisix-dashboard/pull/2447 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [apisix-dashboard] dependabot[bot] commented on pull request #2447: chore(deps): bump docker/build-push-action from 2 to 3

2023-01-30 Thread dependabot
dependabot[bot] commented on PR #2447: URL: https://github.com/apache/apisix-dashboard/pull/2447#issuecomment-1409806105 Superseded by #2725. -- 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-dashboard] dependabot[bot] opened a new pull request, #2725: chore(deps): bump docker/build-push-action from 2 to 4

2023-01-30 Thread dependabot
dependabot[bot] opened a new pull request, #2725: URL: https://github.com/apache/apisix-dashboard/pull/2725 Bumps [docker/build-push-action](https://github.com/docker/build-push-action) from 2 to 4. Release notes Sourced from https://github.com/docker/build-push-action/releases";>

[apisix-dashboard] branch dependabot/github_actions/docker/build-push-action-4 created (now d87dc389a)

2023-01-30 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/docker/build-push-action-4 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git at d87dc389a chore(deps): bump docker/build-push-action from

[GitHub] [apisix] pengkang1991 opened a new issue, #8753: APISIX整合nacos遇到的问题(Problems encountered in APISIX integration of nacos)help request:

2023-01-30 Thread via GitHub
pengkang1991 opened a new issue, #8753: URL: https://github.com/apache/apisix/issues/8753 ### Description 在APISIX的服务发现里,我想整合我的nacos,我搭建的是nacos集群。我使用下面的配置后,当我停止192.168.33.1,192.168.33.2这两台的NACOS 服务后,请求路由(Route)"curl -i http://192.168.33.92:9080"; 会报503的错。不会自己去寻找192.168.33.3的nacos服务。

[apisix-ingress-controller] branch dependabot/go_modules/golang.org/x/net-0.5.0 updated (e243a515 -> b8fad3c2)

2023-01-30 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/golang.org/x/net-0.5.0 in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git omit e243a515 chore(deps): bump golang.org/x/net from 0.4.0 t

[apisix-ingress-controller] branch master updated: doc: update 1.6 upgrade guide (#1592)

2023-01-30 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git The following commit(s) were added to refs/heads/master by this push: new 4c0535ba doc: update 1.6

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #1592: doc: update 1.6 upgrade guide

2023-01-30 Thread via GitHub
tao12345666333 merged PR #1592: URL: https://github.com/apache/apisix-ingress-controller/pull/1592 -- 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: notif

[apisix-ingress-controller] branch dependabot/go_modules/golang.org/x/net-0.5.0 updated (de366cb7 -> e243a515)

2023-01-30 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/golang.org/x/net-0.5.0 in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git omit de366cb7 chore(deps): bump golang.org/x/net from 0.4.0 t

[apisix-ingress-controller] branch dependabot/go_modules/sigs.k8s.io/gateway-api-0.6.0 updated (1446d727 -> 879c9af5)

2023-01-30 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/sigs.k8s.io/gateway-api-0.6.0 in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git discard 1446d727 chore(deps): bump sigs.k8s.io/gateway-ap

[apisix-ingress-controller] branch master updated: chore(deps): bump github.com/onsi/ginkgo/v2 in /test/e2e (#1620)

2023-01-30 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git The following commit(s) were added to refs/heads/master by this push: new 1f4ade7b chore(deps): bu

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #1620: chore(deps): bump github.com/onsi/ginkgo/v2 from 2.7.0 to 2.7.1 in /test/e2e

2023-01-30 Thread via GitHub
tao12345666333 merged PR #1620: URL: https://github.com/apache/apisix-ingress-controller/pull/1620 -- 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: notif

[apisix-ingress-controller] branch master updated: chore(deps): bump k8s.io/code-generator from 0.26.0 to 0.26.1 (#1622)

2023-01-30 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-ingress-controller.git The following commit(s) were added to refs/heads/master by this push: new 26a202d7 chore(deps): bu

[GitHub] [apisix-ingress-controller] tao12345666333 merged pull request #1622: chore(deps): bump k8s.io/code-generator from 0.26.0 to 0.26.1

2023-01-30 Thread via GitHub
tao12345666333 merged PR #1622: URL: https://github.com/apache/apisix-ingress-controller/pull/1622 -- 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: notif

[GitHub] [apisix-dashboard] Baoyuantop opened a new issue, #2724: Update the year in the NOTICE file

2023-01-30 Thread via GitHub
Baoyuantop opened a new issue, #2724: URL: https://github.com/apache/apisix-dashboard/issues/2724 # Feature request ## Please describe your feature Update the year in the NOTICE file ## Describe the solution you'd like A clear and concise description of what you wa

[GitHub] [apisix] An-DJ opened a new pull request, #8752: feat(admin): add a welcome statement to admin api

2023-01-30 Thread via GitHub
An-DJ opened a new pull request, #8752: URL: https://github.com/apache/apisix/pull/8752 ### Description Fixes #8741 A demo: ``` # curl "http://localhost:9180"; Welcome to Apache APISIX! # curl "http://localhost:9180"; --head HTTP/1.1 200 OK

[GitHub] [apisix] xiaoxuanzi commented on a diff in pull request #8604: feat(elasticsearch-logger): support multi elasticsearch endpoints(#8431)

2023-01-30 Thread via GitHub
xiaoxuanzi commented on code in PR #8604: URL: https://github.com/apache/apisix/pull/8604#discussion_r1091428796 ## t/plugin/elasticsearch-logger.t: ## @@ -551,3 +551,32 @@ PTQvJEaPcNOXcOHeErC0XQ== } --- response_body passed + + + +=== TEST 14: to show that different endp

[GitHub] [apisix-website] SkyeYoung commented on issue #1469: Site Usability: Apisix versions are easy mistaken

2023-01-30 Thread via GitHub
SkyeYoung commented on issue #1469: URL: https://github.com/apache/apisix-website/issues/1469#issuecomment-1409715179 > @SkyeYoung AFAIK Apisix comprises of various sub components with differrent versions in releases. Multiple versions lead to this issue and other compatibility issues like

[GitHub] [apisix] dickens7 closed issue #8498: help request: How to implement `apisix.core.json` encode output Unicode characters

2023-01-30 Thread via GitHub
dickens7 closed issue #8498: help request: How to implement `apisix.core.json` encode output Unicode characters URL: https://github.com/apache/apisix/issues/8498 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [apisix] monkeyDluffy6017 commented on issue #8747: docs: Tutorial /tutorials/expose-api routing not working as expected

2023-01-30 Thread via GitHub
monkeyDluffy6017 commented on issue #8747: URL: https://github.com/apache/apisix/issues/8747#issuecomment-1409667098 If use the `node` mode, it will failed when there are serveral nodes. It may be confusing too. https://github.com/apache/apisix/blob/master/t/admin/routes4.t#L603 Ho

[GitHub] [apisix] ronething opened a new pull request, #8751: chore: use operator # instead of string.len

2023-01-30 Thread via GitHub
ronething opened a new pull request, #8751: URL: https://github.com/apache/apisix/pull/8751 ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new fea

[GitHub] [apisix] spacewander commented on a diff in pull request #8604: feat(elasticsearch-logger): support multi elasticsearch endpoints(#8431)

2023-01-30 Thread via GitHub
spacewander commented on code in PR #8604: URL: https://github.com/apache/apisix/pull/8604#discussion_r1091372041 ## t/plugin/elasticsearch-logger.t: ## @@ -551,3 +551,32 @@ PTQvJEaPcNOXcOHeErC0XQ== } --- response_body passed + + + +=== TEST 14: to show that different end

[GitHub] [apisix] spacewander commented on a diff in pull request #8722: feat: Support clickhouse-logger plugin log option of request/response body

2023-01-30 Thread via GitHub
spacewander commented on code in PR #8722: URL: https://github.com/apache/apisix/pull/8722#discussion_r1091366946 ## t/plugin/clickhouse-logger2.t: ## @@ -0,0 +1,245 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. S

[GitHub] [apisix] mscb402 commented on issue #8705: feat: As a user, I want file-logger plugin support record response body in a custom format, so that make the log format more flexible

2023-01-30 Thread via GitHub
mscb402 commented on issue #8705: URL: https://github.com/apache/apisix/issues/8705#issuecomment-1409647582 > Hi @mscb402 Can we make this feature to other plugins? (eg clickhouse-logger, elastic-search logger.) because this should be a common feature instead of dedicated to file logger plu

[GitHub] [apisix] spacewander commented on pull request #8651: fix: fix fetch all service info from consul

2023-01-30 Thread via GitHub
spacewander commented on PR #8651: URL: https://github.com/apache/apisix/pull/8651#issuecomment-1409641123 > > [#8651 (comment)](https://github.com/apache/apisix/pull/8651#discussion_r1073064191) It seems there are still indentation issues. Some places use 8-spaces indentation instead of 4

[GitHub] [apisix] oneto1 opened a new issue, #8749: bug: safari download chinese named file from s3 presigned url via apisix, but get garbled characters name

2023-01-30 Thread via GitHub
oneto1 opened a new issue, #8749: URL: https://github.com/apache/apisix/issues/8749 ### Current Behavior * safari download chinese named file from s3 presigned url via apisix , but get garbled characters name * open a https download url , like this ![sc_20230130163707](https

[GitHub] [apisix] rampagecong closed issue #8463: bug: The BUG that was created when the response header was modified using the plugin

2023-01-30 Thread via GitHub
rampagecong closed issue #8463: bug: The BUG that was created when the response header was modified using the plugin URL: https://github.com/apache/apisix/issues/8463 -- 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] rampagecong commented on issue #8463: bug: The BUG that was created when the response header was modified using the plugin

2023-01-30 Thread via GitHub
rampagecong commented on issue #8463: URL: https://github.com/apache/apisix/issues/8463#issuecomment-1409625885 I found the problem.There was a BUG in the PR I submitted earlier for 'apisix-go-plugin-runner'. https://github.dev/apache/apisix-go-plugin-runner/blob/ccd171d45d59ec252e200672

[GitHub] [apisix-helm-chart] tokers commented on issue #455: feat: allow to modify common labels of the resources

2023-01-30 Thread via GitHub
tokers commented on issue #455: URL: https://github.com/apache/apisix-helm-chart/issues/455#issuecomment-1409623871 I think is feasible and reasonable. -- 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-docker] tokers commented on issue #416: 根据apisix-docker/example安装失败

2023-01-30 Thread via GitHub
tokers commented on issue #416: URL: https://github.com/apache/apisix-docker/issues/416#issuecomment-1409623107 What's the APISIX image version? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [apisix] tokers commented on issue #8705: feat: As a user, I want file-logger plugin support record response body in a custom format, so that make the log format more flexible

2023-01-30 Thread via GitHub
tokers commented on issue #8705: URL: https://github.com/apache/apisix/issues/8705#issuecomment-1409622267 > Hi @mscb402 Can we make this feature to other plugins? (eg clickhouse-logger, elastic-search logger.) because this should be a common feature instead of dedicated to file logger plug

[GitHub] [apisix] gaoxingliang commented on issue #5915: request help: logger plugins supports limit size of request/response body

2023-01-30 Thread via GitHub
gaoxingliang commented on issue #5915: URL: https://github.com/apache/apisix/issues/5915#issuecomment-1409614738 Hi @juzhiyuan The pr is not merged. do we have any plan to support this feature? This is a good idea in production environment to do api statistics. And for now (apisix vers

[GitHub] [apisix] monkeyDluffy6017 commented on issue #8745: feat: As a user, I want to add EDAS registry discovery support, to enable requests redirecting to EDAS hosted micro services directly.

2023-01-30 Thread via GitHub
monkeyDluffy6017 commented on issue #8745: URL: https://github.com/apache/apisix/issues/8745#issuecomment-1409613227 Are you interested in developing this feature? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

[apisix] branch master updated: refactor(admin): stream_routes/upstreams/protos/services/global_rules/consumer_groups/plugin_configs (#8661)

2023-01-30 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 22be6d08c refactor(admin): stream_routes/u

[GitHub] [apisix] spacewander merged pull request #8661: refactor(admin): stream_routes/upstreams/protos/services/global_rules/consumer_groups/plugin_configs

2023-01-30 Thread via GitHub
spacewander merged PR #8661: URL: https://github.com/apache/apisix/pull/8661 -- 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 (5a3a8080c -> a5dc4c336)

2023-01-30 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 5a3a8080c chore: upgrade `casbin` to `1.41.5` (#8744) add a5dc4c336 feat: support send error-log to kafka br

[GitHub] [apisix] spacewander closed issue #8678: feat: As a user, I want to support send error-log to kafka, so that we can get error logs from kafka

2023-01-30 Thread via GitHub
spacewander closed issue #8678: feat: As a user, I want to support send error-log to kafka, so that we can get error logs from kafka URL: https://github.com/apache/apisix/issues/8678 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [apisix] spacewander merged pull request #8693: feat: support send error-log to kafka brokers

2023-01-30 Thread via GitHub
spacewander merged PR #8693: URL: https://github.com/apache/apisix/pull/8693 -- 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 opened a new pull request, #8748: chore: hi 2023

2023-01-30 Thread via GitHub
spacewander opened a new pull request, #8748: URL: https://github.com/apache/apisix/pull/8748 Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have

[apisix] branch master updated: chore: upgrade `casbin` to `1.41.5` (#8744)

2023-01-30 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 5a3a8080c chore: upgrade `casbin` to `1.41.

[GitHub] [apisix] spacewander merged pull request #8744: chore: upgrade `casbin` to `1.41.5`

2023-01-30 Thread via GitHub
spacewander merged PR #8744: URL: https://github.com/apache/apisix/pull/8744 -- 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] gaoxingliang commented on issue #8705: feat: As a user, I want file-logger plugin support record response body in a custom format, so that make the log format more flexible

2023-01-30 Thread via GitHub
gaoxingliang commented on issue #8705: URL: https://github.com/apache/apisix/issues/8705#issuecomment-1409600592 Hi @mscb402 Can we make this feature to other plugins? (eg clickhouse-logger, elastic-search logger.) because this should be a common feature instead of dedicated to file logger

[GitHub] [apisix] schubidos opened a new issue, #8747: docs: Tutorial /tutorials/expose-api routing not working as expected

2023-01-30 Thread via GitHub
schubidos opened a new issue, #8747: URL: https://github.com/apache/apisix/issues/8747 ### Current State Following the description the test request is forewarded to example.com instead of httpbin.org. Problem occured with version 3.1 apisix docker on windows 11 running with docker

[GitHub] [apisix-website] nfrankel commented on issue #980: [Site]: support structured schema (JSON-LD)

2023-01-30 Thread via GitHub
nfrankel commented on issue #980: URL: https://github.com/apache/apisix-website/issues/980#issuecomment-1408884935 Yes, I wanted to work on that months ago... and did nothing. Feel free to own it -- This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1627: bug: cannot delete consumer if secret used in KeyAuth secret ref has been deleted first

2023-01-30 Thread via GitHub
tao12345666333 commented on issue #1627: URL: https://github.com/apache/apisix-ingress-controller/issues/1627#issuecomment-1408869448 thanks for the report, add this to my queue. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

[GitHub] [apisix-ingress-controller] davidpoltorak-io opened a new issue, #1627: bug: cannot delete consumer if secret used in KeyAuth secret ref has been deleted first

2023-01-30 Thread via GitHub
davidpoltorak-io opened a new issue, #1627: URL: https://github.com/apache/apisix-ingress-controller/issues/1627 ### Issue description TL;DR When deleting Consumer manifests - and - if using an authentication scheme such as `KeyAuth` which references a Kubernetes secret - if th

[GitHub] [apisix-website] navendu-pottekkat commented on issue #980: [Site]: support structured schema (JSON-LD)

2023-01-30 Thread via GitHub
navendu-pottekkat commented on issue #980: URL: https://github.com/apache/apisix-website/issues/980#issuecomment-1408773072 @juzhiyuan @nfrankel @SkyeYoung we have to work on this to improve discoverability in Google. From experience, adding schema to the site can help in getting a "Knowled

[GitHub] [apisix-website] RajVarsani commented on issue #1469: Site Usability: Apisix versions are easy mistaken

2023-01-30 Thread via GitHub
RajVarsani commented on issue #1469: URL: https://github.com/apache/apisix-website/issues/1469#issuecomment-1408683210 Update: I've discarded all previous changes and updated vs code prefs, so LF-CRLF issue seems to be resolved. -- This is an automated message from the Apache Git Service

[GitHub] [apisix-website] SkyeYoung commented on issue #1469: Site Usability: Apisix versions are easy mistaken

2023-01-30 Thread via GitHub
SkyeYoung commented on issue #1469: URL: https://github.com/apache/apisix-website/issues/1469#issuecomment-1408680347 @RajVarsani For the end-of-line symbol problem, I think you can fix it by clicking `Ctrl + Shift + P` and typing `Change End of Line Sequence` and selecting `LF`.

[GitHub] [apisix-website] RajVarsani commented on issue #1469: Site Usability: Apisix versions are easy mistaken

2023-01-30 Thread via GitHub
RajVarsani commented on issue #1469: URL: https://github.com/apache/apisix-website/issues/1469#issuecomment-1408657773 @SkyeYoung, I set up the codebase in my locally. I have encountered a couple of issues. I opened it in the Dev container, and it shows 624 (Which is the total numbe

[GitHub] [apisix-website] RajVarsani commented on issue #1469: Site Usability: Apisix versions are easy mistaken

2023-01-30 Thread via GitHub
RajVarsani commented on issue #1469: URL: https://github.com/apache/apisix-website/issues/1469#issuecomment-1408655707 @SkyeYoung, I set up the codebase in my locally. I have encountered a couple of issues. I opened it in the Dev container, and it shows 624 (Which is the total number of

[GitHub] [apisix-ingress-controller] goblain opened a new pull request, #1626: Epnotfound

2023-01-30 Thread via GitHub
goblain opened a new pull request, #1626: URL: https://github.com/apache/apisix-ingress-controller/pull/1626 ### Type of change: Bugfix ### What this PR does / why we need it: Solves #1625 -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1625: bug: lots of recurring error logs on removal of helm release (deletion of multiple resources in short period of time)

2023-01-30 Thread via GitHub
tao12345666333 commented on issue #1625: URL: https://github.com/apache/apisix-ingress-controller/issues/1625#issuecomment-1408382709 Thanks for your report -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [apisix-docker] WVenus opened a new issue, #416: 根据apisix-docker/example安装失败

2023-01-30 Thread via GitHub
WVenus opened a new issue, #416: URL: https://github.com/apache/apisix-docker/issues/416 背景: 根据Installing APISIX 文章中,使用dockers部署apisix,启动失败。 报错信息: ![image](https://user-images.githubusercontent.com/39941928/215453007-014bd4f0-9a50-46cd-8bbb-ad6b95b01d7e.png) git分支信息: ![imag

[GitHub] [apisix-ingress-controller] goblain opened a new issue, #1625: bug: lots of recurring error logs on removal of helm release (deletion of multiple resources in short period of time)

2023-01-30 Thread via GitHub
goblain opened a new issue, #1625: URL: https://github.com/apache/apisix-ingress-controller/issues/1625 ### Issue description Whenever we perform a large scale removal of resources with removal of our helm releases, we see a significant amount of error logs due to ingress controller

[GitHub] [apisix] github-actions[bot] commented on pull request #8385: docs: refactor doc dir

2023-01-30 Thread github-actions
github-actions[bot] commented on PR #8385: URL: https://github.com/apache/apisix/pull/8385#issuecomment-1408323022 This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull

[GitHub] [apisix] github-actions[bot] commented on issue #6304: request help: use redis or other database storage data

2023-01-30 Thread github-actions
github-actions[bot] commented on issue #6304: URL: https://github.com/apache/apisix/issues/6304#issuecomment-1408322665 This issue has been marked as stale due to 350 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please simp

[GitHub] [apisix-website] SkyeYoung commented on a diff in pull request #1488: docs:add something about api gateway policy blog

2023-01-30 Thread via GitHub
SkyeYoung commented on code in PR #1488: URL: https://github.com/apache/apisix-website/pull/1488#discussion_r1090404831 ## blog/zh/blog/2023/01/30/something-about-api-gateway-policy.md: ## @@ -0,0 +1,226 @@ +--- +title: "API 网关策略的二三事" +authors: + - name: "暴渊" +title: "Autho

[GitHub] [apisix-helm-chart] ikatlinsky commented on issue #455: feat: allow to modify common labels of the resources

2023-01-30 Thread via GitHub
ikatlinsky commented on issue #455: URL: https://github.com/apache/apisix-helm-chart/issues/455#issuecomment-1408292303 Just want to know if the proposed option is ok and is not breaking anything. In case the solution does not match any best practices - I am open to proposals and improveme

[GitHub] [apisix-helm-chart] ikatlinsky opened a new issue, #455: feat: allow to modify common labels of the resources

2023-01-30 Thread via GitHub
ikatlinsky opened a new issue, #455: URL: https://github.com/apache/apisix-helm-chart/issues/455 **Issue** Currently, common label structure and some values are hardcoded in the following form ```yaml {{- define "apisix.selectorLabels" -}} app.kubernetes.io/name: {{ inclu

[GitHub] [apisix] alickreborn opened a new issue, #8745: feat: As a user, I want to add EDAS registry discovery support, to enable requests redirecting to EDAS hosted micro services directly.

2023-01-30 Thread via GitHub
alickreborn opened a new issue, #8745: URL: https://github.com/apache/apisix/issues/8745 ### Description # Background [EDAS](www.aliyun.com/product/edas) is an Alibaba Cloud product used for hosting micro service applications, which supports micro service frameworks such as Du

[GitHub] [apisix-docker] tokers commented on issue #411: Not same user (root VS apisix) used depending on Linux distribution

2023-01-30 Thread via GitHub
tokers commented on issue #411: URL: https://github.com/apache/apisix-docker/issues/411#issuecomment-1408242094 > @soulbird Some users have this requirement, and modified the user permissions specifically for debian. What do you think about this phenomenon of inconsistent permissions? #326

[GitHub] [apisix-docker] tokers commented on issue #415: 路由配置中关闭插件无法生效

2023-01-30 Thread via GitHub
tokers commented on issue #415: URL: https://github.com/apache/apisix-docker/issues/415#issuecomment-1408240435 @bzp2010 Is this due to the incompatibility between APISIX and dashboard? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix] leslie-tsang opened a new pull request, #8744: fix: use fixed `luaposix` version to avoid build error

2023-01-30 Thread via GitHub
leslie-tsang opened a new pull request, #8744: URL: https://github.com/apache/apisix/pull/8744 ### Description Fixes # (issue) Currently [luaposix](https://luarocks.org/modules/gvvaughan/luaposix) has released version [36.0-1](https://luarocks.org/modules/gvvaughan

[GitHub] [apisix] spacewander commented on issue #8741: feat: As a user, I want APISIX to support the head HTTP request, so that we can check if the APISIX is available easily

2023-01-30 Thread via GitHub
spacewander commented on issue #8741: URL: https://github.com/apache/apisix/issues/8741#issuecomment-1408182820 Reasonable if it is only used to get starting -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abo

[GitHub] [apisix] rampagecong commented on issue #8463: bug: The BUG that was created when the response header was modified using the plugin

2023-01-30 Thread via GitHub
rampagecong commented on issue #8463: URL: https://github.com/apache/apisix/issues/8463#issuecomment-1408160241 @spacewander Please have a look at the above problems, can you provide a solution? https://github.dev/apache/apisix/blob/edb8edfd42943c1e112b259f1c1c29bd41136512/apisix/cor

[GitHub] [apisix] xiaoxuanzi commented on a diff in pull request #8604: feat(elasticsearch-logger): support multi elasticsearch endpoints(#8431)

2023-01-30 Thread via GitHub
xiaoxuanzi commented on code in PR #8604: URL: https://github.com/apache/apisix/pull/8604#discussion_r1090206581 ## t/plugin/elasticsearch-logger.t: ## @@ -515,3 +515,39 @@ apisix: --- response_body 123456 PTQvJEaPcNOXcOHeErC0XQ== + + + +=== TEST 13: add plugin on routes usin

[GitHub] [apisix] An-DJ commented on issue #8741: feat: As a user, I want APISIX to support the head HTTP request, so that we can check if the APISIX is available easily

2023-01-30 Thread via GitHub
An-DJ commented on issue #8741: URL: https://github.com/apache/apisix/issues/8741#issuecomment-1408154278 @spacewander It mey be useful to the new user for their first try with APISIX. After the APISIX is installed (the Admin API is usually opened by default), the user can easily ch