[GitHub] [apisix-website] juzhiyuan commented on pull request #73: feat: added desc

juzhiyuan commented on pull request #73: URL: https://github.com/apache/apisix-website/pull/73#issuecomment-692485620 ping @LiteSun This is an automated message from the Apache Git Service. To respond to the message, please

[apisix-dashboard] 01/01: feat: added new plugin dependency

This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch feat-plugin in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git commit 2329b1dbe4267a2e38d71fb978701ed0d609311f Author: juzhiyuan AuthorDate: Tue Sep 15 14:38:36 2020 +0800

[GitHub] [apisix] membphis commented on a change in pull request #2230: bug: removed default access token for Admin API

membphis commented on a change in pull request #2230: URL: https://github.com/apache/apisix/pull/2230#discussion_r488435973 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix-dashboard] liuxiran commented on issue #474: Publish route failed

liuxiran commented on issue #474: URL: https://github.com/apache/apisix-dashboard/issues/474#issuecomment-692525992 something like mysql has not been updated, Will mysql be updated synchronously when the demo environment updated? @juzhiyuan

[GitHub] [apisix] MrJLuo opened a new issue #2232: request help: dashboard 404

MrJLuo opened a new issue #2232: URL: https://github.com/apache/apisix/issues/2232 ### Issue description 

[GitHub] [apisix] Yiyiyimu commented on issue #2227: CLI: Check the etcd version in CLI. The version must be 3.4 or greater.

Yiyiyimu commented on issue #2227: URL: https://github.com/apache/apisix/issues/2227#issuecomment-69218 Sorry @tokers I just noticed you pick up this issue. I fixed it in the [new commit](https://github.com/apache/apisix/pull/2036/commits/58e8401a24ceed0313702903103ea4e608128021). But

[apisix-website] branch asf-site updated: Deploy to GitHub pages

This is an automated email from the ASF dual-hosted git repository. github-bot pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 885dadd Deploy to GitHub pages

[GitHub] [apisix-website] juzhiyuan merged pull request #73: feat: added desc

juzhiyuan merged pull request #73: URL: https://github.com/apache/apisix-website/pull/73 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-website] juzhiyuan closed issue #72: add reference to "Apache APISIX software"

juzhiyuan closed issue #72: URL: https://github.com/apache/apisix-website/issues/72 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

