[GitHub] [apisix-website] yzeng25 commented on a change in pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
yzeng25 commented on a change in pull request #809: URL: https://github.com/apache/apisix-website/pull/809#discussion_r767482767 ## File path: website/blog/2021/12/13/monitor-apisix-ingress-controller-with-prometheus.md ## @@ -0,0 +1,184 @@ +--- +title: "Monitoring APISIX Ingress Controller with Prometheus" +authors: + - name: "Chao Zhang" +title: "Author" +url: "https://github.com/tokers; +image_url: "https://avatars.githubusercontent.com/u/10428333?v=4; + - name: "Sylvia" +title: "Technical Writer" +url: "https://github.com/SylviaBABY; +image_url: "https://avatars.githubusercontent.com/u/39793568?v=4; +keywords: +- Apache APISIX +- APISIX Ingress Controller +- Prometheus +- Metrics +- Grafana +description: This article introduces the relevant steps of how to use Prometheus to monitor APISIX Ingress Controller and the display effect of some indicators. +tags: [Technology] +--- + +> This article introduces the relevant steps of how to use Prometheus to monitor APISIX Ingress Controller and the display effect of some indicators. + + + +Whether in the days of monolithic applications or today's cloud-native era, "monitoring functions" have always played a very important role. A good monitoring system can help engineers quickly understand the status of services running in production environments, and quickly locate problems or warn of anomalies when they occur. + +Apache APISIX Ingress Controller has been enhanced to support Prometheus Metrics in recent releases. In this article, we will introduce how to use Prometheus to collect Metrics data from APISIX Ingress Controller and subsequently visualize it through Grafana. + +## Step 1: Install APISIX Ingress Controller + +First we deploy Apache APISIX, ETCD and APISIX Ingress Controller to the local Kubernetes cluster via [Helm](https://helm.sh/). + +```shell +helm repo add apisix https://charts.apiseven.com +helm repo update +kubectl create namespace ingress-apisix +helm install apisix apisix/apisix --namespace ingress-apisix \ + --set ingress-controller.enabled=true +``` + +After installation, please wait until all services are up and running. Specific status confirmation can be checked with the following command. + +```shell +kubectl get all -n ingress-apisix +``` + +## Step 2: Enable the Prometheus Plugin + +In most cases, the monitoring function must involve more than just the APISIX Ingress Controller component. If you need to monitor Apache APISIX at the same time, you can create the following `ApisixClusterConfig` resource. + +### Installing Prometheus and Grafana + +Next, we will enable the Prometheus service through the Prometheus Operator, so you will need to install the Prometheus Operator first. + +>The following command will also install Grafana. Review comment: could you also apply `:::note` here? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on a change in pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
yzeng25 commented on a change in pull request #809: URL: https://github.com/apache/apisix-website/pull/809#discussion_r767481907 ## File path: website/i18n/zh/docusaurus-plugin-content-blog/2021/12/13/monitor-apisix-ingress-controller-with-prometheus.md ## @@ -0,0 +1,194 @@ +--- +title: "使用 Prometheus 监控 APISIX Ingress Controller" +authors: + - name: "张超" +title: "Author" +url: "https://github.com/tokers; +image_url: "https://avatars.githubusercontent.com/u/10428333?v=4; + - name: "苏钰" +title: "Technical Writer" +url: "https://github.com/SylviaBABY; +image_url: "https://avatars.githubusercontent.com/u/39793568?v=4; +keywords: +- Apache APISIX +- APISIX Ingress Controller +- Prometheus +- Metrics +- Grafana +description: 本文介绍了如何使用 Prometheus 监控 APISIX Ingress Controller 的相关步骤和部分指标的展示效果。 +tags: [Technology] +--- + +> 本文介绍了如何使用 Prometheus 监控 APISIX Ingress Controller 的相关步骤和部分指标的展示效果。 + + + +无论是之前单体应用的时代，或是云原生大行其道的今天，「监控功能」一直扮演着十分重要的角色。一个良好的监控体系能够帮助工程师们快速了解运行在生产环境中的服务状态，并在服务异常时快速定位问题或者提前预警异常的发生。 + +Apache APISIX Ingress Controller 在最近几个版本中加强了对 Prometheus Metrics 的支持。本文将为大家介绍如何使用 Prometheus 收集 APISIX Ingress Controller 的 Metrics 数据，并通过 Grafana 进行后续可视化呈现。 + +## 步骤一：安装 APISIX Ingress Controller + +首先我们通过 [Helm](https://helm.sh/) 将 Apache APISIX、ETCD 和 APISIX Ingress Controller 部署到本地的 Kubernetes 集群中。 + +```shell +helm repo add apisix https://charts.apiseven.com +helm repo update +kubectl create namespace ingress-apisix +helm install apisix apisix/apisix --namespace ingress-apisix \ + --set ingress-controller.enabled=true +``` + +安装完毕后请等待，直至所有服务都已正常运行。具体状态确认可通过下述命令进行检查。 + +```shell +kubectl get all -n ingress-apisix +``` + +## 步骤二：启用 Prometheus 插件 + +大部分情况下，监控功能的涉及面肯定不止 APISIX Ingress Controller 这一个组件，如果需要同时监控 Apache APISIX，可以创建以下 `ApisixClusterConfig` 资源。 + +```yaml +apiVersion: apisix.apache.org/v2alpha1 +kind: ApisixClusterConfig +metadata: + name: default +spec: + monitoring: +prometheus: + enable: true +``` + +### 安装 Prometheus 和 Grafana + +接下来我们将通过 Prometheus Operator 来启用 Prometheus 服务，因此需要先安装 Prometheus Operator。 + +> 下述命令同时也会安装 Grafana Review comment: I see `:::note` is used in English version, could you also use it here? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-go-plugin-runner] spacewander commented on issue #43: bug: WARN server/server.go:59 key not found
spacewander commented on issue #43: URL: https://github.com/apache/apisix-go-plugin-runner/issues/43#issuecomment-992199023 https://github.com/apache/apisix/pull/5782 Fix this issue more thoroughly. When the token key in the cache doesn't match the conf in the runner (probably in the dev mode, after people restart the runner), a new lrucache will be created. This lrucache hasn't invalid stale cache until this PR fixes it. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bisakhmondal commented on issue #5775: request help: stream-proxy as mysql proxy not work
bisakhmondal commented on issue #5775: URL: https://github.com/apache/apisix/issues/5775#issuecomment-992198640 @yunfhu Glad, it worked. You can also take a look at the documentation added at #5783 Have a nice day! -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
netlify[bot] edited a comment on pull request #809: URL: https://github.com/apache/apisix-website/pull/809#issuecomment-992180553 ✔️ Deploy Preview for *apache-apisix* ready! Explore the source changes: 81ee4a43e444c5d79e9a7ed2ea782318cb243235 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6fa82c896930008afb6f5](https://app.netlify.com/sites/apache-apisix/deploys/61b6fa82c896930008afb6f5) Browse the preview: [https://deploy-preview-809--apache-apisix.netlify.app](https://deploy-preview-809--apache-apisix.netlify.app) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] yunfhu commented on issue #5775: request help: stream-proxy as mysql proxy not work
yunfhu commented on issue #5775: URL: https://github.com/apache/apisix/issues/5775#issuecomment-992194387 @bisakhmondal thanks for your advice, the access works now -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] yunfhu closed issue #5775: request help: stream-proxy as mysql proxy not work
yunfhu closed issue #5775: URL: https://github.com/apache/apisix/issues/5775 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
netlify[bot] edited a comment on pull request #809: URL: https://github.com/apache/apisix-website/pull/809#issuecomment-992180553 Deploy Preview for *apache-apisix* processing. Explore the source changes: 81ee4a43e444c5d79e9a7ed2ea782318cb243235 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6fa82c896930008afb6f5](https://app.netlify.com/sites/apache-apisix/deploys/61b6fa82c896930008afb6f5) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on a change in pull request #5745: feat(vault): vault lua module, integration with jwt-auth authentication plugin
spacewander commented on a change in pull request #5745:
URL: https://github.com/apache/apisix/pull/5745#discussion_r767475036
##
File path: t/plugin/jwt-auth-vault.t
##
@@ -39,6 +39,16 @@ _EOC_
$block->set_value("http_config", $http_config);
+my $vault_config = $block->extra_yaml_config // <<_EOC_;
+vault:
+ host: "http://0.0.0.0:8200;
+ timeout: 10
+ prefix: kv/apisix
+ token: root
+_EOC_
+
+$block->set_value("extra_yaml_config", $vault_config);
+
Review comment:
LGTM
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] spacewander commented on a change in pull request #5782: fix(ext-plugin): don't use stale key
spacewander commented on a change in pull request #5782:
URL: https://github.com/apache/apisix/pull/5782#discussion_r767474155
##
File path: apisix/plugins/ext-plugin/init.lua
##
@@ -677,6 +677,7 @@ local function create_lrucache()
lrucache = core.lrucache.new({
type = "plugin",
+ invalid_stale = true,
Review comment:
Could you do some refactor so we don't need to specify the lrucache
options twice? Thanks!
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
netlify[bot] edited a comment on pull request #809: URL: https://github.com/apache/apisix-website/pull/809#issuecomment-992180553 ✔️ Deploy Preview for *apache-apisix* ready! Explore the source changes: 8c70d734e25aba6e4921e51a230bcfbb6f71e36b Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6f628fc7d5d0007db232b](https://app.netlify.com/sites/apache-apisix/deploys/61b6f628fc7d5d0007db232b) Browse the preview: [https://deploy-preview-809--apache-apisix.netlify.app](https://deploy-preview-809--apache-apisix.netlify.app) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
netlify[bot] edited a comment on pull request #809: URL: https://github.com/apache/apisix-website/pull/809#issuecomment-992180553 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] commented on pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
netlify[bot] commented on pull request #809: URL: https://github.com/apache/apisix-website/pull/809#issuecomment-992180553 Deploy Preview for *apache-apisix* processing. Explore the source changes: 4476f6e79a0de3b5276f5df13a2d2ffef334ac33 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6f5a82f245d0008c9b844](https://app.netlify.com/sites/apache-apisix/deploys/61b6f5a82f245d0008c9b844) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] SylviaBABY opened a new pull request #809: docs: add Monitor Ingress Controller with Prometheus blog
SylviaBABY opened a new pull request #809: URL: https://github.com/apache/apisix-website/pull/809 Changes: add Monitor APISIX Ingress Controller with Prometheus blog -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] qq54903099 opened a new issue #5784: request help: hmac-auth how to resist replay attacks
qq54903099 opened a new issue #5784: URL: https://github.com/apache/apisix/issues/5784 ### Issue description how to use hmac-auth plugin to resist replay attacks. ### Environment - apisix version (cmd: `apisix version`): 2.10.2 - OS (cmd: `uname -a`): - OpenResty / Nginx version (cmd: `nginx -V` or `openresty -V`): - etcd version, if have (cmd: run `curl http://127.0.0.1:9090/v1/server_info` to get the info from server-info API): - apisix-dashboard version, if have: - the plugin runner version, if the issue is about a plugin runner (cmd: depended on the kind of runner): - luarocks version, if the issue is about installation (cmd: `luarocks --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 the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bisakhmondal opened a new pull request #5783: docs: improve stream proxy filtering with example
bisakhmondal opened a new pull request #5783: URL: https://github.com/apache/apisix/pull/5783 ### What this PR does / why we need it: ### Pre-submission checklist: * [x] Did you explain what problem does this PR solve? Or what new features have been added? * [ ] Have you added corresponding test cases? * [x] Have you modified the corresponding document? * [x] Is this PR backward compatible? **If it is not backward compatible, please discuss on the [mailing list](https://github.com/apache/apisix/tree/master#community) first** -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] codecov-commenter commented on pull request #2242: chore(deps): bump actions/stale from 4.0.0 to 4.1.0
codecov-commenter commented on pull request #2242: URL: https://github.com/apache/apisix-dashboard/pull/2242#issuecomment-992149473 # [Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2242?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) Report > Merging [#2242](https://codecov.io/gh/apache/apisix-dashboard/pull/2242?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) (fcf9bd1) into [master](https://codecov.io/gh/apache/apisix-dashboard/commit/0db2d464049b0d8ae4dcde1c71a68c7ea10a2a18?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) (0db2d46) will **increase** coverage by `2.94%`. > The diff coverage is `n/a`. [](https://codecov.io/gh/apache/apisix-dashboard/pull/2242?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) ```diff @@Coverage Diff @@ ## master#2242 +/- ## == + Coverage 65.17% 68.12% +2.94% == Files 184 127 -57 Lines7274 3369-3905 Branches 829 829 == - Hits 4741 2295-2446 + Misses 2250 1074-1176 + Partials 2830 -283 ``` | Flag | Coverage Δ | | |---|---|---| | backend-e2e-test | `?` | | | backend-unit-test | `?` | | | frontend-e2e-test | `68.12% <ø> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment) to find out more. | [Impacted Files](https://codecov.io/gh/apache/apisix-dashboard/pull/2242?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) | Coverage Δ | | |---|---|---| | [api/internal/utils/json\_patch.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL3V0aWxzL2pzb25fcGF0Y2guZ28=) | | | | [api/internal/core/migrate/migrate.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL2NvcmUvbWlncmF0ZS9taWdyYXRlLmdv) | | | | [api/internal/utils/utils.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL3V0aWxzL3V0aWxzLmdv) | | | | [api/internal/core/store/validate.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL2NvcmUvc3RvcmUvdmFsaWRhdGUuZ28=) | | | | [api/internal/core/migrate/dataset.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL2NvcmUvbWlncmF0ZS9kYXRhc2V0Lmdv) | | | | [api/internal/handler/data\_loader/route\_import.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL2hhbmRsZXIvZGF0YV9sb2FkZXIvcm91dGVfaW1wb3J0Lmdv) | | | | [api/internal/utils/version.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL3V0aWxzL3ZlcnNpb24uZ28=) | | | | [api/internal/core/entity/format.go](https://codecov.io/gh/apache/apisix-dashboard/pull/2242/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-YXBpL2ludGVybmFsL2NvcmUvZW50aXR5L2Zvcm1hdC5nbw==) | | | |
[GitHub] [apisix] spacewander commented on issue #5755: [DISCUSS]: Improving the performance of the prometheus plugin
spacewander commented on issue #5755: URL: https://github.com/apache/apisix/issues/5755#issuecomment-992141666 I think this solution is similar to https://github.com/Kong/kong/commit/3fc396173856d6377d906543d6e538fccd3fea28? Maybe we can also use counter for it: https://github.com/Kong/kong/blob/d65101fe80fd7ac9870a84d34d81bda8bcb461ac/kong/tools/utils.lua#L1438 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] dependabot[bot] opened a new pull request #2242: chore(deps): bump actions/stale from 4.0.0 to 4.1.0
dependabot[bot] opened a new pull request #2242: URL: https://github.com/apache/apisix-dashboard/pull/2242 Bumps [actions/stale](https://github.com/actions/stale) from 4.0.0 to 4.1.0. Release notes Sourced from https://github.com/actions/stale/releases;>actions/stale's releases. v4.1.0 Features https://github.com/actions/stale/commit/9912fa74d1c01b5d6187793d97441019cbe325d0;>Ability to exempt draft PRs Changelog Sourced from https://github.com/actions/stale/blob/main/CHANGELOG.md;>actions/stale's changelog. https://github.com/actions/stale/compare/v3.0.19...v4.1.0;>4.1.0 (2021-07-14) Features https://github.com/actions/stale/commit/9912fa74d1c01b5d6187793d97441019cbe325d0;>Ability to exempt draft PRs Commits https://github.com/actions/stale/commit/7fb802b3079a276cf3c7e6ba9aa003c665b3f838;>7fb802b Bump release to 4.1.0 https://github.com/actions/stale/commit/54197c7137c7221265ea1097805e48ab88497b94;>54197c7 Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/602;>#602 from actions/revert-breaking-change https://github.com/actions/stale/commit/3a971aeb80eea6ad458e4732610b5f50f2dd4708;>3a971ae Revert Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/586;>#586 from C0ZEN/feature/split-pr-and-issue-options https://github.com/actions/stale/commit/fc4a5ff9425599c9a0575c9320984977b05c2e9e;>fc4a5ff Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/595;>#595 from actions/dependabot/npm_and_yarn/types/jest-27.0.2 https://github.com/actions/stale/commit/db699ab3b1799e48475827e8e83083088927fd54;>db699ab Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/586;>#586 from C0ZEN/feature/split-pr-and-issue-options https://github.com/actions/stale/commit/f8e08de81bfdd55cf8e28a406355757430f05b5f;>f8e08de build(deps-dev): bump @types/jest from 27.0.1 to 27.0.2 https://github.com/actions/stale/commit/b83d488cb987fd3a570b32d75c9f04a7563b7979;>b83d488 Update dependencies (https://github-redirect.dependabot.com/actions/stale/issues/592;>#592) https://github.com/actions/stale/commit/1ff6cd74cb1c5a5c87c8950202e80ce0f2b2a979;>1ff6cd7 build(deps): bump tmpl from 1.0.4 to 1.0.5 (https://github-redirect.dependabot.com/actions/stale/issues/575;>#575) https://github.com/actions/stale/commit/86fed0e1f119e59b0690309530b9e60717977ea2;>86fed0e build(deps-dev): bump jest-silent-reporter from 0.4.0 to 0.5.0 (https://github-redirect.dependabot.com/actions/stale/issues/573;>#573) https://github.com/actions/stale/commit/67004407a6a06b621969a09f818cc9b477f0d733;>6700440 docs(readme): split the options between common, issues and prs Additional commits viewable in https://github.com/actions/stale/compare/v4.0.0...v4.1.0;>compare view [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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.apache.org For queries
[apisix-dashboard] branch dependabot/github_actions/actions/stale-4.1.0 created (now fcf9bd1)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/actions/stale-4.1.0 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at fcf9bd1 chore(deps): bump actions/stale from 4.0.0 to 4.1.0 No new revisions were added by this update.
[GitHub] [apisix] bisakhmondal commented on a change in pull request #5745: feat(vault): vault lua module, integration with jwt-auth authentication plugin
bisakhmondal commented on a change in pull request #5745:
URL: https://github.com/apache/apisix/pull/5745#discussion_r767432449
##
File path: t/plugin/jwt-auth-vault.t
##
@@ -39,6 +39,16 @@ _EOC_
$block->set_value("http_config", $http_config);
+my $vault_config = $block->extra_yaml_config // <<_EOC_;
+vault:
+ host: "http://0.0.0.0:8200;
+ timeout: 10
+ prefix: kv/apisix
+ token: root
+_EOC_
+
+$block->set_value("extra_yaml_config", $vault_config);
+
Review comment:
How is it now? @spacewander
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] S96EA opened a new pull request #5782: fix(ext-plugin): don't use stale key
S96EA opened a new pull request #5782: URL: https://github.com/apache/apisix/pull/5782 ### What this PR does / why we need it: ext_plugin an expired key in lrucachd that could not find in go runner。 patch for [#5309](https://github.com/apache/apisix/pull/5309) ### Pre-submission checklist: * [*] Did you explain what problem does this PR solve? Or what new features have been added? * [ ] Have you added corresponding test cases? * [ ] Have you modified the corresponding document? * [*] Is this PR backward compatible? **If it is not backward compatible, please discuss on the [mailing list](https://github.com/apache/apisix/tree/master#community) first** -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bisakhmondal commented on issue #5775: request help: stream-proxy as mysql proxy not work
bisakhmondal commented on issue #5775:
URL: https://github.com/apache/apisix/issues/5775#issuecomment-992123547
Hi @yunfhu, thanks for the detailed analysis
> login apisix pod with kubectl exec command,I use the telnet command to
check the stream proxy route, it shows able route to mysql host
>
> ```
> @MacBook-Pro APISIX % kubectl exec -it apisix-557f5f646f-r69z9 sh
> kubectl exec [POD] [COMMAND] is DEPRECATED and will be removed in a future
version. Use kubectl exec [POD] -- [COMMAND] instead.
> sh-4.2# telnet localhost 9100
> Trying 127.0.0.1...
> Connected to localhost.
> Escape character is '^]'.
> J
> 5.7.36@1->t-::kmLEXU'7Lmysql_native_password
> ```
Yes, I also have tested locally, there is no issue from the APISIX side
while proxying the request to the MySQL server with the correct route
configuration.
> but when I logout the pod and tried to request with the node port, the
connection was closed
>
> ```
> @MacBook-Pro ~ % telnet localhost 30100
> Trying ::1...
> Connected to localhost.
> Escape character is '^]'.
> Connection closed by foreign host.
> ```
Got your point.
>"value": {
"create_time": 1639291584,
"id": "1",
"server_addr": "10.1.2.177",
"server_port": 9100,
"update_time": 1639291584,
"upstream": {
"hash_on": "vars",
"nodes": {
"10.1.2.171:3306": 1
},
"pass_host": "pass",
"scheme": "http",
"type": "roundrobin"
}
Just curious why do you have `server_addr` filter enabled for port
`10.1.2.177` while from the kubectl I can see your APISIX server is running on
`10.1.2.186`. (see
[here](https://nginx.org/en/docs/http/ngx_http_core_module.html#var_server_addr))
Could you please check if you can access the mysql through the k8s nodeport
for this stream route config:
```json
{
"upstream": {
"nodes": {
"10.1.2.171:3306": 1
},
"type": "roundrobin"
}
}
```
assuming you can access the mysql at `10.1.2.171:3306`. Thanks.
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] tzssangglass closed issue #5749: bug: apisix reload Cannot clear uninitialized plug-ins
tzssangglass closed issue #5749: URL: https://github.com/apache/apisix/issues/5749 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on a change in pull request #5745: feat(vault): vault lua module, integration with jwt-auth authentication plugin
spacewander commented on a change in pull request #5745:
URL: https://github.com/apache/apisix/pull/5745#discussion_r767416526
##
File path: t/plugin/jwt-auth-vault.t
##
@@ -0,0 +1,381 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements. See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License. You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+use t::APISIX 'no_plan';
+
+repeat_each(1);
+no_long_string();
+no_root_location();
+no_shuffle();
+
+add_block_preprocessor(sub {
+my ($block) = @_;
+
+my $http_config = $block->http_config // <<_EOC_;
+
+server {
+listen 8777;
+
+location /secure-endpoint {
+content_by_lua_block {
+ngx.say("successfully invoked secure endpoint")
+}
+}
+}
+_EOC_
+
+$block->set_value("http_config", $http_config);
+
+if (!$block->request) {
+$block->set_value("request", "GET /t");
+}
+if (!$block->no_error_log && !$block->error_log) {
+$block->set_value("no_error_log", "[error]\n[alert]");
+}
+});
+
+run_tests;
+
+__DATA__
+
+=== TEST 1: schema check
+--- config
+location /t {
+content_by_lua_block {
+local plugin = require("apisix.plugins.jwt-auth")
+local core = require("apisix.core")
+for _, conf in ipairs({
+{
+-- public and private key are not provided for RS256,
returns error
+key = "key-1",
+algorithm = "RS256"
+},
+{
+-- public and private key are not provided but vault
config is enabled.
+key = "key-1",
+algorithm = "RS256",
+vault = {}
+}
+}) do
+local ok, err = plugin.check_schema(conf,
core.schema.TYPE_CONSUMER)
+if not ok then
+ngx.say(err)
+else
+ngx.say("ok")
+end
+end
+}
+}
+--- response_body
+failed to validate dependent schema for "algorithm": value should match only
one schema, but matches none
+ok
+
+
+
+=== TEST 2: create a consumer with plugin and username
+--- config
+location /t {
+content_by_lua_block {
+local t = require("lib.test_admin").test
+local code, body = t('/apisix/admin/consumers',
+ngx.HTTP_PUT,
+[[{
+"username": "jack",
+"plugins": {
+"jwt-auth": {
+"key": "key-hs256",
+"algorithm": "HS256",
+"vault":{}
+}
+}
+}]]
+)
+
+if code >= 300 then
+ngx.status = code
+end
+ngx.say(body)
+}
+}
+--- response_body
+passed
+
+
+
+=== TEST 3: enable jwt auth plugin using admin api
+--- config
+location /t {
+content_by_lua_block {
+local t = require("lib.test_admin").test
+local code, body = t('/apisix/admin/routes/1',
+ngx.HTTP_PUT,
+[[{
+"plugins": {
+"jwt-auth": {}
+},
+"upstream": {
+"nodes": {
+"127.0.0.1:8777": 1
+},
+"type": "roundrobin"
+},
+"uri": "/secure-endpoint"
+}]]
+)
+
+if code >= 300 then
+ngx.status = code
+end
+ngx.say(body)
+}
+}
+--- response_body
+passed
+
+
+
+=== TEST 4: sign a jwt and access/verify /secure-endpoint, fails as no secret
entry into vault
+--- extra_yaml_config
+vault:
+ host: "http://0.0.0.0:8200;
+ timeout: 10
+ prefix: kv/apisix
+ token: root
+#END
+--- config
+location /t {
+content_by_lua_block {
+local t = require("lib.test_admin").test
+local code, err, sign = t('/apisix/plugin/jwt/sign?key=key-hs256',
+ngx.HTTP_GET
+)
+
+if code > 200 then
+
[apisix] branch dependabot/github_actions/actions/stale-4.1.0 created (now 59eb89a)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/github_actions/actions/stale-4.1.0 in repository https://gitbox.apache.org/repos/asf/apisix.git. at 59eb89a chore(deps): bump actions/stale from 4.0.0 to 4.1.0 No new revisions were added by this update.
[GitHub] [apisix] dependabot[bot] opened a new pull request #5781: chore(deps): bump actions/stale from 4.0.0 to 4.1.0
dependabot[bot] opened a new pull request #5781: URL: https://github.com/apache/apisix/pull/5781 Bumps [actions/stale](https://github.com/actions/stale) from 4.0.0 to 4.1.0. Release notes Sourced from https://github.com/actions/stale/releases;>actions/stale's releases. v4.1.0 Features https://github.com/actions/stale/commit/9912fa74d1c01b5d6187793d97441019cbe325d0;>Ability to exempt draft PRs Changelog Sourced from https://github.com/actions/stale/blob/main/CHANGELOG.md;>actions/stale's changelog. https://github.com/actions/stale/compare/v3.0.19...v4.1.0;>4.1.0 (2021-07-14) Features https://github.com/actions/stale/commit/9912fa74d1c01b5d6187793d97441019cbe325d0;>Ability to exempt draft PRs Commits https://github.com/actions/stale/commit/7fb802b3079a276cf3c7e6ba9aa003c665b3f838;>7fb802b Bump release to 4.1.0 https://github.com/actions/stale/commit/54197c7137c7221265ea1097805e48ab88497b94;>54197c7 Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/602;>#602 from actions/revert-breaking-change https://github.com/actions/stale/commit/3a971aeb80eea6ad458e4732610b5f50f2dd4708;>3a971ae Revert Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/586;>#586 from C0ZEN/feature/split-pr-and-issue-options https://github.com/actions/stale/commit/fc4a5ff9425599c9a0575c9320984977b05c2e9e;>fc4a5ff Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/595;>#595 from actions/dependabot/npm_and_yarn/types/jest-27.0.2 https://github.com/actions/stale/commit/db699ab3b1799e48475827e8e83083088927fd54;>db699ab Merge pull request https://github-redirect.dependabot.com/actions/stale/issues/586;>#586 from C0ZEN/feature/split-pr-and-issue-options https://github.com/actions/stale/commit/f8e08de81bfdd55cf8e28a406355757430f05b5f;>f8e08de build(deps-dev): bump @types/jest from 27.0.1 to 27.0.2 https://github.com/actions/stale/commit/b83d488cb987fd3a570b32d75c9f04a7563b7979;>b83d488 Update dependencies (https://github-redirect.dependabot.com/actions/stale/issues/592;>#592) https://github.com/actions/stale/commit/1ff6cd74cb1c5a5c87c8950202e80ce0f2b2a979;>1ff6cd7 build(deps): bump tmpl from 1.0.4 to 1.0.5 (https://github-redirect.dependabot.com/actions/stale/issues/575;>#575) https://github.com/actions/stale/commit/86fed0e1f119e59b0690309530b9e60717977ea2;>86fed0e build(deps-dev): bump jest-silent-reporter from 0.4.0 to 0.5.0 (https://github-redirect.dependabot.com/actions/stale/issues/573;>#573) https://github.com/actions/stale/commit/67004407a6a06b621969a09f818cc9b477f0d733;>6700440 docs(readme): split the options between common, issues and prs Additional commits viewable in https://github.com/actions/stale/compare/v4.0.0...v4.1.0;>compare view [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- Dependabot commands and options You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) -- 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.apache.org For queries about this
[GitHub] [apisix] gxthrj opened a new issue #5780: DISCUSS: Add a plugin that filters specific requests through regular matching
gxthrj opened a new issue #5780:
URL: https://github.com/apache/apisix/issues/5780
### Issue description
Thinking of the recent [security vulnerabilities in Apache
log4j2](http://cve.mitre.org/cgi-bin/cvename.cgi?name=2021-44228), We can
provide such a universal plug-in for APISIX to filter out some illegal requests
through some regularities, although this is not the ultimate solution to the
vulnerability.
The fastest solution is to use the serverless plug-in to quickly define a
filtering logic.
But we know that the serverless plug-in is not recommended for production
use, and can only be turned on temporarily.
Therefore, I propose to create a plug-in for regular matching to filter
illegal requests. Everyone is welcome to participate in the design and
development.
Note: This plug-in is not intended as a solution to security problems, but
as a general-purpose plug-in that filters out specified requests by configuring
regular matching conditions.
The following is a reference design, welcome to discuss.
Plug-in name: reg_blocker
Plug-in configuration examples, which can be continuously expanded:
```json
{
"block_rules": [
{
"key": "$http_uri",
"rules":
["(?:\$|\%24)(?:\{|\%7b)[^\w]*?j[^\w]*?n[^\w]*?d[^\w]*?i[^\w]*?(?:\:|\%3a)"]
},
{
"key": "$args_name",
"rules": ["root.exe", "root.m+"]
}
],
"reject_code": 403,
"message": "Illegal request"
}
```
### Environment
- apisix version (cmd: `apisix version`): master
- OS (cmd: `uname -a`): linux
- OpenResty / Nginx version (cmd: `nginx -V` or `openresty -V`): 1.19.*
- etcd version, if have (cmd: run `curl
http://127.0.0.1:9090/v1/server_info` to get the info from server-info API):
- apisix-dashboard version, if have:
- the plugin runner version, if the issue is about a plugin runner (cmd:
depended on the kind of runner):
- luarocks version, if the issue is about installation (cmd: `luarocks
--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 the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] spacewander commented on issue #4812: increase the open-policy-agent plug-in support
spacewander commented on issue #4812: URL: https://github.com/apache/apisix/issues/4812#issuecomment-992106357 Already done. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander closed issue #4812: increase the open-policy-agent plug-in support
spacewander closed issue #4812: URL: https://github.com/apache/apisix/issues/4812 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bzp2010 edited a comment on issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 edited a comment on issue #5714:
URL: https://github.com/apache/apisix/issues/5714#issuecomment-992074159
Hi, @spacewander.
I noticed in the OPA documentation the Envoy related use case, which
contains the input content of the OPA filter to the OPA service in the
following format. Is it necessary to adjust our current input to make it
somewhat compatible with the OPA policy that was written for Envoy filter? This
will help users to try our software more easily.
```json
{
"attributes": {
"source": {
"address": {
"socketAddress": {
"address": "172.17.0.1",
"portValue": 61402
}
}
},
"destination": {
"address": {
"socketAddress": {
"address": "172.17.06",
"portValue": 8000
}
}
},
"request": {
"time": "2020-11-20T09:47:47.722473Z",
"http": {
"id": "13519049518330544501",
"method": "POST",
"headers": {
":authority": "192.168.99.206:30164",
":method": "POST",
":path": "/people?lang=en",
"accept": "*/*",
"authorization": "Bearer
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJyb2xlIjoiYWRtaW4iLCJzdWIiOiJZbTlpIiwibmJmIjoxNTE0ODUxMTM5LCJleHAiOjE2NDEwODE1Mzl9.WCxNAveAVAdRCmkpIObOTaSd0AJRECY2Ch2Qdic3kU8",
"content-length": "41",
"content-type": "application/json",
"user-agent": "curl/7.54.0",
"x-forwarded-proto": "http",
"x-request-id": "7bca5c86-bf55-432c-b212-8c0f1dc999ec"
},
"host": "192.168.99.206:30164",
"path": "/people?lang=en",
"protocol": "HTTP/1.1",
"body": "{\"firstname\":\"Charlie\", \"lastname\":\"Opa\"}",
"size": 41
}
},
"metadataContext": {}
},
"parsed_body": {
"firstname": "Charlie",
"lastname": "Opa"
},
"parsed_path": [
"people"
],
"parsed_query": {
"lang": [
"en"
]
},
"truncated_body": false,
"version": {
"encoding": "protojson",
"ext_authz": "v3"
}
}
```
Reference:
[https://www.openpolicyagent.org/docs/latest/envoy-primer/](https://www.openpolicyagent.org/docs/latest/envoy-primer/)
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] bisakhmondal commented on a change in pull request #5745: feat(vault): vault lua module, integration with jwt-auth authentication plugin
bisakhmondal commented on a change in pull request #5745: URL: https://github.com/apache/apisix/pull/5745#discussion_r767399928 ## File path: t/plugin/jwt-auth-vault.t ## @@ -163,6 +148,13 @@ passed === TEST 4: sign a jwt and access/verify /secure-endpoint, fails as no secret entry into vault +--- yaml_config Review comment: Resolved -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] bzp2010 commented on issue #2146: support set scope as Path in Advanced Routing Matching Conditions
bzp2010 commented on issue #2146: URL: https://github.com/apache/apisix-dashboard/issues/2146#issuecomment-992087406 It should work fine on the current latest version. So I will close this issue. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] bzp2010 closed issue #2146: support set scope as Path in Advanced Routing Matching Conditions
bzp2010 closed issue #2146: URL: https://github.com/apache/apisix-dashboard/issues/2146 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-go-plugin-runner] GhangZh edited a comment on issue #53: request help: support http proxy
GhangZh edited a comment on issue #53:
URL:
https://github.com/apache/apisix-go-plugin-runner/issues/53#issuecomment-992085667
I wrote a demo, but I found that the response body is empty ,For example, if
I add this plugin to luacode.com, and I request a.com, the plugin will request
b.com and do the processing, now the processing is successful but there is no
response content from luacode.com.
```bash
package plugins
import (
"encoding/json"
"fmt"
pkgHTTP "github.com/apache/apisix-go-plugin-runner/pkg/http"
"github.com/apache/apisix-go-plugin-runner/pkg/log"
"github.com/apache/apisix-go-plugin-runner/pkg/plugin"
"github.com/sirupsen/logrus"
"net/http"
"time"
)
func init() {
err := plugin.RegisterPlugin({})
if err != nil {
log.Fatalf("failed to register plugin ForwardAuth: %s", err)
}
}
// ForwardAuth is a demo to show how to return data directly instead of
proxying
// it to the upstream.
type ForwardAuth struct {
}
type ForwardAuthConf struct {
Body string `json:"body"`
}
func (p *ForwardAuth) Name() string {
return "forward-auth"
}
func (p *ForwardAuth) ParseConf(in []byte) (interface{}, error) {
conf := ForwardAuthConf{}
err := json.Unmarshal(in, )
return conf, err
}
func (p *ForwardAuth) Filter(conf interface{}, w http.ResponseWriter, r
pkgHTTP.Request) {
req, err := http.NewRequest("GET", "http://b.com;, nil)
if err != nil {
logrus.Errorf("send request failed err:%v", err)
return
}
//req.Header.Set("Cookie", r.Header().Get("Cookie"))
client := http.Client{
Timeout: 5 * time.Second,
}
resp, err := client.Do(req)
if err != nil {
logrus.Errorf("get response failed err:%v", err)
return
}
if resp == nil {
logrus.Errorf("response is nil")
return
}
defer resp.Body.Close()
if len(resp.Header) > 0 {
for k, v := range resp.Header {
if v != nil {
w.Header().Set(k, v[0])
}
}
}
w.Header().Add("X-Resp-A6-Runner", "Go")
return
}
```
not use plugin
after use pulgin
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-go-plugin-runner] GhangZh edited a comment on issue #53: request help: support http proxy
GhangZh edited a comment on issue #53:
URL:
https://github.com/apache/apisix-go-plugin-runner/issues/53#issuecomment-992085667
I wrote a demo, but I found that the response body is empty ,For example, if
I add this plugin to a.com, and I request a.com, the plugin will request b.com
and do the processing, now the processing is successful but there is no
response content from a.com.
```bash
package plugins
import (
"encoding/json"
"fmt"
pkgHTTP "github.com/apache/apisix-go-plugin-runner/pkg/http"
"github.com/apache/apisix-go-plugin-runner/pkg/log"
"github.com/apache/apisix-go-plugin-runner/pkg/plugin"
"github.com/sirupsen/logrus"
"net/http"
"time"
)
func init() {
err := plugin.RegisterPlugin({})
if err != nil {
log.Fatalf("failed to register plugin ForwardAuth: %s", err)
}
}
// ForwardAuth is a demo to show how to return data directly instead of
proxying
// it to the upstream.
type ForwardAuth struct {
}
type ForwardAuthConf struct {
Body string `json:"body"`
}
func (p *ForwardAuth) Name() string {
return "forward-auth"
}
func (p *ForwardAuth) ParseConf(in []byte) (interface{}, error) {
conf := ForwardAuthConf{}
err := json.Unmarshal(in, )
return conf, err
}
func (p *ForwardAuth) Filter(conf interface{}, w http.ResponseWriter, r
pkgHTTP.Request) {
req, err := http.NewRequest("GET", "http://b.com;, nil)
if err != nil {
logrus.Errorf("send request failed err:%v", err)
return
}
//req.Header.Set("Cookie", r.Header().Get("Cookie"))
client := http.Client{
Timeout: 5 * time.Second,
}
resp, err := client.Do(req)
if err != nil {
logrus.Errorf("get response failed err:%v", err)
return
}
if resp == nil {
logrus.Errorf("response is nil")
return
}
defer resp.Body.Close()
if len(resp.Header) > 0 {
for k, v := range resp.Header {
if v != nil {
w.Header().Set(k, v[0])
}
}
}
w.Header().Add("X-Resp-A6-Runner", "Go")
return
}
```
not use plugin
after use pulgin
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-go-plugin-runner] GhangZh edited a comment on issue #53: request help: support http proxy
GhangZh edited a comment on issue #53:
URL:
https://github.com/apache/apisix-go-plugin-runner/issues/53#issuecomment-992085667
I wrote a demo, but I found that the response body is empty ,For example, if
I add this plugin to a.com, and I request a.com, the plugin will request b.com
and do the processing, now the processing is successful but there is no
response content from a.com.
```bash
package plugins
import (
"encoding/json"
"fmt"
pkgHTTP "github.com/apache/apisix-go-plugin-runner/pkg/http"
"github.com/apache/apisix-go-plugin-runner/pkg/log"
"github.com/apache/apisix-go-plugin-runner/pkg/plugin"
"github.com/sirupsen/logrus"
"net/http"
"time"
)
func init() {
err := plugin.RegisterPlugin({})
if err != nil {
log.Fatalf("failed to register plugin ForwardAuth: %s", err)
}
}
// ForwardAuth is a demo to show how to return data directly instead of
proxying
// it to the upstream.
type ForwardAuth struct {
}
type ForwardAuthConf struct {
Body string `json:"body"`
}
func (p *ForwardAuth) Name() string {
return "forward-auth"
}
func (p *ForwardAuth) ParseConf(in []byte) (interface{}, error) {
conf := ForwardAuthConf{}
err := json.Unmarshal(in, )
return conf, err
}
func (p *ForwardAuth) Filter(conf interface{}, w http.ResponseWriter, r
pkgHTTP.Request) {
req, err := http.NewRequest("GET", "http://xxx;, nil)
if err != nil {
logrus.Errorf("send request failed err:%v", err)
return
}
//req.Header.Set("Cookie", r.Header().Get("Cookie"))
client := http.Client{
Timeout: 5 * time.Second,
}
resp, err := client.Do(req)
if err != nil {
logrus.Errorf("get response failed err:%v", err)
return
}
if resp == nil {
logrus.Errorf("response is nil")
return
}
defer resp.Body.Close()
if len(resp.Header) > 0 {
for k, v := range resp.Header {
if v != nil {
w.Header().Set(k, v[0])
}
}
}
w.Header().Add("X-Resp-A6-Runner", "Go")
return
}
```
not use plugin
after use pulgin
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-go-plugin-runner] GhangZh commented on issue #53: request help: support http proxy
GhangZh commented on issue #53:
URL:
https://github.com/apache/apisix-go-plugin-runner/issues/53#issuecomment-992085667
I wrote a demo, but I found that the response body is empty ,For example, if
I add this plugin to a.com, and I request a.com, the plugin will request b.com
and do the processing, now the processing is successful but there is no
response content from a.com.
```bash
package plugins
import (
"encoding/json"
"fmt"
pkgHTTP "github.com/apache/apisix-go-plugin-runner/pkg/http"
"github.com/apache/apisix-go-plugin-runner/pkg/log"
"github.com/apache/apisix-go-plugin-runner/pkg/plugin"
"github.com/sirupsen/logrus"
"net/http"
"time"
)
func init() {
err := plugin.RegisterPlugin({})
if err != nil {
log.Fatalf("failed to register plugin ForwardAuth: %s", err)
}
}
// ForwardAuth is a demo to show how to return data directly instead of
proxying
// it to the upstream.
type ForwardAuth struct {
}
type ForwardAuthConf struct {
Body string `json:"body"`
}
func (p *ForwardAuth) Name() string {
return "forward-auth"
}
func (p *ForwardAuth) ParseConf(in []byte) (interface{}, error) {
conf := ForwardAuthConf{}
err := json.Unmarshal(in, )
return conf, err
}
func (p *ForwardAuth) Filter(conf interface{}, w http.ResponseWriter, r
pkgHTTP.Request) {
req, err := http.NewRequest("GET", "http://xxx;, nil)
if err != nil {
logrus.Errorf("send request failed err:%v", err)
return
}
//req.Header.Set("Cookie", r.Header().Get("Cookie"))
client := http.Client{
Timeout: 5 * time.Second,
}
resp, err := client.Do(req)
if err != nil {
logrus.Errorf("get response failed err:%v", err)
return
}
if resp == nil {
logrus.Errorf("response is nil")
return
}
defer resp.Body.Close()
if len(resp.Header) > 0 {
for k, v := range resp.Header {
if v != nil {
w.Header().Set(k, v[0])
}
}
}
w.Header().Add("X-Resp-A6-Runner", "Go")
return
}
```
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] wujie1993 removed a comment on issue #5440: request help: the connection established by the Websocket proxy client will be automatically disconnected by the server in about 5 seconds
wujie1993 removed a comment on issue #5440: URL: https://github.com/apache/apisix/issues/5440#issuecomment-992079104 Any solutions? I have the same issue -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] wujie1993 commented on issue #5440: request help: the connection established by the Websocket proxy client will be automatically disconnected by the server in about 5 seconds
wujie1993 commented on issue #5440: URL: https://github.com/apache/apisix/issues/5440#issuecomment-992079104 Any solutions? I have the same issue -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] Jon-Gaara commented on issue #5749: bug: apisix reload Cannot clear uninitialized plug-ins
Jon-Gaara commented on issue #5749: URL: https://github.com/apache/apisix/issues/5749#issuecomment-992075913 ok,i know -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bzp2010 edited a comment on issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 edited a comment on issue #5714:
URL: https://github.com/apache/apisix/issues/5714#issuecomment-992074159
Hi, @spacewander.
I noticed in the OPA documentation the Envoy related use case, which
contains the input content of the OPA filter to the OPA service in the
following format. Is it necessary to adjust our current input to make it
somewhat compatible with the OPA policy that was written for Envoy filter? This
will help users to try our software more easily.
```json
{
"attributes": {
"source": {
"address": {
"socketAddress": {
"address": "172.17.0.1",
"portValue": 61402
}
}
},
"destination": {
"address": {
"socketAddress": {
"address": "172.17.06",
"portValue": 8000
}
}
},
"request": {
"time": "2020-11-20T09:47:47.722473Z",
"http": {
"id": "13519049518330544501",
"method": "POST",
"headers": {
":authority": "192.168.99.206:30164",
":method": "POST",
":path": "/people?lang=en",
"accept": "*/*",
"authorization": "Bearer
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJyb2xlIjoiYWRtaW4iLCJzdWIiOiJZbTlpIiwibmJmIjoxNTE0ODUxMTM5LCJleHAiOjE2NDEwODE1Mzl9.WCxNAveAVAdRCmkpIObOTaSd0AJRECY2Ch2Qdic3kU8",
"content-length": "41",
"content-type": "application/json",
"user-agent": "curl/7.54.0",
"x-forwarded-proto": "http",
"x-request-id": "7bca5c86-bf55-432c-b212-8c0f1dc999ec"
},
"host": "192.168.99.206:30164",
"path": "/people?lang=en",
"protocol": "HTTP/1.1",
"body": "{\"firstname\":\"Charlie\", \"lastname\":\"Opa\"}",
"size": 41
}
},
"metadataContext": {}
},
"parsed_body": {
"firstname": "Charlie",
"lastname": "Opa"
},
"parsed_path": [
"people"
],
"parsed_query": {
"lang": [
"en"
]
},
"truncated_body": false,
"version": {
"encoding": "protojson",
"ext_authz": "v3"
}
}
```
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] bzp2010 commented on issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 commented on issue #5714:
URL: https://github.com/apache/apisix/issues/5714#issuecomment-992074159
Hi, @spacewander.
I noticed in the OPA documentation the Envoy related use case, which
contains the input content of the OPA filter to the OPA service in the
following format. Is it necessary to adjust our current input to make it
somewhat compatible with the OPA policy that was written for Envoy filter?
```json
{
"attributes": {
"source": {
"address": {
"socketAddress": {
"address": "172.17.0.1",
"portValue": 61402
}
}
},
"destination": {
"address": {
"socketAddress": {
"address": "172.17.06",
"portValue": 8000
}
}
},
"request": {
"time": "2020-11-20T09:47:47.722473Z",
"http": {
"id": "13519049518330544501",
"method": "POST",
"headers": {
":authority": "192.168.99.206:30164",
":method": "POST",
":path": "/people?lang=en",
"accept": "*/*",
"authorization": "Bearer
eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJyb2xlIjoiYWRtaW4iLCJzdWIiOiJZbTlpIiwibmJmIjoxNTE0ODUxMTM5LCJleHAiOjE2NDEwODE1Mzl9.WCxNAveAVAdRCmkpIObOTaSd0AJRECY2Ch2Qdic3kU8",
"content-length": "41",
"content-type": "application/json",
"user-agent": "curl/7.54.0",
"x-forwarded-proto": "http",
"x-request-id": "7bca5c86-bf55-432c-b212-8c0f1dc999ec"
},
"host": "192.168.99.206:30164",
"path": "/people?lang=en",
"protocol": "HTTP/1.1",
"body": "{\"firstname\":\"Charlie\", \"lastname\":\"Opa\"}",
"size": 41
}
},
"metadataContext": {}
},
"parsed_body": {
"firstname": "Charlie",
"lastname": "Opa"
},
"parsed_path": [
"people"
],
"parsed_query": {
"lang": [
"en"
]
},
"truncated_body": false,
"version": {
"encoding": "protojson",
"ext_authz": "v3"
}
}
```
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-java-plugin-runner] tzssangglass commented on a change in pull request #95: docs: add zh Quick Start doc
tzssangglass commented on a change in pull request #95:
URL:
https://github.com/apache/apisix-java-plugin-runner/pull/95#discussion_r767388124
##
File path: docs/zh/Quick Start.md
##
@@ -0,0 +1,110 @@
+### Quick Start
+
+ 一、准备工作
+
+* JDK 8
+
+* Clone the
[apisix-java-plugin-runner](https://github.com/apache/apisix-java-plugin-runner)
project.
+
+ 二、开发扩展插件过滤器
+
+在`runner-plugin` 模块 `org.apache.apisix.plugin.runner.filter`
包下编写过滤器处理请求，过滤器要实现`PluginFilter` 接口，可以参考 `apisix-runner-sample`
模块下的样例，官方提供了两个样例还是很全面的，一个是请求重写[RewriteRequestDemoFilter](https://github.com/apache/apisix-java-plugin-runner/blob/main/sample/src/main/java/org/apache/apisix/plugin/runner/filter/RewriteRequestDemoFilter.java)，一个是请求拦截[StopRequestDemoFilter](https://github.com/apache/apisix-java-plugin-runner/blob/main/sample/src/main/java/org/apache/apisix/plugin/runner/filter/StopRequestDemoFilter.java)。
+
+```java
+@Component
+public class CheckTokenFilter implements PluginFilter {
+@Override
+public String name() {
+return "CheckTokenFilter";
+}
+
+@Override
+public Mono filter(HttpRequest request, HttpResponse response,
PluginFilterChain chain) {
+/*
+ * todo your business here
+ */
+
+
+return chain.filter(request, response);
+}
+}
+```
+
+ 三、部署
+
+插件写好后怎么部署是关键，apisix-java-plugin-runner 与 APISIX 用 Unix Domain Socket 进行进程内通讯，
+所以他们要部署在一个服务实例，并且APISIX启动的过程中会带着apisix-java-plugin-runner一起启动，如果是容器化部署就必须在一个容器里运行。
+
+所以如果是容器部署就需要把apisix-java-plugin-runner 与 APISIX 生成在一个docker image里。
Review comment:
@cws1981
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix-java-plugin-runner] tzssangglass commented on a change in pull request #95: docs: add zh Quick Start doc
tzssangglass commented on a change in pull request #95:
URL:
https://github.com/apache/apisix-java-plugin-runner/pull/95#discussion_r767387952
##
File path: docs/zh/Quick Start.md
##
@@ -0,0 +1,110 @@
+### Quick Start
+
+ 一、准备工作
+
+* JDK 8
+
+* Clone the
[apisix-java-plugin-runner](https://github.com/apache/apisix-java-plugin-runner)
project.
+
+ 二、开发扩展插件过滤器
+
+在`runner-plugin` 模块 `org.apache.apisix.plugin.runner.filter`
包下编写过滤器处理请求，过滤器要实现`PluginFilter` 接口，可以参考 `apisix-runner-sample`
模块下的样例，官方提供了两个样例还是很全面的，一个是请求重写[RewriteRequestDemoFilter](https://github.com/apache/apisix-java-plugin-runner/blob/main/sample/src/main/java/org/apache/apisix/plugin/runner/filter/RewriteRequestDemoFilter.java)，一个是请求拦截[StopRequestDemoFilter](https://github.com/apache/apisix-java-plugin-runner/blob/main/sample/src/main/java/org/apache/apisix/plugin/runner/filter/StopRequestDemoFilter.java)。
+
+```java
+@Component
+public class CheckTokenFilter implements PluginFilter {
+@Override
+public String name() {
+return "CheckTokenFilter";
+}
+
+@Override
+public Mono filter(HttpRequest request, HttpResponse response,
PluginFilterChain chain) {
+/*
+ * todo your business here
+ */
+
+
+return chain.filter(request, response);
+}
+}
+```
+
+ 三、部署
+
+插件写好后怎么部署是关键，apisix-java-plugin-runner 与 APISIX 用 Unix Domain Socket 进行进程内通讯，
+所以他们要部署在一个服务实例，并且APISIX启动的过程中会带着apisix-java-plugin-runner一起启动，如果是容器化部署就必须在一个容器里运行。
+
+所以如果是容器部署就需要把apisix-java-plugin-runner 与 APISIX 生成在一个docker image里。
Review comment:
can you see this:
I suspect that github has eaten this
--
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.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
[GitHub] [apisix] bzp2010 commented on issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 commented on issue #5714: URL: https://github.com/apache/apisix/issues/5714#issuecomment-992065110 Hi, @anderseknert. The development of `stage 1: basic support` in the current roadmap has been completed and the PR is here #5734. I have some questions for you here. Does the OPA community have any use case or best practices for "response overwriting (e.g., status codes or headers) in case of authentication failure"? Thanks. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan closed issue #582: Support PR preview
juzhiyuan closed issue #582: URL: https://github.com/apache/apisix-website/issues/582 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan commented on issue #582: Support PR preview
juzhiyuan commented on issue #582: URL: https://github.com/apache/apisix-website/issues/582#issuecomment-992064469 consider closed because we use Netlify back -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] neverCase commented on pull request #792: feat: add ApisixPluginConfigs crd to v2beta3 (#746) (#638) (#791)
neverCase commented on pull request #792: URL: https://github.com/apache/apisix-ingress-controller/pull/792#issuecomment-992063623 @tao12345666333 Run e2e test cases failed again. Does this problem cause by the ingress status bug which you mentioned before？ -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bzp2010 opened a new pull request #5779: feat: supprot OPA plugin complex response
bzp2010 opened a new pull request #5779: URL: https://github.com/apache/apisix/pull/5779 ### What this PR does / why we need it: Enhance OPA plug-in functionality to support overwriting of status codes, response headers, and response bodies when validation is rejected. stage 2 of #5714 ### Pre-submission checklist: * [x] Did you explain what problem does this PR solve? Or what new features have been added? * [x] Have you added corresponding test cases? * [x] Have you modified the corresponding document? * [x] Is this PR backward compatible? **If it is not backward compatible, please discuss on the [mailing list](https://github.com/apache/apisix/tree/master#community) first** -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] yunfhu commented on issue #5775: request help: stream-proxy as mysql proxy not work
yunfhu commented on issue #5775: URL: https://github.com/apache/apisix/issues/5775#issuecomment-992060431 > @bisakhmondal > Could you check if MySQL can be proxied via APISIX? Thanks! pls read previous comments which described very clearly -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] yzeng25 commented on issue #5714: Proposal: support OpenPolicyAgent for access control
yzeng25 commented on issue #5714: URL: https://github.com/apache/apisix/issues/5714#issuecomment-992059734 Cool! Got it! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #808: style: adjsut Netlify position
netlify[bot] edited a comment on pull request #808: URL: https://github.com/apache/apisix-website/pull/808#issuecomment-992054436 ✔️ Deploy Preview for *apache-apisix* ready! Explore the source changes: 645cb4b41c80258d7423e2c7dbcb39c4667a1a35 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6b29cfc7d5d0007d6f8a1](https://app.netlify.com/sites/apache-apisix/deploys/61b6b29cfc7d5d0007d6f8a1) Browse the preview: [https://deploy-preview-808--apache-apisix.netlify.app](https://deploy-preview-808--apache-apisix.netlify.app) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] juzhiyuan commented on issue #5714: Proposal: support OpenPolicyAgent for access control
juzhiyuan commented on issue #5714: URL: https://github.com/apache/apisix/issues/5714#issuecomment-992059321 > ## Update > The current OPA basic support plugin has been completed and will begin the phase 2 of feature development on the roadmap. So I reopened this issue to track the status of the next PR. cc @yzeng25 to know this. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] yzeng25 commented on pull request #722: refactor: update img source blog/2020
yzeng25 commented on pull request #722: URL: https://github.com/apache/apisix-website/pull/722#issuecomment-992059284 > Hi, I'm going to close this PR because of too many conflicts ok -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan commented on pull request #722: refactor: update img source blog/2020
juzhiyuan commented on pull request #722: URL: https://github.com/apache/apisix-website/pull/722#issuecomment-992058803 Hi, I'm going to close this PR because of too many conflicts -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan closed pull request #722: refactor: update img source blog/2020
juzhiyuan closed pull request #722: URL: https://github.com/apache/apisix-website/pull/722 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan merged pull request #808: style: adjsut Netlify position
juzhiyuan merged pull request #808: URL: https://github.com/apache/apisix-website/pull/808 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: style: adjsut Netlify position (#808)
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-website.git
The following commit(s) were added to refs/heads/master by this push:
new 796151e style: adjsut Netlify position (#808)
796151e is described below
commit 796151e591fcbb32740f7678e83628a07ee5718c
Author: 琚致远
AuthorDate: Mon Dec 13 10:44:06 2021 +0800
style: adjsut Netlify position (#808)
---
website/config/ssrTemplate.js | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/website/config/ssrTemplate.js b/website/config/ssrTemplate.js
index e7e2ea8..b3c1361 100644
--- a/website/config/ssrTemplate.js
+++ b/website/config/ssrTemplate.js
@@ -34,8 +34,8 @@ module.exports = {
window.onload = function() {
if (window.location.host.endsWith(".netlify.app")) {
- var footerDOM = document.querySelector('footer');
- footerDOM.insertAdjacentHTML('afterend', 'This site is powered by https://www.netlify.com/"; target="_blank">Netlify');
+ var dom = document.querySelector("#__docusaurus");
+ dom.insertAdjacentHTML('afterend', 'This site is powered by https://www.netlify.com/"; target="_blank">Netlify');
}
}
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #808: style: adjsut Netlify position
netlify[bot] edited a comment on pull request #808: URL: https://github.com/apache/apisix-website/pull/808#issuecomment-992054436 Deploy Preview for *apache-apisix* processing. Explore the source changes: 645cb4b41c80258d7423e2c7dbcb39c4667a1a35 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6b29cfc7d5d0007d6f8a1](https://app.netlify.com/sites/apache-apisix/deploys/61b6b29cfc7d5d0007d6f8a1) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch style-netlify updated (8fd48a8 -> 645cb4b)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch style-netlify in repository https://gitbox.apache.org/repos/asf/apisix-website.git. from 8fd48a8 style: adjsut Netlify position add 645cb4b Update ssrTemplate.js No new revisions were added by this update. Summary of changes: website/config/ssrTemplate.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #808: style: adjsut Netlify position
netlify[bot] edited a comment on pull request #808: URL: https://github.com/apache/apisix-website/pull/808#issuecomment-992054436 ✔️ Deploy Preview for *apache-apisix* ready! Explore the source changes: 8fd48a84bcaa0c93b4dedea40ef7e474bb30d250 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6b068566ce80008710f97](https://app.netlify.com/sites/apache-apisix/deploys/61b6b068566ce80008710f97) Browse the preview: [https://deploy-preview-808--apache-apisix.netlify.app](https://deploy-preview-808--apache-apisix.netlify.app) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] commented on pull request #808: style: adjsut Netlify position
netlify[bot] commented on pull request #808: URL: https://github.com/apache/apisix-website/pull/808#issuecomment-992054436 Deploy Preview for *apache-apisix* processing. Explore the source changes: 8fd48a84bcaa0c93b4dedea40ef7e474bb30d250 Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6b068566ce80008710f97](https://app.netlify.com/sites/apache-apisix/deploys/61b6b068566ce80008710f97) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan opened a new pull request #808: style: adjsut Netlify position
juzhiyuan opened a new pull request #808: URL: https://github.com/apache/apisix-website/pull/808 **Changes:** This PR aims to insert Netlify's dom after Docusaurus's main dom. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch style-netlify created (now 8fd48a8)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch style-netlify in repository https://gitbox.apache.org/repos/asf/apisix-website.git. at 8fd48a8 style: adjsut Netlify position This branch includes the following new commits: new 8fd48a8 style: adjsut Netlify position The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference.
[apisix-website] 01/01: style: adjsut Netlify position
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch style-netlify
in repository https://gitbox.apache.org/repos/asf/apisix-website.git
commit 8fd48a84bcaa0c93b4dedea40ef7e474bb30d250
Author: juzhiyuan
AuthorDate: Mon Dec 13 10:30:00 2021 +0800
style: adjsut Netlify position
---
website/config/ssrTemplate.js | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/website/config/ssrTemplate.js b/website/config/ssrTemplate.js
index e7e2ea8..1cd4bb3 100644
--- a/website/config/ssrTemplate.js
+++ b/website/config/ssrTemplate.js
@@ -34,8 +34,8 @@ module.exports = {
window.onload = function() {
if (window.location.host.endsWith(".netlify.app")) {
- var footerDOM = document.querySelector('footer');
- footerDOM.insertAdjacentHTML('afterend', 'This site is powered by https://www.netlify.com/"; target="_blank">Netlify');
+ var dom = document.querySelector("#__docusaurus");
+ dom.insertAdjacentHTML('afterend', 'This site is powered by https://www.netlify.com/"; target="_blank">Netlify');
}
}
[GitHub] [apisix-java-plugin-runner] tzssangglass merged pull request #98: chore: upgrade log4j2 version to 2.15.0
tzssangglass merged pull request #98: URL: https://github.com/apache/apisix-java-plugin-runner/pull/98 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-java-plugin-runner] branch main updated: chore: upgrade log4j2 version to 2.15.0 (#98)
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 8b89e33 chore: upgrade log4j2 version to 2.15.0 (#98)
8b89e33 is described below
commit 8b89e339b991ac571bcc6242c0bc468bc4e5a2d9
Author: Daming
AuthorDate: Mon Dec 13 10:26:38 2021 +0800
chore: upgrade log4j2 version to 2.15.0 (#98)
---
pom.xml | 7 +++
1 file changed, 7 insertions(+)
diff --git a/pom.xml b/pom.xml
index ffd085c..5178f67 100644
--- a/pom.xml
+++ b/pom.xml
@@ -75,6 +75,13 @@
+org.apache.logging.log4j
+log4j-bom
+2.15.0
+pom
+import
+
+
org.springframework.boot
spring-boot-dependencies
${spring-boot.version}
[GitHub] [apisix] bzp2010 commented on issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 commented on issue #5714: URL: https://github.com/apache/apisix/issues/5714#issuecomment-992052188 ## Update The current OPA basic support plugin has been completed and will begin the phase 2 of feature development on the roadmap. So I reopened this issue to track the status of the next PR. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-helm-chart] tokers closed issue #189: docs: add READEME.md to artifacthub.io
tokers closed issue #189: URL: https://github.com/apache/apisix-helm-chart/issues/189 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-helm-chart] tokers merged pull request #193: docs: add README.md for apisix-dashboard
tokers merged pull request #193: URL: https://github.com/apache/apisix-helm-chart/pull/193 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-helm-chart] branch master updated: docs: add README.md for apisix-dashboard (#193)
This is an automated email from the ASF dual-hosted git repository. tokers pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new 650fcda docs: add README.md for apisix-dashboard (#193) 650fcda is described below commit 650fcda7c26dc73d0b4eb0e8d086975c8043c52a Author: Yu.Bozhong AuthorDate: Mon Dec 13 10:16:37 2021 +0800 docs: add README.md for apisix-dashboard (#193) --- charts/apisix-dashboard/README.md | 57 +++ 1 file changed, 57 insertions(+) diff --git a/charts/apisix-dashboard/README.md b/charts/apisix-dashboard/README.md new file mode 100644 index 000..1dedf99 --- /dev/null +++ b/charts/apisix-dashboard/README.md @@ -0,0 +1,57 @@ +# Apache APISIX Dashboard + +[APISIX Dashboard](https://github.com/apache/apisix-dashboard/) is designed to make it as easy as possible for users to operate Apache APISIX through a frontend interface. + +This chart bootstraps an apisix-dashboard deployment on a [Kubernetes](http://kubernetes.io) cluster using the [Helm](https://helm.sh) package manager. + +## Prerequisites + +APISIX Dashboard requires Kubernetes version 1.14+. + +## Get Repo Info + +```console +helm repo add apisix https://charts.apiseven.com +helm repo update +``` + +## Install Chart + +**Important:** only helm3 is supported + +```console +helm install [RELEASE_NAME] apisix/apisix-dashboard +``` + +The command deploys apisix-dashboard on the Kubernetes cluster in the default configuration. + +_See [configuration](#configuration) below._ + +_See [helm install](https://helm.sh/docs/helm/helm_install/) for command documentation._ + +## Uninstall Chart + +```console +helm uninstall [RELEASE_NAME] +``` + +This removes all the Kubernetes components associated with the chart and deletes the release. + +_See [helm uninstall](https://helm.sh/docs/helm/helm_uninstall/) for command documentation._ + +## Upgrading Chart + +```console +helm upgrade [RELEASE_NAME] [CHART] --install +``` + +_See [helm upgrade](https://helm.sh/docs/helm/helm_upgrade/) for command documentation._ + + +## Configuration + +See [Customizing the Chart Before Installing](https://helm.sh/docs/intro/using_helm/#customizing-the-chart-before-installing). To see all configurable options with detailed comments, visit the chart's [values.yaml](./values.yaml), or run these configuration commands: + +```console +helm show values apisix/apisix-dashboard +```
[GitHub] [apisix-java-plugin-runner] dmsolr opened a new pull request #98: chore: upgrade log4j2 to 2.15
dmsolr opened a new pull request #98: URL: https://github.com/apache/apisix-java-plugin-runner/pull/98 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - Related issues ___ ### Bugfix - Description - How to fix? ___ ### New feature or improvement - Describe the details and related test reports. - Source branch - Related commits and pull requests - Target branch -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] juzhiyuan commented on issue #5714: Proposal: support OpenPolicyAgent for access control
juzhiyuan commented on issue #5714: URL: https://github.com/apache/apisix/issues/5714#issuecomment-992044615 Great!! -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bzp2010 closed issue #5714: Proposal: support OpenPolicyAgent for access control
bzp2010 closed issue #5714: URL: https://github.com/apache/apisix/issues/5714 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] bzp2010 merged pull request #5734: feat: basic support OPA plugin
bzp2010 merged pull request #5734: URL: https://github.com/apache/apisix/pull/5734 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch master updated (14ad3f3 -> 1e53ccb)
This is an automated email from the ASF dual-hosted git repository.
bzp2010 pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix.git.
from 14ad3f3 chore: depend apisix-base in Centos (#5776)
add 1e53ccb feat: basic support OPA plugin (#5734)
No new revisions were added by this update.
Summary of changes:
Makefile | 3 +
apisix/core/request.lua | 17 --
apisix/plugins/{openwhisk.lua => opa.lua}| 61
apisix/plugins/opa/helper.lua| 61
ci/linux-ci-init-service.sh | 11
ci/pod/docker-compose.yml| 11
conf/config-default.yaml | 1 +
t/admin/plugins.t| 1 +
t/core/request.t | 27 +
t/{node/not-exist-service.t => plugin/opa.t} | 86 +++-
10 files changed, 198 insertions(+), 81 deletions(-)
copy apisix/plugins/{openwhisk.lua => opa.lua} (61%)
create mode 100644 apisix/plugins/opa/helper.lua
copy t/{node/not-exist-service.t => plugin/opa.t} (56%)
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #807: chore: adjust branch protection
netlify[bot] edited a comment on pull request #807: URL: https://github.com/apache/apisix-website/pull/807#issuecomment-992040075 ✔️ Deploy Preview for *apache-apisix* canceled. Explore the source changes: afc8769213579b073dc001469dde1d450b164e4e Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6a818566ce800087040e0](https://app.netlify.com/sites/apache-apisix/deploys/61b6a818566ce800087040e0) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: chore: adjust branch protection (#807)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new 94cde81 chore: adjust branch protection (#807) 94cde81 is described below commit 94cde810462bf9527d69aa4c32f53f04d59f46e6 Author: 琚致远 AuthorDate: Mon Dec 13 09:59:27 2021 +0800 chore: adjust branch protection (#807) --- .asf.yaml | 6 ++ 1 file changed, 6 insertions(+) diff --git a/.asf.yaml b/.asf.yaml index 109ccb6..e353b67 100644 --- a/.asf.yaml +++ b/.asf.yaml @@ -38,5 +38,11 @@ github: squash: true merge: false rebase: false + protected_branches: +master: + required_pull_request_reviews: +dismiss_stale_reviews: true +require_code_owner_reviews: true +required_approving_review_count: 2 publish: whoami: asf-site
[GitHub] [apisix-website] juzhiyuan merged pull request #807: chore: adjust branch protection
juzhiyuan merged pull request #807: URL: https://github.com/apache/apisix-website/pull/807 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] commented on pull request #807: chore: adjust branch protection
netlify[bot] commented on pull request #807: URL: https://github.com/apache/apisix-website/pull/807#issuecomment-992040075 Deploy Preview for *apache-apisix* processing. Explore the source changes: afc8769213579b073dc001469dde1d450b164e4e Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6a818566ce800087040e0](https://app.netlify.com/sites/apache-apisix/deploys/61b6a818566ce800087040e0) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan opened a new pull request #807: chore: adjust branch protection
juzhiyuan opened a new pull request #807: URL: https://github.com/apache/apisix-website/pull/807 **Changes:** Due to asf-site branch's protection setting, we need to adjust settings to allow force-push. see https://github.com/apache/apisix-website/runs/4500450580?check_suite_focus=true Screenshots of the change: -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on issue #5775: request help: stream-proxy as mysql proxy not work
spacewander commented on issue #5775: URL: https://github.com/apache/apisix/issues/5775#issuecomment-992039348 @bisakhmondal Could you check if MySQL can be proxied via APISIX? Thanks! -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch infra-branch-protection created (now afc8769)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch infra-branch-protection in repository https://gitbox.apache.org/repos/asf/apisix-website.git. at afc8769 chore: adjust branch protection This branch includes the following new commits: new afc8769 chore: adjust branch protection The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference.
[apisix-website] 01/01: chore: adjust branch protection
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch infra-branch-protection in repository https://gitbox.apache.org/repos/asf/apisix-website.git commit afc8769213579b073dc001469dde1d450b164e4e Author: juzhiyuan AuthorDate: Mon Dec 13 09:53:37 2021 +0800 chore: adjust branch protection --- .asf.yaml | 6 ++ 1 file changed, 6 insertions(+) diff --git a/.asf.yaml b/.asf.yaml index 109ccb6..e353b67 100644 --- a/.asf.yaml +++ b/.asf.yaml @@ -38,5 +38,11 @@ github: squash: true merge: false rebase: false + protected_branches: +master: + required_pull_request_reviews: +dismiss_stale_reviews: true +require_code_owner_reviews: true +required_approving_review_count: 2 publish: whoami: asf-site
[GitHub] [apisix] tokers commented on issue #5755: [DISCUSS]: Improving the performance of the prometheus plugin
tokers commented on issue #5755: URL: https://github.com/apache/apisix/issues/5755#issuecomment-992036611 I think we may only need to focus on some critical paths, not all of them need to be yielded. Could you generate a flamegraph? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #806: feat: show netlify tip in Preview mode
netlify[bot] edited a comment on pull request #806: URL: https://github.com/apache/apisix-website/pull/806#issuecomment-991926290 ✔️ Deploy Preview for *apache-apisix* ready! Explore the source changes: 18fe0ecabae304c91a70f6a6bad7e6bb72e28d9d Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6a355e953620007f69fd2](https://app.netlify.com/sites/apache-apisix/deploys/61b6a355e953620007f69fd2) Browse the preview: [https://deploy-preview-806--apache-apisix.netlify.app](https://deploy-preview-806--apache-apisix.netlify.app) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] zero-create commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time
zero-create commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-992035608 > @zero-create Could you give the plugin attribute setting for `server-info`? 没有配置过，都是docker部署的，默认参数，已经反馈给QQ群的联系人了，在提供的环境中，复现了这个问题 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tokers commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time
tokers commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-992034832 @zero-create Could you give the plugin attribute setting for `server-info`? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander merged pull request #5776: chore: depend apisix-base in Centos
spacewander merged pull request #5776: URL: https://github.com/apache/apisix/pull/5776 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch master updated (71c256b -> 14ad3f3)
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 71c256b feat: enable L4 stream logging (#5768) add 14ad3f3 chore: depend apisix-base in Centos (#5776) No new revisions were added by this update. Summary of changes: utils/install-dependencies.sh | 20 1 file changed, 16 insertions(+), 4 deletions(-)
[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #806: feat: show netlify tip in Preview mode
netlify[bot] edited a comment on pull request #806: URL: https://github.com/apache/apisix-website/pull/806#issuecomment-991926290 Deploy Preview for *apache-apisix* processing. Explore the source changes: 18fe0ecabae304c91a70f6a6bad7e6bb72e28d9d Inspect the deploy log: [https://app.netlify.com/sites/apache-apisix/deploys/61b6a355e953620007f69fd2](https://app.netlify.com/sites/apache-apisix/deploys/61b6a355e953620007f69fd2) -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] juzhiyuan merged pull request #806: feat: show netlify tip in Preview mode
juzhiyuan merged pull request #806: URL: https://github.com/apache/apisix-website/pull/806 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: feat: show netlify tip in Preview mode (#806)
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-website.git
The following commit(s) were added to refs/heads/master by this push:
new 2de6d60 feat: show netlify tip in Preview mode (#806)
2de6d60 is described below
commit 2de6d60beb42452ca312b9f4b02d31e2d5ff8525
Author: 琚致远
AuthorDate: Mon Dec 13 09:35:22 2021 +0800
feat: show netlify tip in Preview mode (#806)
---
website/config/ssrTemplate.js | 8
1 file changed, 8 insertions(+)
diff --git a/website/config/ssrTemplate.js b/website/config/ssrTemplate.js
index 43f5ea7..e7e2ea8 100644
--- a/website/config/ssrTemplate.js
+++ b/website/config/ssrTemplate.js
@@ -31,5 +31,13 @@ module.exports = {
<% }); %>
<%~ it.postBodyTags %>
+
+ window.onload = function() {
+if (window.location.host.endsWith(".netlify.app")) {
+ var footerDOM = document.querySelector('footer');
+ footerDOM.insertAdjacentHTML('afterend', 'This site is powered by https://www.netlify.com/"; target="_blank">Netlify');
+}
+ }
+
`
}
[apisix-website] branch feat-netlify-badge updated (ce49518 -> 18fe0ec)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch feat-netlify-badge in repository https://gitbox.apache.org/repos/asf/apisix-website.git. from ce49518 Update ssrTemplate.js add 18fe0ec Update ssrTemplate.js No new revisions were added by this update. Summary of changes: website/config/ssrTemplate.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
[GitHub] [apisix] juzhiyuan commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time
juzhiyuan commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-992031895 Not sure if it's a bug, I will reopen this one. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] juzhiyuan commented on issue #5723: CPU usage gets higher when Apache APISIX is running a long time
juzhiyuan commented on issue #5723: URL: https://github.com/apache/apisix/issues/5723#issuecomment-992031744 > > @zero-create Would you try to disable the `server-info` plugin to see what happens. > > 禁用server-info插件后，cpu恢复正常，没有升高了 Hi @tzssangglass @leslie-tsang, could you please have a check on why this happened? -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-helm-chart] juzhiyuan commented on a change in pull request #193: docs: add README.md for apisix-dashboard
juzhiyuan commented on a change in pull request #193: URL: https://github.com/apache/apisix-helm-chart/pull/193#discussion_r767364213 ## File path: charts/apisix-dashboard/README.md ## @@ -0,0 +1,57 @@ +# Apache APISIX Dashboard + +[APISIX Dashboard](https://github.com/apache/apisix-dashboard/) is designed to make it as easy as possible for users to operate Apache APISIX through a frontend interface. + +This chart bootstraps an apisix-dashboard deployment on a [Kubernetes](http://kubernetes.io) cluster using the [Helm](https://helm.sh) package manager. + +## Prerequisites + +APISIX Dashboard requires Kubernetes version 1.14+. + +## Get Repo Info + +```console Review comment: get it -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on a change in pull request #5745: feat(vault): vault lua module, integration with jwt-auth authentication plugin
spacewander commented on a change in pull request #5745: URL: https://github.com/apache/apisix/pull/5745#discussion_r767363371 ## File path: t/plugin/jwt-auth-vault.t ## @@ -163,6 +148,13 @@ passed === TEST 4: sign a jwt and access/verify /secure-endpoint, fails as no secret entry into vault +--- yaml_config Review comment: We can share the config via extra_yaml_config See https://github.com/apache/apisix/blob/71c256be81d95d56ea38f9874699801cbbad6a2e/t/plugin/ext-plugin/http-req-call.t#L47 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander merged pull request #5768: feat: enable L4 stream logging
spacewander merged pull request #5768: URL: https://github.com/apache/apisix/pull/5768 -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
