[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #816: docs: add weekly report 12.15

2021-12-19 Thread GitBox
netlify[bot] edited a comment on pull request #816: URL: https://github.com/apache/apisix-website/pull/816#issuecomment-997663426 ✔️ Deploy Preview for *apache-apisix* ready! 🔨 Explore the source changes: 1239680c25bd53023448af74a76a8b8eaf652a80 🔍 Inspect the deploy log: [

[GitHub] [apisix-ingress-controller] zaunist commented on pull request #805: docs(READEME.md): change image size

2021-12-19 Thread GitBox
zaunist commented on pull request #805: URL: https://github.com/apache/apisix-ingress-controller/pull/805#issuecomment-997663791 cc @gxthrj -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [apisix-website] lijing-21 opened a new pull request #816: docs: add weekly report 12.15

2021-12-19 Thread GitBox
lijing-21 opened a new pull request #816: URL: https://github.com/apache/apisix-website/pull/816 add weekly report 12.15 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

[GitHub] [apisix-website] netlify[bot] commented on pull request #816: docs: add weekly report 12.15

2021-12-19 Thread GitBox
netlify[bot] commented on pull request #816: URL: https://github.com/apache/apisix-website/pull/816#issuecomment-997663426 👷 Deploy Preview for *apache-apisix* processing. 🔨 Explore the source changes: 1239680c25bd53023448af74a76a8b8eaf652a80 🔍 Inspect the deploy log: [htt

[GitHub] [apisix] tisonkun opened a new issue #5860: docs: team page broken

2021-12-19 Thread GitBox
tisonkun opened a new issue #5860: URL: https://github.com/apache/apisix/issues/5860 ### Improved or typo fixed. ![image](https://user-images.githubusercontent.com/18818196/146727616-698a9390-bd39-4d0c-b8dd-f58b5a7150ee.png) https://apisix.apache.org/team/ ### Solution

[GitHub] [apisix] zjzwjcbj commented on issue #5791: request help: Custom plugin reload but unknown plugin

2021-12-19 Thread GitBox
zjzwjcbj commented on issue #5791: URL: https://github.com/apache/apisix/issues/5791#issuecomment-997647616 > See https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/README.md, and you check focus on the configurations about `apisix.customPlugins`. I add the plugin

[GitHub] [apisix-helm-chart] lookhua opened a new issue #197: Error: bad member ID arg (strconv.ParseUint: parsing "": invalid syntax), expecting ID in Hex

2021-12-19 Thread GitBox
lookhua opened a new issue #197: URL: https://github.com/apache/apisix-helm-chart/issues/197 k8s version: 1.20.11-aliyun.1 **first time deploy command:** helm install apisix -n apisix --set apisix.replicaCount=3 --set apisix.resources.requests.cpu=100m --set apisix.resource

[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #815: chore: add bisakhmondal (Bisakh Mondal) with committers in team page

2021-12-19 Thread GitBox
netlify[bot] edited a comment on pull request #815: URL: https://github.com/apache/apisix-website/pull/815#issuecomment-997629225 ✔️ Deploy Preview for *apache-apisix* ready! 🔨 Explore the source changes: 3d6de97b03bf3099f5174e284f73e09b336c0f76 🔍 Inspect the deploy log: [

[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #815: chore: add bisakhmondal (Bisakh Mondal) with committers in team page

2021-12-19 Thread GitBox
netlify[bot] edited a comment on pull request #815: URL: https://github.com/apache/apisix-website/pull/815#issuecomment-997629225 👷 Deploy Preview for *apache-apisix* processing. 🔨 Explore the source changes: 3d6de97b03bf3099f5174e284f73e09b336c0f76 🔍 Inspect the deploy log

[GitHub] [apisix-website] netlify[bot] edited a comment on pull request #815: chore: add bisakhmondal (Bisakh Mondal) with committers in team page

2021-12-19 Thread GitBox
netlify[bot] edited a comment on pull request #815: URL: https://github.com/apache/apisix-website/pull/815#issuecomment-997629225 ✔️ Deploy Preview for *apache-apisix* ready! 🔨 Explore the source changes: 5e37f34780bd96bef2c904a7e4a41df6e362ef67 🔍 Inspect the deploy log: [

[GitHub] [apisix-website] netlify[bot] commented on pull request #815: chore: add bisakhmondal with committers in team page

2021-12-19 Thread GitBox
netlify[bot] commented on pull request #815: URL: https://github.com/apache/apisix-website/pull/815#issuecomment-997629225 👷 Deploy Preview for *apache-apisix* processing. 🔨 Explore the source changes: 5e37f34780bd96bef2c904a7e4a41df6e362ef67 🔍 Inspect the deploy log: [htt

[GitHub] [apisix-website] bisakhmondal opened a new pull request #815: chore: add bisakhmondal with committers in team page

2021-12-19 Thread GitBox
bisakhmondal opened a new pull request #815: URL: https://github.com/apache/apisix-website/pull/815 Changes: Add @bisakhmondal in team page : ) 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

[GitHub] [apisix] Horus-K commented on issue #5847: request help: proxy_pass cannot use https

2021-12-19 Thread GitBox
Horus-K commented on issue #5847: URL: https://github.com/apache/apisix/issues/5847#issuecomment-997625617 location / { index index.html; proxy_pass https://xxx.com; proxy_set_header Host $proxy_host; proxy_set_header X-Real-IP $remote_

[GitHub] [apisix] lizhouyang commented on issue #5834: request help: how to set redirect_uri, when i use openid-connect

2021-12-19 Thread GitBox
lizhouyang commented on issue #5834: URL: https://github.com/apache/apisix/issues/5834#issuecomment-997608346 还是不行,贴一下我的思路,您帮我看下: 1. 建一个服务,提供OAuth的访问 ```JSON { "name": "oauth", "upstream_id": "386066599586562754", "plugins": { "openid-connect": { "

[GitHub] [apisix] leslie-tsang opened a new pull request #5859: chore: improve insta

2021-12-19 Thread GitBox
leslie-tsang opened a new pull request #5859: URL: https://github.com/apache/apisix/pull/5859 ### What this PR does / why we need it: ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new features have been added? *

[GitHub] [apisix] shuaijinchao commented on pull request #5819: feat(splunk): support splunk hec logging plugin

2021-12-19 Thread GitBox
shuaijinchao commented on pull request #5819: URL: https://github.com/apache/apisix/pull/5819#issuecomment-997579146 > need to update `README.md` 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 abo

[GitHub] [apisix] shuaijinchao commented on a change in pull request #5819: feat(splunk): support splunk hec logging plugin

2021-12-19 Thread GitBox
shuaijinchao commented on a change in pull request #5819: URL: https://github.com/apache/apisix/pull/5819#discussion_r772069719 ## File path: apisix/plugins/splunk-hec-logging.lua ## @@ -0,0 +1,189 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-

[GitHub] [apisix] shuaijinchao commented on a change in pull request #5819: feat(splunk): support splunk hec logging plugin

2021-12-19 Thread GitBox
shuaijinchao commented on a change in pull request #5819: URL: https://github.com/apache/apisix/pull/5819#discussion_r772069626 ## File path: docs/zh/latest/plugins/splunk-hec-logging.md ## @@ -0,0 +1,143 @@ +--- +title: splunk-hec-logging +--- + + + +## 摘要 + +- [**定义**](#定义) +

[GitHub] [apisix] tzssangglass commented on a change in pull request #5840: fix(core.log): log level need to be updated in some scenario

2021-12-19 Thread GitBox
tzssangglass commented on a change in pull request #5840: URL: https://github.com/apache/apisix/pull/5840#discussion_r772063743 ## File path: apisix/core/log.lua ## @@ -99,7 +102,10 @@ setmetatable(_M, {__index = function(self, cmd) -- cache the lazily generated method i

[GitHub] [apisix] gaobo0202 opened a new issue #5858: bug: Using proxy-rewrite to overwrite the request header x-forwarded-proto failed

2021-12-19 Thread GitBox
gaobo0202 opened a new issue #5858: URL: https://github.com/apache/apisix/issues/5858 ### Issue description Our upstream service relies on the parameter of x-forwarded-proto and uses the proxy rewrite plug-in to rewrite the parameter. The upstream service does not get the correct re

[GitHub] [apisix] shuaijinchao commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
shuaijinchao commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r772058029 ## File path: apisix/plugins/google-cloud-logging.lua ## @@ -136,23 +136,23 @@ local function send_to_google(oauth, entries) end -local function

[GitHub] [apisix] tzssangglass commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
tzssangglass commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r772058737 ## File path: apisix/plugins/google-cloud-logging.lua ## @@ -136,23 +136,23 @@ local function send_to_google(oauth, entries) end -local function

[GitHub] [apisix] tzssangglass commented on a change in pull request #5819: feat(splunk): support splunk hec logging plugin

2021-12-19 Thread GitBox
tzssangglass commented on a change in pull request #5819: URL: https://github.com/apache/apisix/pull/5819#discussion_r772057171 ## File path: docs/zh/latest/plugins/splunk-hec-logging.md ## @@ -0,0 +1,143 @@ +--- +title: splunk-hec-logging +--- + + + +## 摘要 + +- [**定义**](#定义) +

[GitHub] [apisix] shuaijinchao commented on pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
shuaijinchao commented on pull request #5839: URL: https://github.com/apache/apisix/pull/5839#issuecomment-997560020 > We can use `core.lrucache.plugin_ctx` to ensure each rule has its own auth object: > > https://github.com/apache/apisix/blob/4423509680a0c2103ce23b30a0da71735ef73e8

[GitHub] [apisix] shuaijinchao commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
shuaijinchao commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r772058029 ## File path: apisix/plugins/google-cloud-logging.lua ## @@ -136,23 +136,23 @@ local function send_to_google(oauth, entries) end -local function

[GitHub] [apisix] Zhang-Aoqi opened a new issue #5857: request help: Prometheus Apisix

2021-12-19 Thread GitBox
Zhang-Aoqi opened a new issue #5857: URL: https://github.com/apache/apisix/issues/5857 ### Issue description I want to deploy Apisix and Prometheus on K8S through the Helm package, and monitor Apisix using Prometheus to count all requests that go through Apisix. Now how do I deploy

[GitHub] [apisix-ingress-controller] codecov-commenter edited a comment on pull request #791: feat: add ApisixPluginConfig translator (#638)

2021-12-19 Thread GitBox
codecov-commenter edited a comment on pull request #791: URL: https://github.com/apache/apisix-ingress-controller/pull/791#issuecomment-997183430 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/791?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comme

[GitHub] [apisix] tzssangglass commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
tzssangglass commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r772054988 ## File path: apisix/plugins/google-cloud-logging.lua ## @@ -136,23 +136,23 @@ local function send_to_google(oauth, entries) end -local function

[GitHub] [apisix-ingress-controller] neverCase commented on a change in pull request #791: feat: add ApisixPluginConfig translator (#638)

2021-12-19 Thread GitBox
neverCase commented on a change in pull request #791: URL: https://github.com/apache/apisix-ingress-controller/pull/791#discussion_r772055047 ## File path: pkg/kube/translation/apisix_pluginconfig_test.go ## @@ -0,0 +1,72 @@ +// Licensed to the Apache Software Foundation (ASF)

[GitHub] [apisix-ingress-controller] neverCase commented on a change in pull request #791: feat: add ApisixPluginConfig translator (#638)

2021-12-19 Thread GitBox
neverCase commented on a change in pull request #791: URL: https://github.com/apache/apisix-ingress-controller/pull/791#discussion_r772053053 ## File path: pkg/kube/translation/apisix_pluginconfig_test.go ## @@ -0,0 +1,72 @@ +// Licensed to the Apache Software Foundation (ASF)

[GitHub] [apisix] tzssangglass commented on issue #5850: request help:About use key-auth to control consumer access to different routes

2021-12-19 Thread GitBox
tzssangglass commented on issue #5850: URL: https://github.com/apache/apisix/issues/5850#issuecomment-997549378 > In addition, when you want to give a consumer access to all routes under the service, you need to traverse all routes under the service and then set them one by one through the

[GitHub] [apisix] leslie-tsang commented on a change in pull request #5840: fix(core.log): log level need to be updated in some scenario

2021-12-19 Thread GitBox
leslie-tsang commented on a change in pull request #5840: URL: https://github.com/apache/apisix/pull/5840#discussion_r772047430 ## File path: apisix/core/log.lua ## @@ -42,17 +43,27 @@ local log_levels = { } -local cur_level = ngx.config.subsystem == "http" and -

[GitHub] [apisix] leslie-tsang commented on a change in pull request #5840: fix(core.log): log level need to be updated in some scenario

2021-12-19 Thread GitBox
leslie-tsang commented on a change in pull request #5840: URL: https://github.com/apache/apisix/pull/5840#discussion_r772047430 ## File path: apisix/core/log.lua ## @@ -42,17 +43,27 @@ local log_levels = { } -local cur_level = ngx.config.subsystem == "http" and -

[GitHub] [apisix] leslie-tsang commented on a change in pull request #5840: fix(core.log): log level need to be updated in some scenario

2021-12-19 Thread GitBox
leslie-tsang commented on a change in pull request #5840: URL: https://github.com/apache/apisix/pull/5840#discussion_r772046737 ## File path: apisix/core/log.lua ## @@ -74,8 +85,9 @@ end setmetatable(_M, {__index = function(self, cmd) local log_level = log_levels[cmd] -

[GitHub] [apisix] zjzwjcbj commented on issue #5791: request help: Custom plugin reload but unknown plugin

2021-12-19 Thread GitBox
zjzwjcbj commented on issue #5791: URL: https://github.com/apache/apisix/issues/5791#issuecomment-997539324 > See https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/README.md, and you check focus on the configurations about `apisix.customPlugins`. Can I use like t

[GitHub] [apisix] Horus-K commented on issue #5847: request help: proxy_pass cannot use https

2021-12-19 Thread GitBox
Horus-K commented on issue #5847: URL: https://github.com/apache/apisix/issues/5847#issuecomment-997537822 > Please describe your architecture elaborately. And I need to know how to use proxy_pass https -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix] Horus-K commented on issue #5847: request help: proxy_pass cannot use https

2021-12-19 Thread GitBox
Horus-K commented on issue #5847: URL: https://github.com/apache/apisix/issues/5847#issuecomment-997536530 > Please describe your architecture elaborately. We have 5 nginx online servers, we need to use proxy_pass to gray some traffic to apisix -- This is an automated message from

[GitHub] [apisix-docker] bzp2010 commented on a change in pull request #254: feat: bump APISIX Dashboard to 2.10

2021-12-19 Thread GitBox
bzp2010 commented on a change in pull request #254: URL: https://github.com/apache/apisix-docker/pull/254#discussion_r772040332 ## File path: .github/workflows/dashboard-docker-test.yaml ## @@ -13,7 +13,7 @@ jobs: name: build dashboard & test runs-on: ubuntu-latest

[GitHub] [apisix-ingress-controller] zaunist commented on a change in pull request #805: docs(READEME.md): change image size

2021-12-19 Thread GitBox
zaunist commented on a change in pull request #805: URL: https://github.com/apache/apisix-ingress-controller/pull/805#discussion_r772036906 ## File path: README.md ## @@ -55,7 +55,7 @@ This project is currently general availability. ## Internal Architecture - + Review co

[GitHub] [apisix] guoqqqi commented on pull request #5831: feat: add `file-logger` plugin

2021-12-19 Thread GitBox
guoqqqi commented on pull request #5831: URL: https://github.com/apache/apisix/pull/5831#issuecomment-997520694 TODO: - [ ] add control API - [ ] test - [ ] docs -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

[GitHub] [apisix] tzssangglass commented on issue #5837: request: Expect monitoring metrics to add monitoring of nginx shared memory usage

2021-12-19 Thread GitBox
tzssangglass commented on issue #5837: URL: https://github.com/apache/apisix/issues/5837#issuecomment-997520329 It seems to be quite difficult, and if you use the ngx.shared.DICT API to monitor it, it will lose performance. refer to: https://github.com/openresty/lua-nginx-module#ngxs

[GitHub] [apisix-ingress-controller] tokers commented on a change in pull request #805: docs(READEME.md): change image size

2021-12-19 Thread GitBox
tokers commented on a change in pull request #805: URL: https://github.com/apache/apisix-ingress-controller/pull/805#discussion_r772035239 ## File path: README.md ## @@ -55,7 +55,7 @@ This project is currently general availability. ## Internal Architecture - + Review com

[GitHub] [apisix-ingress-controller] zaunist opened a new pull request #805: docs(READEME.md): change image size

2021-12-19 Thread GitBox
zaunist opened a new pull request #805: URL: https://github.com/apache/apisix-ingress-controller/pull/805 ### Type of change: - [ ] Bugfix - [ ] New feature provided - [x] Improve performance - [ ] Backport patches ### What this PR does / why we need it

[GitHub] [apisix-docker] tokers commented on a change in pull request #254: feat: bump APISIX Dashboard to 2.10

2021-12-19 Thread GitBox
tokers commented on a change in pull request #254: URL: https://github.com/apache/apisix-docker/pull/254#discussion_r772034352 ## File path: .github/workflows/dashboard-docker-test.yaml ## @@ -13,7 +13,7 @@ jobs: name: build dashboard & test runs-on: ubuntu-latest

[GitHub] [apisix-ingress-controller] tokers commented on issue #796: DOC: Adjust the size of the architecture diagram in the README

2021-12-19 Thread GitBox
tokers commented on issue #796: URL: https://github.com/apache/apisix-ingress-controller/issues/796#issuecomment-997516617 > Hi, @gxthrj . I'm pleasure to do this contribution. Thanks! Assigned to you~ -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix] spacewander commented on issue #5855: bug: unambiguous fully-qualified domain name(FQDN), i.e. ended in a dot.

2021-12-19 Thread GitBox
spacewander commented on issue #5855: URL: https://github.com/apache/apisix/issues/5855#issuecomment-997515503 Thanks for the report. Could you submit a PR for this? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [apisix] tzssangglass closed issue #5851: request help: how to install in ubuntu

2021-12-19 Thread GitBox
tzssangglass closed issue #5851: URL: https://github.com/apache/apisix/issues/5851 -- 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-u

[GitHub] [apisix] tzssangglass commented on issue #5851: request help: how to install in ubuntu

2021-12-19 Thread GitBox
tzssangglass commented on issue #5851: URL: https://github.com/apache/apisix/issues/5851#issuecomment-997510186 consider as solved, feel free to reopen this if you need. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [apisix] tokers commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
tokers commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r772031363 ## File path: apisix/plugins/google-cloud-logging.lua ## @@ -61,13 +61,13 @@ local schema = { type = "string", def

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on pull request #101: chore: upgrade log4j2 version to 2.17.0

2021-12-19 Thread GitBox
tzssangglass commented on pull request #101: URL: https://github.com/apache/apisix-java-plugin-runner/pull/101#issuecomment-997504704 > @tzssangglass please release a new version of this project in mail listing got -- This is an automated message from the Apache Git Service. To re

[GitHub] [apisix-dashboard] Carson-T commented on issue #2256: docs: invalid URL when clone repo

2021-12-19 Thread GitBox
Carson-T commented on issue #2256: URL: https://github.com/apache/apisix-dashboard/issues/2256#issuecomment-997411161 Thank you!!! @zaunist @juzhiyuan -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [apisix] Carson-T removed a comment on issue #5851: request help: how to install in ubuntu

2021-12-19 Thread GitBox
Carson-T removed a comment on issue #5851: URL: https://github.com/apache/apisix/issues/5851#issuecomment-997410144 thank you!! -- 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 c

[GitHub] [apisix] Carson-T commented on issue #5851: request help: how to install in ubuntu

2021-12-19 Thread GitBox
Carson-T commented on issue #5851: URL: https://github.com/apache/apisix/issues/5851#issuecomment-997410198 > Hi, @Carson-T , we have a [repository](https://github.com/api7/apisix-build-tools) to packaging apisix for ubuntu, you can refer to package apisix on your machine. thank you

[GitHub] [apisix] Carson-T commented on issue #5851: request help: how to install in ubuntu

2021-12-19 Thread GitBox
Carson-T commented on issue #5851: URL: https://github.com/apache/apisix/issues/5851#issuecomment-997410144 thank you!! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #2256: docs: invalid URL when clone repo

2021-12-19 Thread GitBox
juzhiyuan commented on issue #2256: URL: https://github.com/apache/apisix-dashboard/issues/2256#issuecomment-997409495 Hi @Carson-T, @zaunist has fixed the issue by https://github.com/apache/apisix-dashboard/pull/2257/files -- This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] juzhiyuan closed issue #2256: docs: invalid URL when clone repo

2021-12-19 Thread GitBox
juzhiyuan closed issue #2256: URL: https://github.com/apache/apisix-dashboard/issues/2256 -- 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: notifica

[GitHub] [apisix] bzp2010 edited a comment on issue #5834: request help: how to set redirect_uri, when i use openid-connect

2021-12-19 Thread GitBox
bzp2010 edited a comment on issue #5834: URL: https://github.com/apache/apisix/issues/5834#issuecomment-997328733 Hi, @lizhouyang Enable this plugin on the route, when the user request arrives, APISIX will get the session in the request, if it cannot find it, it will jump to the OIDC

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #2257: fix(docs): download link of install.md

2021-12-19 Thread GitBox
juzhiyuan merged pull request #2257: URL: https://github.com/apache/apisix-dashboard/pull/2257 -- 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: not

[apisix-dashboard] branch release/2.10 updated: fix(docs): download link of install.md (#2257)

2021-12-19 Thread juzhiyuan
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch release/2.10 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git The following commit(s) were added to refs/heads/release/2.10 by this push: new 4a9b101 fix(docs): dow

[GitHub] [apisix-dashboard] zaunist commented on pull request #2257: fix(docs): download link of install.md

2021-12-19 Thread GitBox
zaunist commented on pull request #2257: URL: https://github.com/apache/apisix-dashboard/pull/2257#issuecomment-997394838 cc @okaybase -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specif

[GitHub] [apisix-dashboard] bzp2010 closed issue #2217: request help: Ask how to configure the undefined domain name to point to a service

2021-12-19 Thread GitBox
bzp2010 closed issue #2217: URL: https://github.com/apache/apisix-dashboard/issues/2217 -- 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: notificati

[GitHub] [apisix-dashboard] bzp2010 commented on issue #2217: request help: Ask how to configure the undefined domain name to point to a service

2021-12-19 Thread GitBox
bzp2010 commented on issue #2217: URL: https://github.com/apache/apisix-dashboard/issues/2217#issuecomment-997393560 I will close this issue temporarily, but you can always open it again if you need to. -- This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [apisix-dashboard] bzp2010 edited a comment on issue #2217: request help: Ask how to configure the undefined domain name to point to a service

2021-12-19 Thread GitBox
bzp2010 edited a comment on issue #2217: URL: https://github.com/apache/apisix-dashboard/issues/2217#issuecomment-997393459 Hi, @justinmclean. If you want to set up a route for handling unmatched requests, set up a low-priority route that does not contain request information matches, fo

[GitHub] [apisix-dashboard] bzp2010 commented on issue #2217: request help: Ask how to configure the undefined domain name to point to a service

2021-12-19 Thread GitBox
bzp2010 commented on issue #2217: URL: https://github.com/apache/apisix-dashboard/issues/2217#issuecomment-997393459 If you want to set up a route for handling unmatched requests, set up a low-priority route that does not contain request information matches, for which you can perform cust

[GitHub] [apisix-ingress-controller] codecov-commenter edited a comment on pull request #804: fix: some wrong or invalid logs

2021-12-19 Thread GitBox
codecov-commenter edited a comment on pull request #804: URL: https://github.com/apache/apisix-ingress-controller/pull/804#issuecomment-997387269 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/804?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comme

[GitHub] [apisix-ingress-controller] zaunist commented on issue #796: DOC: Adjust the size of the architecture diagram in the README

2021-12-19 Thread GitBox
zaunist commented on issue #796: URL: https://github.com/apache/apisix-ingress-controller/issues/796#issuecomment-997391428 Hi, @gxthrj . I'm pleasure to do this contribution. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [apisix] dickens7 commented on issue #5855: bug: unambiguous fully-qualified domain name(FQDN), i.e. ended in a dot.

2021-12-19 Thread GitBox
dickens7 commented on issue #5855: URL: https://github.com/apache/apisix/issues/5855#issuecomment-997390832 Domain names ended in a dot can solve this problem #4874 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [apisix] dickens7 opened a new issue #5855: bug: unambiguous fully-qualified domain name(FQDN), i.e. ended in a dot.

2021-12-19 Thread GitBox
dickens7 opened a new issue #5855: URL: https://github.com/apache/apisix/issues/5855 ### Issue description need to upgrade lua-resty-dns-client >=5.2.2 - https://github.com/Kong/lua-resty-dns-client#522-11-mar-2021 ### Environment - apisix version (cmd: `apisix ve

[GitHub] [apisix-dashboard] zaunist commented on issue #2256: wrong command

2021-12-19 Thread GitBox
zaunist commented on issue #2256: URL: https://github.com/apache/apisix-dashboard/issues/2256#issuecomment-997389184 Hi, @Carson-T . I have submitted a PR to fix this issue, please wait for merge to release branch. -- This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix-dashboard] zaunist commented on pull request #2257: fix(docs): download link of install.md

2021-12-19 Thread GitBox
zaunist commented on pull request #2257: URL: https://github.com/apache/apisix-dashboard/pull/2257#issuecomment-997388913 cc @juzhiyuan @bzp2010 -- 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

[GitHub] [apisix-dashboard] zaunist opened a new pull request #2257: fix(docs): download link of install.md

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

[GitHub] [apisix] spacewander edited a comment on pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
spacewander edited a comment on pull request #5839: URL: https://github.com/apache/apisix/pull/5839#issuecomment-997388461 Or we can bind the auth_config with the `process` pass to add_entry_to_new_processor, which looks better to me. -- This is an automated message from the Apache Git

[GitHub] [apisix] spacewander commented on pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
spacewander commented on pull request #5839: URL: https://github.com/apache/apisix/pull/5839#issuecomment-997388461 Or we can bind the auth_config with the `process` pass to add_entry_to_new_processor. -- This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [apisix-ingress-controller] codecov-commenter commented on pull request #804: fix: fields in logs do not match variables

2021-12-19 Thread GitBox
codecov-commenter commented on pull request #804: URL: https://github.com/apache/apisix-ingress-controller/pull/804#issuecomment-997387269 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/804?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_

[GitHub] [apisix] spacewander commented on a change in pull request #5819: feat(splunk): support splunk hec logging plugin

2021-12-19 Thread GitBox
spacewander commented on a change in pull request #5819: URL: https://github.com/apache/apisix/pull/5819#discussion_r771945523 ## File path: apisix/plugins/splunk-hec-logging.lua ## @@ -0,0 +1,189 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix-ingress-controller] nic-6443 opened a new pull request #804: fix: fields in logs do not match variables

2021-12-19 Thread GitBox
nic-6443 opened a new pull request #804: URL: https://github.com/apache/apisix-ingress-controller/pull/804 ### Type of change: - [x] Bugfix -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

[GitHub] [apisix-ingress-controller] gxthrj commented on a change in pull request #791: feat: add ApisixPluginConfig translator (#638)

2021-12-19 Thread GitBox
gxthrj commented on a change in pull request #791: URL: https://github.com/apache/apisix-ingress-controller/pull/791#discussion_r771944347 ## File path: pkg/kube/translation/apisix_pluginconfig.go ## @@ -0,0 +1,36 @@ +// Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [apisix-ingress-controller] gxthrj commented on a change in pull request #791: feat: add ApisixPluginConfig translator (#638)

2021-12-19 Thread GitBox
gxthrj commented on a change in pull request #791: URL: https://github.com/apache/apisix-ingress-controller/pull/791#discussion_r771944230 ## File path: pkg/kube/translation/apisix_pluginconfig.go ## @@ -0,0 +1,36 @@ +// Licensed to the Apache Software Foundation (ASF) under o

[GitHub] [apisix] spacewander commented on pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
spacewander commented on pull request #5839: URL: https://github.com/apache/apisix/pull/5839#issuecomment-997385527 We can use `core.lrucache.plugin_ctx` to ensure each rule has its own auth object: https://github.com/apache/apisix/blob/4423509680a0c2103ce23b30a0da71735ef73e8f/apisix/plugi

[GitHub] [apisix] spacewander commented on pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
spacewander commented on pull request #5839: URL: https://github.com/apache/apisix/pull/5839#issuecomment-997384916 Look like there is another issue: `auth_config_cache` is not updated even the auth_config has changed. -- This is an automated message from the Apache Git Service. To respo

[GitHub] [apisix] spacewander commented on a change in pull request #5839: fix(google-cloud-logging): move ssl_verify to plugin configuration top level

2021-12-19 Thread GitBox
spacewander commented on a change in pull request #5839: URL: https://github.com/apache/apisix/pull/5839#discussion_r771942631 ## File path: apisix/plugins/google-cloud-logging/oauth.lua ## @@ -111,10 +111,10 @@ function _M:new(config) access_token_expire_time = 0,

[GitHub] [apisix-docker] gxthrj commented on a change in pull request #254: feat: bump APISIX Dashboard to 2.10

2021-12-19 Thread GitBox
gxthrj commented on a change in pull request #254: URL: https://github.com/apache/apisix-docker/pull/254#discussion_r771943158 ## File path: Makefile ## @@ -160,6 +160,21 @@ push-dashboard: @$(call func_echo_success_status, "$@ -> [ Done ]") +### push-multiarch-dash

[GitHub] [apisix-docker] spacewander commented on issue #255: bug: wrong APISIX ARM64 image

2021-12-19 Thread GitBox
spacewander commented on issue #255: URL: https://github.com/apache/apisix-docker/issues/255#issuecomment-997380667 Good catch! Is it urgent to fix? -- 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

[apisix] branch master updated: fix(ua-restriction): refine plugin configuration check logic (#5728)

2021-12-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git The following commit(s) were added to refs/heads/master by this push: new 4423509 fix(ua-restriction): refine plugin

[GitHub] [apisix] spacewander merged pull request #5728: fix(ua-restriction): refine plugin configuration check logic

2021-12-19 Thread GitBox
spacewander merged pull request #5728: URL: https://github.com/apache/apisix/pull/5728 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificatio

[GitHub] [apisix] spacewander closed issue #5724: bug: regular matches have bugs in specific scenarios

2021-12-19 Thread GitBox
spacewander closed issue #5724: URL: https://github.com/apache/apisix/issues/5724 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-un

[GitHub] [apisix-dashboard] Carson-T opened a new issue #2256: wrong command

2021-12-19 Thread GitBox
Carson-T opened a new issue #2256: URL: https://github.com/apache/apisix-dashboard/issues/2256 ### Issue description wrong command ### Expected behavior The install.md file has a wrong command git clone -b release/2.10.0 https://github.com/apache/apisix-dashboard.

[GitHub] [apisix-dashboard] chanjet1213 opened a new issue #2255: request help: dashboard认证登录

2021-12-19 Thread GitBox
chanjet1213 opened a new issue #2255: URL: https://github.com/apache/apisix-dashboard/issues/2255 ### Issue description dashboard何时支持认证登录?比如支持openLDAP。 ### Environment - apisix version (cmd: `apisix version`): - OS (cmd: `uname -a`): - OpenResty / Nginx version (cm

[apisix] branch master updated (5cde5ae -> a11c2d1)

2021-12-19 Thread spacewander
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git. from 5cde5ae feat: upgrade luarocks to 3.8.0 which converts git:// to https:// (#5825) add a11c2d1 docs: sync

[GitHub] [apisix] spacewander merged pull request #5793: docs: sync stream proxy Chinese version

2021-12-19 Thread GitBox
spacewander merged pull request #5793: URL: https://github.com/apache/apisix/pull/5793 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notificatio

[GitHub] [apisix-java-plugin-runner] moonming commented on pull request #101: chore: upgrade log4j2 version to 2.17.0

2021-12-19 Thread GitBox
moonming commented on pull request #101: URL: https://github.com/apache/apisix-java-plugin-runner/pull/101#issuecomment-997373026 @tzssangglass please release a new version of this project in mail listing -- This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [apisix-java-plugin-runner] membphis merged pull request #101: chore: upgrade log4j2 version to 2.17.0

2021-12-19 Thread GitBox
membphis merged pull request #101: URL: https://github.com/apache/apisix-java-plugin-runner/pull/101 -- 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-mai

[apisix-java-plugin-runner] branch main updated (d5adbe0 -> 4e3279f)

2021-12-19 Thread membphis
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/apisix-java-plugin-runner.git. from d5adbe0 docs: add zh Quick Start (#95) add 4e3279f chore: upgrade log4j2 to 2.17.0 (#101)

[GitHub] [apisix] membphis commented on pull request #5728: fix(ua-restriction): refine the ua-restriction plugin configuration checksum logic

2021-12-19 Thread GitBox
membphis commented on pull request #5728: URL: https://github.com/apache/apisix/pull/5728#issuecomment-997370246 I rerun the CI right now. one of the ci failed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

[GitHub] [apisix-dashboard] github-actions[bot] commented on issue #2217: request help: Ask how to configure the undefined domain name to point to a service

2021-12-19 Thread GitBox
github-actions[bot] commented on issue #2217: URL: https://github.com/apache/apisix-dashboard/issues/2217#issuecomment-997362732 This issue has been marked as stale due to 30 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, p

[GitHub] [apisix-docker] bzp2010 commented on pull request #254: feat: bump APISIX Dashboard to 2.10

2021-12-19 Thread GitBox
bzp2010 commented on pull request #254: URL: https://github.com/apache/apisix-docker/pull/254#issuecomment-997357433 ## Update I've done a local test of multi-architecture packaging and it completes in about 10 minutes or so [[demo]](https://github.com/bzp2010/apisix-docker/runs/4573474

[GitHub] [apisix] zuiyangqingzhou commented on a change in pull request #5728: fix(ua-restriction): refine the ua-restriction plugin configuration checksum logic

2021-12-19 Thread GitBox
zuiyangqingzhou commented on a change in pull request #5728: URL: https://github.com/apache/apisix/pull/5728#discussion_r771922260 ## File path: t/plugin/ua-restriction.t ## @@ -783,5 +790,7 @@ load_full_data(): failed to check item data of [/apisix/routes] err:failed to ch