[apisix-website] branch master updated: feat: added desc (#73)

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 fe9bd58 feat: added desc (#73)

[GitHub] [apisix] membphis commented on pull request #2230: bug: removed default access token for Admin API

membphis commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692528591 > why `**` is invalid? Is this hard-coded in the code? you can take a look at this source code in CLI:

[GitHub] [apisix] Firstsawyou commented on a change in pull request #2192: feat: add AK/SK auth plugin

Firstsawyou commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488476189 ## File path: t/plugin/hmac-auth.t ## @@ -0,0 +1,596 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] [apisix] membphis closed pull request #2092: bug: removed default access token for Admin API.

membphis closed pull request #2092: URL: https://github.com/apache/apisix/pull/2092 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] membphis commented on pull request #2092: bug: removed default access token for Admin API.

membphis commented on pull request #2092: URL: https://github.com/apache/apisix/pull/2092#issuecomment-692493280 I have to close this PR, this PR contains other people's commit logs. This is an automated message from the

[GitHub] [apisix-dashboard] juzhiyuan closed issue #162: Allow set filter_func in router configuration page

juzhiyuan closed issue #162: URL: https://github.com/apache/apisix-dashboard/issues/162 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] membphis commented on pull request #2092: bug: removed default access token for Admin API.

membphis commented on pull request #2092: URL: https://github.com/apache/apisix/pull/2092#issuecomment-692503939 @moonming I created a new PR https://github.com/apache/apisix/pull/2230, with the right git commit log. welcome to review the new PR.

[GitHub] [apisix] moonming commented on a change in pull request #2230: bug: removed default access token for Admin API

moonming commented on a change in pull request #2230: URL: https://github.com/apache/apisix/pull/2230#discussion_r488430805 ## File path: FAQ.md ## @@ -80,7 +80,7 @@ An example, `foo.com/product/index.html?id=204=2`, gray release based on `i here is the way: ```shell

[GitHub] [apisix] moonming commented on pull request #2230: bug: removed default access token for Admin API

moonming commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692518089 why `**` is invalid? Is this hard-coded in the code? Thanks, Ming Wen Twitter: _WenMing YuanSheng Wang 于2020年9月15日周二 下午3:15写道： >

[GitHub] [apisix] moonming commented on pull request #2230: bug: removed default access token for Admin API

moonming commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692560517 No, `YOUR_OWN_API_TOKEN` is just hint in doc, not api token itself. Thanks, Ming Wen Twitter: _WenMing YuanSheng Wang 于2020年9月15日周二 下午4:29写道： >

[GitHub] [apisix] gy09535 edited a comment on pull request #2177: feature: support plugin for aliyun log service

gy09535 edited a comment on pull request #2177: URL: https://github.com/apache/apisix/pull/2177#issuecomment-692061891 > @gy09535 Apache APISIX already has the syslog plugin, can these two plugins be combined into one? For combined into one I think the aliyun has some special

[GitHub] [apisix] gy09535 edited a comment on pull request #2177: feature: support plugin for aliyun log service

gy09535 edited a comment on pull request #2177: URL: https://github.com/apache/apisix/pull/2177#issuecomment-692061891 > @gy09535 Apache APISIX already has the syslog plugin, can these two plugins be combined into one? For combined into one I think the aliyun has some special

[GitHub] [apisix] membphis opened a new pull request #2230: bug: removed default access token for Admin API

membphis opened a new pull request #2230: URL: https://github.com/apache/apisix/pull/2230 ### What this PR does / why we need it: related PR: #2092 ### Pre-submission checklist: * [ ] Did you explain what problem does this PR solve? Or what new features have

[GitHub] [apisix-dashboard] juzhiyuan closed issue #437: Feature：route publish management

juzhiyuan closed issue #437: URL: https://github.com/apache/apisix-dashboard/issues/437 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] moonming closed issue #449: CI: add tools to check ASF headers in code files

moonming closed issue #449: URL: https://github.com/apache/apisix-dashboard/issues/449 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-dashboard] juzhiyuan closed issue #431: Feature: route group

juzhiyuan closed issue #431: URL: https://github.com/apache/apisix-dashboard/issues/431 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 #437: Feature：route publish management

juzhiyuan commented on issue #437: URL: https://github.com/apache/apisix-dashboard/issues/437#issuecomment-692502969 > @juzhiyuan > I don't know what progress has been made in the replacement of database. At present, the pr #450 related to this is use mysql. Is this only regarded as a

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #431: Feature: route group

juzhiyuan commented on issue #431: URL: https://github.com/apache/apisix-dashboard/issues/431#issuecomment-692503187 merged and closed. This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #444: Proposal: Improve the Route module

juzhiyuan commented on issue #444: URL: https://github.com/apache/apisix-dashboard/issues/444#issuecomment-692502572 cc @LiteSun Need some time to work on this, I just added the latest plugin. This is an automated message

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #434: Requirements for refactoring the Dashboard with Manager API

juzhiyuan commented on issue #434: URL: https://github.com/apache/apisix-dashboard/issues/434#issuecomment-692502807 Any update? This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [apisix-dashboard] moonming commented on issue #449: CI: add tools to check ASF headers in code files

moonming commented on issue #449: URL: https://github.com/apache/apisix-dashboard/issues/449#issuecomment-692502897 yes, fixed by https://github.com/apache/apisix-dashboard/pull/465 This is an automated message from the

[GitHub] [apisix] Firstsawyou commented on a change in pull request #2192: feat: add AK/SK auth plugin

Firstsawyou commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488482999 ## File path: t/plugin/hmac-auth.t ## @@ -0,0 +1,596 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor

[GitHub] [apisix-dashboard] juzhiyuan opened a new issue #474: Publish route failed

juzhiyuan opened a new issue #474: URL: https://github.com/apache/apisix-dashboard/issues/474 Please answer these questions before submitting your issue. - Why do you submit this issue? - [ ] Question or discussion - [x] Bug - [ ] Requirements - [ ] Feature or performance

[GitHub] [apisix] moonming commented on a change in pull request #2230: bug: removed default access token for Admin API

moonming commented on a change in pull request #2230: URL: https://github.com/apache/apisix/pull/2230#discussion_r488454473 ## File path: bin/apisix ## @@ -643,6 +643,34 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix-dashboard] moonming commented on issue #473: PluginModule: v2

moonming commented on issue #473: URL: https://github.com/apache/apisix-dashboard/issues/473#issuecomment-692548163 plugin categories and priority: - limit traffic: limit-req, limit-conn, limit-conn - observability: prometheus, skywalking, zipkin, request-id - security:

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488485180 ## File path: apisix/core/config_etcd.lua ## @@ -92,17 +105,29 @@ local function waitdir(etcd_cli, key, modified_index, timeout) return nil,

[GitHub] [apisix] membphis commented on a change in pull request #2230: bug: removed default access token for Admin API

membphis commented on a change in pull request #2230: URL: https://github.com/apache/apisix/pull/2230#discussion_r488483897 ## File path: bin/apisix ## @@ -643,6 +643,34 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488484262 ## File path: apisix/core/config_etcd.lua ## @@ -92,17 +105,29 @@ local function waitdir(etcd_cli, key, modified_index, timeout) return nil,

[apisix-dashboard] branch feat-plugin updated (2329b1d -> b469b8b)

This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch feat-plugin in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. from 2329b1d feat: added new plugin dependency add a253fd8 feat: omit shadow var add

[GitHub] [apisix] moonming opened a new pull request #2231: change: remove dashboard sub module.

moonming opened a new pull request #2231: URL: https://github.com/apache/apisix/pull/2231 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] moonming commented on issue #2232: request help: dashboard 404

moonming commented on issue #2232: URL: https://github.com/apache/apisix/issues/2232#issuecomment-692539013 Only 127.0.0.1 can access the dashboard by default, you can modify the ip limit yourself This is an automated

[GitHub] [apisix] moonming commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

moonming commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r487762501 ## File path: apisix/core/config_etcd.lua ## @@ -92,17 +105,29 @@ local function waitdir(etcd_cli, key, modified_index, timeout) return nil,

[GitHub] [apisix] membphis commented on a change in pull request #2092: bug: removed default access token for Admin API.

membphis commented on a change in pull request #2092: URL: https://github.com/apache/apisix/pull/2092#discussion_r488413639 ## File path: bin/apisix ## @@ -637,6 +637,45 @@ local function read_yaml_conf() merge_conf(default_conf, user_conf) end +-- check

[GitHub] [apisix-dashboard] juzhiyuan opened a new pull request #475: feat: added new plugin dependency

juzhiyuan opened a new pull request #475: URL: https://github.com/apache/apisix-dashboard/pull/475 Please answer these questions before submitting a pull request - Why submit this pull request? - [ ] Bug fix - [ ] New feature provided - [x] Improve performance -

[apisix-dashboard] branch feat-plugin created (now 2329b1d)

This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a change to branch feat-plugin in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git. at 2329b1d feat: added new plugin dependency This branch includes the following new commits:

[GitHub] [apisix] moonming commented on pull request #2230: bug: removed default access token for Admin API

moonming commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692515133 YOUR_OWN_API_TOKEN Thanks, Ming Wen Twitter: _WenMing YuanSheng Wang 于2020年9月15日周二 下午3:09写道： > *@membphis* commented on this pull request. >

[GitHub] [apisix] moonming commented on pull request #2230: bug: removed default access token for Admin API

moonming commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692515707 I insist on this, if automatic generation token is not possible, I would rather not remove the default token Thanks, Ming Wen Twitter: _WenMing

[GitHub] [apisix] membphis edited a comment on pull request #2230: bug: removed default access token for Admin API

membphis edited a comment on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692513326 > Users will encounter errors when running `apisix start`, and this experience is bad. I think it is good enough for now. We have told the user how to generate the

[GitHub] [apisix-dashboard] juzhiyuan closed issue #452: An error is reported when creating the second route：RouteGroup name is duplicate

juzhiyuan closed issue #452: URL: https://github.com/apache/apisix-dashboard/issues/452 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 #449: CI: add tools to check ASF headers in code files

juzhiyuan commented on issue #449: URL: https://github.com/apache/apisix-dashboard/issues/449#issuecomment-692502318 Done? This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #452: An error is reported when creating the second route：RouteGroup name is duplicate

juzhiyuan commented on issue #452: URL: https://github.com/apache/apisix-dashboard/issues/452#issuecomment-692502242 merged, feel free to reopen this one. This is an automated message from the Apache Git Service. To respond

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #466: bug: a related to route basic-auth consumer should not be deleted

juzhiyuan commented on issue #466: URL: https://github.com/apache/apisix-dashboard/issues/466#issuecomment-692501456 ping @membphis @nic-chen This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-dashboard] juzhiyuan closed issue #464: request help: 401 when using apisix in k8s

juzhiyuan closed issue #464: URL: https://github.com/apache/apisix-dashboard/issues/464 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 #458: bug: When creating a route, host should not be required

juzhiyuan commented on issue #458: URL: https://github.com/apache/apisix-dashboard/issues/458#issuecomment-692502040 assigned to @LiteSun This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix] moonming commented on issue #2227: CLI: Check the etcd version in CLI. The version must be 3.4 or greater.

moonming commented on issue #2227: URL: https://github.com/apache/apisix/issues/2227#issuecomment-692510383 sure, assigned to you @tokers This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [apisix] membphis commented on pull request #2230: bug: removed default access token for Admin API

membphis commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692516876 > YOUR_OWN_API_TOKEN that is a bad name, because of the `YOUR_OWN_API_TOKEN` is a valid Admin API key. the `**` is a invalid Admin API key. > valid

[GitHub] [apisix] Yiyiyimu edited a comment on pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu edited a comment on pull request #2036: URL: https://github.com/apache/apisix/pull/2036#issuecomment-692553222 > > one more thing, we need to check the `etcd` version in `bin/apisix`, confirm the `etcd` version `>= 3.4` . > > we can fix this in a new PR, here is the related

[GitHub] [apisix] Yiyiyimu commented on pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on pull request #2036: URL: https://github.com/apache/apisix/pull/2036#issuecomment-692553222 > > one more thing, we need to check the `etcd` version in `bin/apisix`, confirm the `etcd` version `>= 3.4` . > > we can fix this in a new PR, here is the related issue:

[GitHub] [apisix] moonming commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

moonming commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488484438 ## File path: apisix/core/etcd.lua ## @@ -44,24 +48,134 @@ end _M.new = new +local function kvs_to_node(kvs) +local node = {} +node.key =

[GitHub] [apisix] membphis commented on pull request #2230: bug: removed default access token for Admin API

membphis commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692558053 > I insist on this, if automatic generation token is not possible, I would rather not remove the default token I think they are two different questions. Let us discuss it

[GitHub] [apisix-dashboard] liuxiran commented on issue #470: Proposal: Health Checks for Upstream

liuxiran commented on issue #470: URL: https://github.com/apache/apisix-dashboard/issues/470#issuecomment-692600340 > hi @liuxiran, would you like to implement this feature (if you have free time)? yes, waiting for my pr :)(later this week)

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #470: Proposal: Health Checks for Upstream

juzhiyuan commented on issue #470: URL: https://github.com/apache/apisix-dashboard/issues/470#issuecomment-692505266 hi @liuxiran, would you like to implement this feature (if you have free time)? This is an automated

[GitHub] [apisix] membphis commented on pull request #2230: bug: removed default access token for Admin API

membphis commented on pull request #2230: URL: https://github.com/apache/apisix/pull/2230#issuecomment-692513326 > Users will encounter errors when running `apisix start`, and this experience is bad. I think it is enough for now. We have told the user how to generate the Admin API

[GitHub] [apisix] tokers opened a new pull request #2233: change: check etcd cluster version when init_etcd

tokers opened a new pull request #2233: URL: https://github.com/apache/apisix/pull/2233 Signed-off-by: tokers ### 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

[GitHub] [apisix] fankeke commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirome

fankeke commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-692671931 Can we divide the 1+ million block urls into groups according to their hosts? === @spacewander the whole 1+million urls only for one host(upstream) :)

[GitHub] [apisix] moonming closed issue #2176: request help: How to configure timeout settings?

moonming closed issue #2176: URL: https://github.com/apache/apisix/issues/2176 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 #2233: change: check etcd cluster version when init_etcd

moonming commented on pull request #2233: URL: https://github.com/apache/apisix/pull/2233#issuecomment-692677880 the etcd in CI is 3.2, so failed: https://github.com/apache/apisix/pull/2233/checks?check_run_id=1116987808#step:6:288

[GitHub] [apisix] Miss-you commented on a change in pull request #2226: doc: added channel at slack and added new qq group.

Miss-you commented on a change in pull request #2226: URL: https://github.com/apache/apisix/pull/2226#discussion_r488705032 ## File path: README_CN.md ## @@ -23,9 +23,11 @@ [![Build

[GitHub] [apisix] nic-chen commented on pull request #2225: move etcd cluster in docker for CI

nic-chen commented on pull request #2225: URL: https://github.com/apache/apisix/pull/2225#issuecomment-692717762 I think we could run ETCD as a service in github actions. [doc about github actions

[GitHub] [apisix] nic-chen commented on a change in pull request #2192: feat: add AK/SK auth plugin

nic-chen commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488679450 ## File path: doc/zh-cn/plugins/hmac-auth.md ## @@ -0,0 +1,157 @@ + + +- [English](../../plugins/hmac-auth.md) + +# 目录 +- [**名字**](#名字) +- [**属性**](#属性)

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488557091 ## File path: apisix/plugins/hmac-auth.lua ## @@ -0,0 +1,303 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +--

[GitHub] [apisix] nic-chen commented on a change in pull request #2192: feat: add AK/SK auth plugin

nic-chen commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488680308 ## File path: doc/plugins/hmac-auth.md ## @@ -0,0 +1,151 @@ + + +- [中文](../zh-cn/plugins/hmac-auth.md) + +# Summary +- [**Name**](#name) +-

[apisix] branch master updated: feat: allow to limit client body size to protect the service (#2214)

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 b6354ec feat: allow to limit client body

[GitHub] [apisix] spacewander merged pull request #2214: feat: allow to limit client body size to protect the service

spacewander merged pull request #2214: URL: https://github.com/apache/apisix/pull/2214 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] moonming commented on pull request #2225: move etcd cluster in docker for CI

moonming commented on pull request #2225: URL: https://github.com/apache/apisix/pull/2225#issuecomment-692752316 Cool, agreed nic-chen 于2020年9月15日 周二下午9:34写道： > > > I think we could run ETCD as a service in github actions. doc about > github actions services >

[GitHub] [apisix] juzhiyuan commented on a change in pull request #2226: doc: added channel at slack and added new qq group.

juzhiyuan commented on a change in pull request #2226: URL: https://github.com/apache/apisix/pull/2226#discussion_r488623153 ## File path: README.md ## @@ -23,9 +23,11 @@ [![Build

[GitHub] [apisix] moonming commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirom

moonming commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-692680602 > Can we divide the 1+ million block urls into groups according to their hosts? > > === > > @spacewander the whole 1+million urls only for one host(upstream) :)

[GitHub] [apisix] xyz2b commented on issue #2176: request help: How to configure timeout settings?

xyz2b commented on issue #2176: URL: https://github.com/apache/apisix/issues/2176#issuecomment-692622830 ok, thanks This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

[GitHub] [apisix-dashboard] nic-chen commented on issue #474: Publish route failed

nic-chen commented on issue #474: URL: https://github.com/apache/apisix-dashboard/issues/474#issuecomment-692708138 I had updated mysql, you could have a try again. @juzhiyuan @liuxiran This is an automated message from

[GitHub] [apisix] Firstsawyou commented on a change in pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

Firstsawyou commented on a change in pull request #2209: URL: https://github.com/apache/apisix/pull/2209#discussion_r488599115 ## File path: t/admin/services.t ## @@ -1085,3 +1085,50 @@ GET /t --- error_code: 400 --- no_error_log [error] + + + +=== TEST 31: set service(id:

[GitHub] [apisix] moonming commented on issue #2188: request help: url_block.lua plugin: Is there any performance bottleneck if there are many block rules, say 1000000 url rules in production envirom

moonming commented on issue #2188: URL: https://github.com/apache/apisix/issues/2188#issuecomment-692679685 > Improving performance mainly depends on the quality of the rules. > > For runtime selection, if better performance is required, streaming matching such as hyperscan is

[GitHub] [apisix-dashboard] juzhiyuan commented on issue #474: Publish route failed

juzhiyuan commented on issue #474: URL: https://github.com/apache/apisix-dashboard/issues/474#issuecomment-692679130 not sure, cc @nic-chen This is an automated message from the Apache Git Service. To respond to the

[GitHub] [apisix-control-plane] Firstsawyou opened a new pull request #26: CI: add tools to check ASF headers in code files #25

Firstsawyou opened a new pull request #26: URL: https://github.com/apache/apisix-control-plane/pull/26 add license check. This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488557288 ## File path: doc/plugins/hmac-auth.md ## @@ -0,0 +1,151 @@ + + +- [中文](../zh-cn/plugins/hmac-auth.md) + +# Summary +- [**Name**](#name) +-

[GitHub] [apisix] membphis commented on a change in pull request #2192: feat: add AK/SK auth plugin

membphis commented on a change in pull request #2192: URL: https://github.com/apache/apisix/pull/2192#discussion_r488558126 ## File path: t/plugin/hmac-auth.t ## @@ -0,0 +1,596 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license

[GitHub] [apisix] nic-chen commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

nic-chen commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488672188 ## File path: utils/install-etcd.sh ## @@ -0,0 +1,24 @@ +#!/bin/sh + +# +# Licensed to the Apache Software Foundation (ASF) under one or more +#

[GitHub] [apisix] Yiyiyimu edited a comment on issue #2227: CLI: Check the etcd version in CLI. The version must be 3.4 or greater.

Yiyiyimu edited a comment on issue #2227: URL: https://github.com/apache/apisix/issues/2227#issuecomment-69218 Sorry @tokers I just noticed you pick up this issue. I fixed it in the [new

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488745722 ## File path: apisix/core/etcd.lua ## @@ -44,24 +48,134 @@ end _M.new = new +local function kvs_to_node(kvs) +local node = {} +node.key =

[GitHub] [apisix-dashboard] liuxiran commented on issue #474: Publish route failed

liuxiran commented on issue #474: URL: https://github.com/apache/apisix-dashboard/issues/474#issuecomment-692790666 > I had updated mysql, you could have a try again. @juzhiyuan @liuxiran passed  This is an

[GitHub] [apisix-dashboard] liuxiran commented on issue #469: Feature: debug a route online

liuxiran commented on issue #469: URL: https://github.com/apache/apisix-dashboard/issues/469#issuecomment-692861654 After research on swagger-ui, and tried to introduce [swagger-ui-react](https://www.npmjs.com/package/swagger-ui-react ) in our project, I found it would be a feasible

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488751665 ## File path: apisix/core/etcd.lua ## @@ -44,24 +49,144 @@ end _M.new = new +local function kvs_to_node(kvs) +local node = {} +node.key =

[GitHub] [apisix] Yiyiyimu commented on pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on pull request #2036: URL: https://github.com/apache/apisix/pull/2036#issuecomment-69271 Currently the log produced by lua-resty-etcd would show message without base64 decode. For example ``` 2020/09/15 23:18:52 [info] 1964#1964: *2 [lua] v3.lua:284: set(): v3

[GitHub] [apisix] Firstsawyou commented on a change in pull request #2209: feature: allow empty `service` object, it does not contain any `upstream` and `plugin`.

Firstsawyou commented on a change in pull request #2209: URL: https://github.com/apache/apisix/pull/2209#discussion_r488819427 ## File path: t/admin/schema.t ## @@ -37,10 +37,47 @@ qr/"plugins":\{"type":"object"}/ === TEST 2: get service schema +--- config +location

[GitHub] [apisix] Yiyiyimu commented on a change in pull request #2036: feature: support etcd v3, by mocking v2 API

Yiyiyimu commented on a change in pull request #2036: URL: https://github.com/apache/apisix/pull/2036#discussion_r488739432 ## File path: apisix/core/etcd.lua ## @@ -44,24 +48,134 @@ end _M.new = new +local function kvs_to_node(kvs) +local node = {} +node.key =

[GitHub] [apisix] moonming commented on pull request #2233: change: check etcd cluster version when init_etcd

moonming commented on pull request #2233: URL: https://github.com/apache/apisix/pull/2233#issuecomment-693135758 nice! we can fix it in this PR :) Thanks, Ming Wen Twitter: _WenMing Alex Zhang 于2020年9月16日周三 上午10:36写道： > @moonming

[GitHub] [apisix] tokers opened a new issue #2234: bug: etcd_version in ./bin/apisix is not defined

tokers opened a new issue #2234: URL: https://github.com/apache/apisix/issues/2234 ### Issue description The variable `etcd_version` in ./bin/apisix is not defined, comment see [here](https://github.com/apache/apisix/pull/2036/files#diff-62edadffc237f13dc28a694080d293a8R918).

[GitHub] [apisix-dashboard] corffee opened a new issue #476: How to configure a TCP upstream by dashboard?

corffee opened a new issue #476: URL: https://github.com/apache/apisix-dashboard/issues/476 Please answer these questions before submitting your issue. - Why do you submit this issue? - [ 1] Question or discussion - [ ] Bug - [ ] Requirements - [ ] Feature or performance

[GitHub] [apisix] tokers commented on pull request #2233: change: check etcd cluster version when init_etcd

tokers commented on pull request #2233: URL: https://github.com/apache/apisix/pull/2233#issuecomment-693130100 @moonming OK, let me rebase my branch. By the way, i think it's better to check out the CHANGELOG of etcd, AKAIK there are some bugs in 3.x series releases, but i'm not sure the

[GitHub] [apisix] membphis commented on a change in pull request #2221: send file contents and not file name

membphis commented on a change in pull request #2221: URL: https://github.com/apache/apisix/pull/2221#discussion_r488333965 ## File path: apisix/utils/log-util.lua ## @@ -64,7 +64,11 @@ local function get_full_log(ngx, conf) else local body_file =

[apisix-dashboard] branch master updated: fix: host should not be required (#477)

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-dashboard.git The following commit(s) were added to refs/heads/master by this push: new cf9d5ec fix: host should not be

[GitHub] [apisix-dashboard] juzhiyuan merged pull request #477: fix: host should not be required

juzhiyuan merged pull request #477: URL: https://github.com/apache/apisix-dashboard/pull/477 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

  1   2   >