[GitHub] [apisix-dashboard] wangxunfan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
wangxunfan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719228934 @juzhiyuan thanks,“deploy.md” file is very deatailed. Successfully deployed. This is an automated message

[GitHub] [apisix] nic-chen commented on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nic-chen commented on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719181158 > I will vote for ingress-nginx's snippet feature: > > https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#configuration-snippet > >

[GitHub] [apisix] moonming opened a new issue #2573: request help: support centos 7, centos 8 in CI

2020-10-29 Thread GitBox
moonming opened a new issue #2573: URL: https://github.com/apache/apisix/issues/2573 ### Issue description ### Environment * apisix version (cmd: `apisix version`): * OS: This is an automated message

[GitHub] [apisix] spacewander merged pull request #2560: doc: Update CHANGELOG_CN.md

2020-10-29 Thread GitBox
spacewander merged pull request #2560: URL: https://github.com/apache/apisix/pull/2560 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: Update CHANGELOG_CN.md (#2560)

2020-10-29 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 1fb000c Update CHANGELOG_CN.md (#2560)

[GitHub] [apisix] spacewander commented on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
spacewander commented on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719163843 I will vote for ingress-nginx's snippet feature: https://kubernetes.github.io/ingress-nginx/user-guide/nginx-configuration/annotations/#configuration-snippet This

[GitHub] [apisix-dashboard] thekingofworld edited a comment on issue #646: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
thekingofworld edited a comment on issue #646: URL: https://github.com/apache/apisix-dashboard/issues/646#issuecomment-719163126 @juzhiyuan here is output, seems like need a Authentication.

[GitHub] [apisix-dashboard] thekingofworld commented on issue #646: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
thekingofworld commented on issue #646: URL: https://github.com/apache/apisix-dashboard/issues/646#issuecomment-719163126 @juzhiyuan ![image](https://user-images.githubusercontent.com/8169665/97659798-e0033080-1aaa-11eb-8780-47d109a019fd.png)

[GitHub] [apisix] tzssangglass commented on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
tzssangglass commented on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719163225 You mean like the nginx_http_include directive from kong? ``` nginx_http_include = /path/to/your/extend.conf ``` the `extend.conf` may be like ```conf

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #612: test: add e2e test for field hosts in `route` api

2020-10-29 Thread GitBox
juzhiyuan commented on a change in pull request #612: URL: https://github.com/apache/apisix-dashboard/pull/612#discussion_r514829914 ## File path: api/test/e2e/route_test.go ## @@ -0,0 +1,164 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [apisix-dashboard] membphis commented on a change in pull request #612: test: add e2e test for field hosts in `route` api

2020-10-29 Thread GitBox
membphis commented on a change in pull request #612: URL: https://github.com/apache/apisix-dashboard/pull/612#discussion_r514822176 ## File path: api/go.sum ## @@ -1,43 +1,14 @@ -cloud.google.com/go v0.34.0/go.mod h1:aQUYkXzVsufM+DwF1aE+0xfcU+56JwCaLick0ClmMTw= Review

[GitHub] [apisix-docker] hfhimage opened a new issue #82: request help: can not find apache/apisix:2.0-alpine image in dockerhub

2020-10-29 Thread GitBox
hfhimage opened a new issue #82: URL: https://github.com/apache/apisix-docker/issues/82 ### Issue description I can't find apache/apisix:2.0-alpine in hub.docker.com after Apisix 2.0 released. So I wonder if it's much more recommend to use centos version instead of alpine version in

[GitHub] [apisix-dashboard] juzhiyuan opened a new issue #647: Support deploy with Docker

2020-10-29 Thread GitBox
juzhiyuan opened a new issue #647: URL: https://github.com/apache/apisix-dashboard/issues/647 TODO 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] juzhiyuan commented on a change in pull request #612: test: add e2e test for field hosts in `route` api

2020-10-29 Thread GitBox
juzhiyuan commented on a change in pull request #612: URL: https://github.com/apache/apisix-dashboard/pull/612#discussion_r514812818 ## File path: .github/workflows/test-api.yml ## @@ -42,8 +42,9 @@ jobs: sudo apt install golang-1.14-go - name: run test +

[GitHub] [apisix] hfhimage opened a new issue #2572: request help: can not find apache/apisix:2.0-alpine image in dockerhub

2020-10-29 Thread GitBox
hfhimage opened a new issue #2572: URL: https://github.com/apache/apisix/issues/2572 ### Issue description I can't find apache/apisix:2.0-alpine in hub.docker.com after Apisix 2.0 released. So I wonder if it's much more recommend to use centos version instead of alpine version in

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #646: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
juzhiyuan commented on issue #646: URL: https://github.com/apache/apisix-dashboard/issues/646#issuecomment-719157575 BTW, please use the latest dashboard 2.0 RC2 to have a test. https://github.com/apache/apisix-dashboard/blob/v2.0/docs/deploy.md

[GitHub] [apisix] idbeta commented on issue #2501: [discuss]: How to distinguish whether the 5xx status code (eg 500) comes from upstream or apisix

2020-10-29 Thread GitBox
idbeta commented on issue #2501: URL: https://github.com/apache/apisix/issues/2501#issuecomment-719157283 I agree with the idea of adding a new field in the response header. This is an automated message from the Apache Git

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #646: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
juzhiyuan commented on issue #646: URL: https://github.com/apache/apisix-dashboard/issues/646#issuecomment-719157222 Hi, please follow those steps to have a check first: 1. Get all plugin list ```sh $ curl $API_HOST/apisix/admin/plugins ``` 2. Check if your

[GitHub] [apisix-dashboard] thekingofworld opened a new issue #646: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
thekingofworld opened a new issue #646: URL: https://github.com/apache/apisix-dashboard/issues/646 ### Issue description I have added a custom plugin, how to let the custom plugin show in dashboard?

[GitHub] [apisix] nanamikon edited a comment on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nanamikon edited a comment on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719153151 Sure! For example Third-party nginx module param ``` req_status_zone host "$server_name" 200M; req_status_zone hostadr "$server_name:$upstream_addr" 200M;

[GitHub] [apisix] nanamikon edited a comment on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nanamikon edited a comment on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719153151 Sure! For example Third-party nginx module param ``` req_status_zone host "$server_name" 200M; req_status_zone hostadr "$server_name:$upstream_addr" 200M;

[GitHub] [apisix] nanamikon commented on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nanamikon commented on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719153151 Sure! For example ``` req_status_zone host "$server_name" 200M; req_status_zone hostadr "$server_name:$upstream_addr" 200M; req_status host hostadr; req_status_lazy

[GitHub] [apisix] starsz commented on a change in pull request #2566: chore: move try_read_attr function into table.lua (#2257)

2020-10-29 Thread GitBox
starsz commented on a change in pull request #2566: URL: https://github.com/apache/apisix/pull/2566#discussion_r514788575 ## File path: apisix/plugins/hmac-auth.lua ## @@ -328,7 +312,7 @@ local function get_params(ctx) local date_key = DATE_KEY local

[GitHub] [apisix-dashboard] juzhiyuan commented on a change in pull request #619: feat: added e2e test for Login page

2020-10-29 Thread GitBox
juzhiyuan commented on a change in pull request #619: URL: https://github.com/apache/apisix-dashboard/pull/619#discussion_r514773085 ## File path: src/e2e/Login.e2e.js ## @@ -1,33 +1,69 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [apisix-dashboard] LiteSun opened a new issue #645: requirement: add e2e test for Login & Logout

2020-10-29 Thread GitBox
LiteSun opened a new issue #645: URL: https://github.com/apache/apisix-dashboard/issues/645 - [ ] login: correct and incorrect password/username - [ ] logout related issue #614 This is an automated message from

[GitHub] [apisix-dashboard] LiteSun opened a new issue #644: requirement: add e2e test for Setting

2020-10-29 Thread GitBox
LiteSun opened a new issue #644: URL: https://github.com/apache/apisix-dashboard/issues/644 Setting page jump: set grafana address: related issue #614 This is an automated message from the Apache Git Service.

[GitHub] [apisix] jeffguorg commented on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg commented on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719150249 @membphis solution confirmed. i recreated the cluster and reconfigured apisix and it works like a charm. still... do you have any plan to backport the patch to 1.x branch?

[GitHub] [apisix-dashboard] LiteSun opened a new issue #643: requirement: add e2e test for Consumer

2020-10-29 Thread GitBox
LiteSun opened a new issue #643: URL: https://github.com/apache/apisix-dashboard/issues/643 Consumer - [ ] create: not enable authentication plugin, create the same name consumer - [ ] edit: - [ ] delete: related issue #614

[GitHub] [apisix] starsz commented on issue #2569: CI: line is too line

2020-10-29 Thread GitBox
starsz commented on issue #2569: URL: https://github.com/apache/apisix/issues/2569#issuecomment-719149362 Okay, I follow the decision of the community. Hmmm. This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] LiteSun opened a new issue #642: requirement: add e2e test for Upstream

2020-10-29 Thread GitBox
LiteSun opened a new issue #642: URL: https://github.com/apache/apisix-dashboard/issues/642 Upstream - [ ] create: long description and name include common symbols - [ ] edit: - [ ] delete: - [ ] search: full match, partial match, no match - [ ] big data environment test:

[GitHub] [apisix] RocFang closed issue #2568: bug: uri-blocker's code and doc are confusing

2020-10-29 Thread GitBox
RocFang closed issue #2568: URL: https://github.com/apache/apisix/issues/2568 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] RocFang commented on issue #2568: bug: uri-blocker's code and doc are confusing

2020-10-29 Thread GitBox
RocFang commented on issue #2568: URL: https://github.com/apache/apisix/issues/2568#issuecomment-719148739 @idbeta sorry, my mistake, ignore it:) This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] LiteSun opened a new issue #641: requirement: added e2e test for SSL

2020-10-29 Thread GitBox
LiteSun opened a new issue #641: URL: https://github.com/apache/apisix-dashboard/issues/641 SSL - [ ] enter correct certificate and private key: - [ ] enter unmatched certificate and private key: - [ ] upload correct certificate and private key: - [ ] upload unmatched

[GitHub] [apisix] tokers commented on pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
tokers commented on pull request #2548: URL: https://github.com/apache/apisix/pull/2548#issuecomment-719148269 > > I think we can also use the etcd proxy localily as an interim solution > > [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd] > > Do you means add sidecar for

[GitHub] [apisix] tokers edited a comment on pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
tokers edited a comment on pull request #2548: URL: https://github.com/apache/apisix/pull/2548#issuecomment-719148269 > > I think we can also use the etcd proxy localily as an interim solution > > [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd] > > Do you means add

[GitHub] [apisix-dashboard] LiteSun opened a new issue #640: requirement: added e2e test for Route

2020-10-29 Thread GitBox
LiteSun opened a new issue #640: URL: https://github.com/apache/apisix-dashboard/issues/640 Route - [ ] create: long description and name include common symbols - [ ] edit: - [ ] delete: - [ ] search: full match, partial match, no match - [ ] big data environment test: route

[GitHub] [apisix] thekingofworld commented on issue #2571: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
thekingofworld commented on issue #2571: URL: https://github.com/apache/apisix/issues/2571#issuecomment-719148060 The plugin has been configured. ![image](https://user-images.githubusercontent.com/8169665/97656378-4df72a00-1aa2-11eb-8d47-21c8c77becba.png)

[GitHub] [apisix] thekingofworld opened a new issue #2571: request help: let custom plugin show in dashboard

2020-10-29 Thread GitBox
thekingofworld opened a new issue #2571: URL: https://github.com/apache/apisix/issues/2571 ### Issue description I have added a custom plugin, how to let the custom plugin show in dashboard?

[GitHub] [apisix-dashboard] LiteSun opened a new issue #639: requirement: added e2e test for Metrics

2020-10-29 Thread GitBox
LiteSun opened a new issue #639: URL: https://github.com/apache/apisix-dashboard/issues/639 - [ ] page jump This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [apisix] spacewander opened a new pull request #2570: chore: update CODE_STYLE to catch up with the reality

2020-10-29 Thread GitBox
spacewander opened a new pull request #2570: URL: https://github.com/apache/apisix/pull/2570 Currently we already have the line length style check in lj-releng. The length we chose is 100. Close #2569. ### What this PR does / why we need it: ###

[GitHub] [apisix] spacewander commented on issue #2569: CI: line is too line

2020-10-29 Thread GitBox
spacewander commented on issue #2569: URL: https://github.com/apache/apisix/issues/2569#issuecomment-719145762 @starsz Currently we already have the line length style check in `lj-releng`. The length we chose is 100.

[GitHub] [apisix] thekingofworld closed issue #2542: request help: configure route to only execute plugins

2020-10-29 Thread GitBox
thekingofworld closed issue #2542: URL: https://github.com/apache/apisix/issues/2542 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

[GitHub] [apisix] wfgydbu commented on issue #2520: bug: "failed to acquire the lock: timeout" in logs/error.log when setuping route at the first time

2020-10-29 Thread GitBox
wfgydbu commented on issue #2520: URL: https://github.com/apache/apisix/issues/2520#issuecomment-719145137 @spacewander ``` [root@node10 apisix]# uptime 11:10:16 up 139 days, 17:03, 5 users, load average: 1.17, 1.15, 1.26 [root@node10 apisix]# grep -c 'model name'

[GitHub] [apisix] idbeta commented on issue #2568: bug: uri-blocker's code and doc are confusing

2020-10-29 Thread GitBox
idbeta commented on issue #2568: URL: https://github.com/apache/apisix/issues/2568#issuecomment-719145049 Can you write more details about regular expressions that go wrong? That would be very helpful This is an automated

[GitHub] [apisix] tokers commented on issue #2569: CI: line is too line

2020-10-29 Thread GitBox
tokers commented on issue #2569: URL: https://github.com/apache/apisix/issues/2569#issuecomment-719144786 The specification of 80 characters forces programmer to design concise variable name, function name and literals, I don't think it makes the code more awkward to read (if so,

[GitHub] [apisix] idbeta commented on issue #2569: CI: line is too line

2020-10-29 Thread GitBox
idbeta commented on issue #2569: URL: https://github.com/apache/apisix/issues/2569#issuecomment-719143124 I agree that 80 characters are enough most of the time, but sometimes 80 characters seem a little bit short and make the code more awkward to read, just for python.

[GitHub] [apisix] starsz opened a new issue #2569: CI: line is too line

2020-10-29 Thread GitBox
starsz opened a new issue #2569: URL: https://github.com/apache/apisix/issues/2569 Hi, while I am reading the code, I found that some lines are too long (More than eighty characters). As the

[GitHub] [apisix] spacewander commented on issue #2520: bug: "failed to acquire the lock: timeout" in logs/error.log when setuping route at the first time

2020-10-29 Thread GitBox
spacewander commented on issue #2520: URL: https://github.com/apache/apisix/issues/2520#issuecomment-719139512 @wfgydbu What was the CPU usage and loadavg when the `failed to acquire` error happened? The difference between v1.5 and v1.2 which causes the issue is that v1.5

[GitHub] [apisix] Miss-you commented on issue #2554: request help: abnormal output after using echo plugin

2020-10-29 Thread GitBox
Miss-you commented on issue #2554: URL: https://github.com/apache/apisix/issues/2554#issuecomment-719138194 > @spacewander It seems that there is still a problem after I use your repaired version. >

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
juzhiyuan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719134057 hmmm @wangxunfan, I mean you are using old codes in the master branch, could you please have a try on the v2.0 branch?

[GitHub] [apisix-dashboard] juzhiyuan closed issue #621: requirement: Support custom listening IP and port for manager api

2020-10-29 Thread GitBox
juzhiyuan closed issue #621: URL: https://github.com/apache/apisix-dashboard/issues/621 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [apisix-dashboard] juzhiyuan edited a comment on issue #615: v2.0-rc2 Build docker image error.

2020-10-29 Thread GitBox
juzhiyuan edited a comment on issue #615: URL: https://github.com/apache/apisix-dashboard/issues/615#issuecomment-719133226 Deploy with Docker Compose is removed, and we are developing the new Dockerfile, so I will close this issue. The new Dockerfile will be added to RC3.

[GitHub] [apisix-dashboard] juzhiyuan closed issue #615: v2.0-rc2 Build docker image error.

2020-10-29 Thread GitBox
juzhiyuan closed issue #615: URL: https://github.com/apache/apisix-dashboard/issues/615 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #615: v2.0-rc2 Build docker image error.

2020-10-29 Thread GitBox
juzhiyuan commented on issue #615: URL: https://github.com/apache/apisix-dashboard/issues/615#issuecomment-719133226 Deploy with Docker Compose is removed, and we are developing the new Dockerfile, so I will close this issue. The new Dockerfile will be added in RC3.

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #621: requirement: Support custom listening IP and port for manager api

2020-10-29 Thread GitBox
juzhiyuan commented on issue #621: URL: https://github.com/apache/apisix-dashboard/issues/621#issuecomment-719133415 This issue has been resolved in 2.0 RC2 This is an automated message from the Apache Git Service. To

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #608: manager服务构建docker镜像使用了可能不稳定的 apisix:master 分支

2020-10-29 Thread GitBox
juzhiyuan commented on issue #608: URL: https://github.com/apache/apisix-dashboard/issues/608#issuecomment-719132807 The new Dockerfile is under development, so I will close this issue. This is an automated message from the

[GitHub] [apisix-dashboard] juzhiyuan closed issue #608: manager服务构建docker镜像使用了可能不稳定的 apisix:master 分支

2020-10-29 Thread GitBox
juzhiyuan closed issue #608: URL: https://github.com/apache/apisix-dashboard/issues/608 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

[GitHub] [apisix-dashboard] wangxunfan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
wangxunfan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719132636 I found a problem. problem: 404 error , http://192.168.202.127:8080/ version: apisix-dashboard 2.0 RC2 file: ./api/run.sh detail: Processing without

[GitHub] [apisix] wfgydbu edited a comment on issue #2520: bug: "failed to acquire the lock: timeout" in logs/error.log when setuping route at the first time

2020-10-29 Thread GitBox
wfgydbu edited a comment on issue #2520: URL: https://github.com/apache/apisix/issues/2520#issuecomment-719110566 > It seems that when check the schema of Routes, APISIX tries to fetch the schema firstly and compile it, the compilation result will be cached inside a LRU cache. The schema

[GitHub] [apisix] RocFang opened a new issue #2568: bug: uri-blocker's code and doc are confusing

2020-10-29 Thread GitBox
RocFang opened a new issue #2568: URL: https://github.com/apache/apisix/issues/2568 ### Issue description In uri-blocker's doc, we can get every block rule is a independent regular expression. But in the code, we can find that every rule is concated into a single regular

[GitHub] [apisix] nic-chen edited a comment on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nic-chen edited a comment on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719114815 Could you please show some example? This is an automated message from the Apache Git Service. To respond to

[GitHub] [apisix] tokers commented on a change in pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
tokers commented on a change in pull request #2548: URL: https://github.com/apache/apisix/pull/2548#discussion_r514696232 ## File path: apisix/core/config_etcd.lua ## @@ -472,6 +472,10 @@ function _M.new(key, opts) etcd_conf.protocol = "v3" etcd_conf.api_prefix =

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
juzhiyuan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719123478 > @juzhiyuan > > 1. Nginx error.log file printing missing api-key >2.Deploy apisix 2. 0 and apisix-dashboard v2.0 rc2 according to the document。 >

[GitHub] [apisix] ShiningRush commented on a change in pull request #2556: feature:support multi service discovery

2020-10-29 Thread GitBox
ShiningRush commented on a change in pull request #2556: URL: https://github.com/apache/apisix/pull/2556#discussion_r514690061 ## File path: apisix/discovery/eureka.lua ## @@ -66,7 +66,8 @@ local _M = { local function service_info() -local host = local_conf.eureka and

[GitHub] [apisix] jeffguorg edited a comment on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg edited a comment on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719119397 setting resolv.conf does not fix this problem. ```bash guochao@chao-20kna04gcd ~ % curl http:/// 502 Bad Gateway 502 Bad Gateway openresty

[GitHub] [apisix] jeffguorg commented on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg commented on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719119397 setting resolv.conf to ```nameserver 172.21.0.10``` does not fix this problem. i'm gonna give 2.0 a try

[GitHub] [apisix-dashboard] moonming commented on issue #607: proposal: e2e test

2020-10-29 Thread GitBox
moonming commented on issue #607: URL: https://github.com/apache/apisix-dashboard/issues/607#issuecomment-719116525 please add tag for those small issues. This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix] jeffguorg edited a comment on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg edited a comment on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719115231 @membphis yes, i started to use apisix just a few day ago before 2.0 is released. so this is already fix in 2.0, right? is it possible to backport it to 1.5?

[GitHub] [apisix] jeffguorg edited a comment on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg edited a comment on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719115231 @membphis yes, i started to use apisix just a few day ago before 2.0 is released. so this is already fix in 2.0, right? is it possible to backport it to 1.5?

[GitHub] [apisix] jeffguorg commented on issue #2533: request help: failed to set server peer [some-k8s-svc.default.svc.cluster.local:5202] err: no host allowed

2020-10-29 Thread GitBox
jeffguorg commented on issue #2533: URL: https://github.com/apache/apisix/issues/2533#issuecomment-719115231 @membphis yes, i started to use apisix just a few day ago before 2.0 is released. so this is already fix in 2.0, right? is it possible to backport it to 1.5? i'm

[GitHub] [apisix] nic-chen commented on issue #2565: support include other nginx config in the apisix(lua file)

2020-10-29 Thread GitBox
nic-chen commented on issue #2565: URL: https://github.com/apache/apisix/issues/2565#issuecomment-719114815 Could you show some example? This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix] wfgydbu commented on issue #2562: bug: APISIX 2.0 run failed on CentOS 7 missing package lua-tinyyaml and lua-resty-template

2020-10-29 Thread GitBox
wfgydbu commented on issue #2562: URL: https://github.com/apache/apisix/issues/2562#issuecomment-719113879 Please try `make deps` before `make init`, and make sure all dependencies has been installed **successfully**. Some of them may fail but the installation will continue.

[GitHub] [apisix-dashboard] wangxunfan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
wangxunfan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719112811 /app/apisix/conf/nginx.conf: server { listen 9080 reuseport; . location /apisix/dashboard { allow 127.0.0.0/24;

[GitHub] [apisix-dashboard] wangxunfan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
wangxunfan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719112506 Nginx error.log file printing: 2020/10/29 07:28:32 [warn] 12213#12213: *28873 [lua] ctx.lua:83: __index(): failed to fetch cookie value by key: cookie_x_api_key error:

[GitHub] [apisix] wfgydbu edited a comment on issue #2520: bug: "failed to acquire the lock: timeout" in logs/error.log when setuping route at the first time

2020-10-29 Thread GitBox
wfgydbu edited a comment on issue #2520: URL: https://github.com/apache/apisix/issues/2520#issuecomment-719110566 > It seems that when check the schema of Routes, APISIX tries to fetch the schema firstly and compile it, the compilation result will be cached inside a LRU cache. The schema

[GitHub] [apisix] wfgydbu commented on issue #2520: bug: "failed to acquire the lock: timeout" in logs/error.log when setuping route at the first time

2020-10-29 Thread GitBox
wfgydbu commented on issue #2520: URL: https://github.com/apache/apisix/issues/2520#issuecomment-719110566 > It seems that when check the schema of Routes, APISIX tries to fetch the schema firstly and compile it, the compilation result will be cached inside a LRU cache. The schema

[GitHub] [apisix] tzssangglass commented on issue #2523: request help: jwt-auth plugin RS256 algorithm is not easy to use

2020-10-29 Thread GitBox
tzssangglass commented on issue #2523: URL: https://github.com/apache/apisix/issues/2523#issuecomment-719110500 > @tzssangglass Do you have any suggestions for this issue? Add two redundant fields, such as public_key, private_key, for the RSA algorithm.

[GitHub] [apisix] nic-chen commented on a change in pull request #2556: feature:support multi service discovery

2020-10-29 Thread GitBox
nic-chen commented on a change in pull request #2556: URL: https://github.com/apache/apisix/pull/2556#discussion_r514656355 ## File path: apisix/discovery/eureka.lua ## @@ -66,7 +66,8 @@ local _M = { local function service_info() -local host = local_conf.eureka and

[GitHub] [apisix-dashboard] LiteSun commented on pull request #619: feat: added e2e test for Login page

2020-10-29 Thread GitBox
LiteSun commented on pull request #619: URL: https://github.com/apache/apisix-dashboard/pull/619#issuecomment-719109644 > just a proposal:define the dom selectors and test datas separately as variables, It may be better to maintain later. > > A test case for a page contains three

[GitHub] [apisix] spacewander commented on issue #2554: request help: abnormal output after using echo plugin

2020-10-29 Thread GitBox
spacewander commented on issue #2554: URL: https://github.com/apache/apisix/issues/2554#issuecomment-719107301 @thekingofworld Because the fix depends on function introduced after 2.0. If the echo plugin is important to you, you should try the master branch.

[GitHub] [apisix-dashboard] wangxunfan commented on issue #623: apisix-dashboard 403 Forbidden

2020-10-29 Thread GitBox
wangxunfan commented on issue #623: URL: https://github.com/apache/apisix-dashboard/issues/623#issuecomment-719105351 @juzhiyuan 1. Nginx error.log file printing missing api-key 2.Deploy apisix 2. 0 and apisix-dashboard v2.0 rc2 according to the document。

[GitHub] [apisix] spacewander commented on a change in pull request #2566: chore: move try_read_attr function into table.lua (#2257)

2020-10-29 Thread GitBox
spacewander commented on a change in pull request #2566: URL: https://github.com/apache/apisix/pull/2566#discussion_r514646591 ## File path: apisix/plugins/log-rotate.lua ## @@ -152,25 +150,11 @@ local function scan_log_folder() end -local function try_attr(t, ...) -

[GitHub] [apisix] spacewander commented on a change in pull request #2525: doc : fixes issue #2327

2020-10-29 Thread GitBox
spacewander commented on a change in pull request #2525: URL: https://github.com/apache/apisix/pull/2525#discussion_r514645942 ## File path: doc/grpc-proxy.md ## @@ -17,7 +17,7 @@ # --> -[中文](zh-cn/grpc-proxy.md) +[Chinese](zh-cn/grpc-proxy.md) Review comment: It

[GitHub] [apisix] spacewander closed issue #2558: doc: English style, " , " -> ", "

2020-10-29 Thread GitBox
spacewander closed issue #2558: URL: https://github.com/apache/apisix/issues/2558 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] spacewander commented on issue #2558: doc: English style, " , " -> ", "

2020-10-29 Thread GitBox
spacewander commented on issue #2558: URL: https://github.com/apache/apisix/issues/2558#issuecomment-719100927 Solved. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix] spacewander closed issue #2505: CI: yaml style

2020-10-29 Thread GitBox
spacewander closed issue #2505: URL: https://github.com/apache/apisix/issues/2505 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] spacewander merged pull request #2524: CI : yaml style

2020-10-29 Thread GitBox
spacewander merged pull request #2524: URL: https://github.com/apache/apisix/pull/2524 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: CI : better yaml style

2020-10-29 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 254952c CI : better yaml style 254952c is

[apisix] branch master updated: change: rename APIX_WORKER_PROCESSES to APISIX_WORKER_PROCESSES (#2552)

2020-10-29 Thread juzhiyuan
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.git The following commit(s) were added to refs/heads/master by this push: new b6eb6f4 change: rename

[GitHub] [apisix] juzhiyuan merged pull request #2552: change: rename APIX_WORKER_PROCESSES to APISIX_WORKER_PROCESSES

2020-10-29 Thread GitBox
juzhiyuan merged pull request #2552: URL: https://github.com/apache/apisix/pull/2552 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

[GitHub] [apisix] juzhiyuan closed issue #2517: Use APISIX_WORKER_PROCESSES instead of APIX_WORKER_PROCESSES?

2020-10-29 Thread GitBox
juzhiyuan closed issue #2517: URL: https://github.com/apache/apisix/issues/2517 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] moonming commented on pull request #2499: bugfix: fix zipkin plugin error when used with limit count plugin

2020-10-29 Thread GitBox
moonming commented on pull request #2499: URL: https://github.com/apache/apisix/pull/2499#issuecomment-719087839 Yes, we should fix it in zipkin plugin nic-chen 于2020年10月30日 周五上午7:46写道: > It's a difficult multiple choice question. > I think we could fix it in zipkin plugin,

[GitHub] [apisix] nic-chen commented on pull request #2552: change: rename APIX_WORKER_PROCESSES to APISIX_WORKER_PROCESSES

2020-10-29 Thread GitBox
nic-chen commented on pull request #2552: URL: https://github.com/apache/apisix/pull/2552#issuecomment-719086827 > We alse need to change manager-api's name to keep same style. > > @nic-chen OK This is an

[GitHub] [apisix] nic-chen commented on pull request #2499: bugfix: fix zipkin plugin error when used with limit count plugin

2020-10-29 Thread GitBox
nic-chen commented on pull request #2499: URL: https://github.com/apache/apisix/pull/2499#issuecomment-719086701 It's a difficult multiple choice question. I think we could fix it in zipkin plugin, add some judge code to make sure the span is exits. And let user custom its priority.

[GitHub] [apisix] nic-chen commented on issue #2377: bug:Use zipkin plugin and limit-req plugin together, zipkin plugin can not work when is limit

2020-10-29 Thread GitBox
nic-chen commented on issue #2377: URL: https://github.com/apache/apisix/issues/2377#issuecomment-719080867 > > if we need it, we can set a higher priority. https://github.com/apache/apisix/blob/master/apisix/plugins/zipkin.lua#L51 > > > > > > @moonming please confirm if

[GitHub] [apisix] moonming commented on pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
moonming commented on pull request #2548: URL: https://github.com/apache/apisix/pull/2548#issuecomment-719071635 > I think we can also use the etcd proxy localily as an interim solution > > > > [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd] Do you means add

[GitHub] [apisix] juzhiyuan commented on a change in pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
juzhiyuan commented on a change in pull request #2548: URL: https://github.com/apache/apisix/pull/2548#discussion_r514614005 ## File path: apisix/core/config_etcd.lua ## @@ -472,6 +472,10 @@ function _M.new(key, opts) etcd_conf.protocol = "v3" etcd_conf.api_prefix =

[GitHub] [apisix] Miss-you edited a comment on pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
Miss-you edited a comment on pull request #2548: URL: https://github.com/apache/apisix/pull/2548#issuecomment-718889945 I think we can also use the etcd proxy localily as an interim solution [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd]

[GitHub] [apisix] Miss-you commented on pull request #2548: feat: support TLS connection with etcd.

2020-10-29 Thread GitBox
Miss-you commented on pull request #2548: URL: https://github.com/apache/apisix/pull/2548#issuecomment-718889945 I think we can also use the etcd proxy localily. [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd]

[GitHub] [apisix] Miss-you edited a comment on issue #2551: apisix 2.X 支持etcd ssl配置吗,想共用k8s集群中的etcd

2020-10-29 Thread GitBox
Miss-you edited a comment on issue #2551: URL: https://github.com/apache/apisix/issues/2551#issuecomment-71623 I think you can use the etcd proxy localily. [Apache APISIX -> etcd proxy(cert) ]--http mTLS-->[etcd] ping #yangzcy423911

  1   2   3   >