liuximu opened a new issue #2198:
URL: https://github.com/apache/apisix/issues/2198
### Issue description
求官方确认一下：
apache/apisix 包括两个职责，有两种部署方式：
1 被放到线上部署成API-Gateway集群，关闭api相关的接口，作为数据面
2 被放到内网部署成管理平台，开源被 apisix-dashboard 调用完成配置的变更
是这样子吗？谢谢
-
y934317258 opened a new issue #2199:
URL: https://github.com/apache/apisix/issues/2199
### Issue description
The parameter meaning of JWT auth plug-in. What are the functions of
parameter configuration in consumer and routing
 were added to refs/heads/master by this push:
new 47d39e7 bugfix: serverless plugin not work in
moonming merged pull request #454:
URL: https://github.com/apache/apisix-dashboard/pull/454
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 t
This is an automated email from the ASF dual-hosted git repository.
wenming pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git.
from b93388f feat: preview pluginchart (#456)
add 54f7a98 add tools to check ASF headers in code files.
moonming commented on pull request #2193:
URL: https://github.com/apache/apisix/pull/2193#issuecomment-689408350
```
apisix/plugins/zipkin/codec.lua:41:13: value assigned to variable sample
is unused
```
please remove `sample` variable, thx
---
moonming removed a comment on pull request #2193:
URL: https://github.com/apache/apisix/pull/2193#issuecomment-689408350
```
apisix/plugins/zipkin/codec.lua:41:13: value assigned to variable sample
is unused
```
please remove `sample` variable, thx
---
juzhiyuan commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689432449
@liuxiran CI failed, please take a look
This is an automated message from the Apache Git Service.
To re
liuxiran commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689433677
> @liuxiran CI failed, please take a look
fixing :)
This is an automated message from the Apache
tengzhinei opened a new issue #458:
URL: https://github.com/apache/apisix-dashboard/issues/458
### Issue description
When creating a route, host should not be required. We are a sass project.
Thousands of domain names point to the same system. Now, we are using Kong.
kong is not r
juzhiyuan commented on issue #458:
URL:
https://github.com/apache/apisix-dashboard/issues/458#issuecomment-689436642
cc @membphis @nic-chen so it's not needed? 🤔
This is an automated message from the Apache Git Service.
To
dickens7 commented on pull request #2193:
URL: https://github.com/apache/apisix/pull/2193#issuecomment-689441251
I think it’s not a good idea to remove the sample variable, I moved the
sample decision logic to create_tracer function. Keep upstream decided to
sample the request, we do too.
y934317258 opened a new issue #459:
URL: https://github.com/apache/apisix-dashboard/issues/459
Please answer these questions before submitting your issue.
- Why do you submit this issue?
- [ ] Question or discussion
- [ ] Bug
- [ ] Requirements
- [ ] Feature or performance
liuxiran opened a new pull request #460:
URL: https://github.com/apache/apisix-dashboard/pull/460
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bug fix
- [ ] New feature provided
- [ ] Improve performance
- Related
liuxiran commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689467089
the reason for ci failed is the same with
https://github.com/apache/apisix-dashboard/issues/452#issuecomment-688666125 ,
we updated the logic of routeGroup. this pr is d
spacewander commented on issue #2194:
URL: https://github.com/apache/apisix/issues/2194#issuecomment-689467983
Duplicate of #2006 ?
This is an automated message from the Apache Git Service.
To respond to the message, please l
juzhiyuan commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689468983
@liuxiran ok, so just waiting for others' review :DD
This is an automated message from the Apache Git S
liuxiran commented on a change in pull request #433:
URL: https://github.com/apache/apisix-dashboard/pull/433#discussion_r485506244
##
File path: src/pages/RouteGroup/components/Step1.tsx
##
@@ -0,0 +1,59 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or
liuxiran commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689477596
> @liuxiran ok, so just waiting for others' review :DD
thanks for your time~
This is an automated
juzhiyuan commented on pull request #433:
URL: https://github.com/apache/apisix-dashboard/pull/433#issuecomment-689477819
cc @LiteSun
This is an automated message from the Apache Git Service.
To respond to the message, pleas
hityc2019 opened a new issue #74:
URL: https://github.com/apache/apisix-docker/issues/74
```
# docker-compose -p docker-apisix up -d
Creating network "docker-apisix_apisix" with driver "bridge"
Creating docker-apisix_etcd_1 ... done
Creating docker-apisix_web1_1 ... done
Creat
membphis commented on a change in pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#discussion_r485675296
##
File path: bin/apisix
##
@@ -940,6 +940,22 @@ function _M.start(...)
end
end
+local argparse = require "argparse"
+local parser
membphis commented on issue #74:
URL: https://github.com/apache/apisix-docker/issues/74#issuecomment-689617716
@gxthrj ping
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
membphis commented on issue #2169:
URL: https://github.com/apache/apisix/issues/2169#issuecomment-689621640
I think we can send the log to aliyun sls is an easy way, and that is a
stable way.
use `lua-resty-logger-socket` is a better way.
---
membphis commented on issue #2169:
URL: https://github.com/apache/apisix/issues/2169#issuecomment-689623663
another way, set `syslog` server in `conf/config.yaml` is another good method
This is an automated message from the A
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a change to branch juzhiyuan-patch-1
in repository https://gitbox.apache.org/repos/asf/apisix.git.
at 9e5b3b1 Update health-check.md
This branch includes the following new commits:
new 9e5b3b1 Upd
This is an automated email from the ASF dual-hosted git repository.
juzhiyuan pushed a commit to branch juzhiyuan-patch-1
in repository https://gitbox.apache.org/repos/asf/apisix.git
commit 9e5b3b141ff25d1157c954c022fdc212a707b2c0
Author: 琚致远
AuthorDate: Thu Sep 10 00:07:51 2020 +0800
Updat
juzhiyuan opened a new pull request #2200:
URL: https://github.com/apache/apisix/pull/2200
### What this PR does / why we need it:
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been added?
* [ ] Have y
swayamraina commented on issue #2196:
URL: https://github.com/apache/apisix/issues/2196#issuecomment-689745070
Hi @membphis
I would like to pick this up. Can you describe what exactly is required here?
This is an automate
moonming commented on a change in pull request #454:
URL: https://github.com/apache/apisix-dashboard/pull/454#discussion_r485994194
##
File path: .actions/ASF-Release.cfg
##
@@ -0,0 +1,105 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
contributor
+#
moonming commented on a change in pull request #454:
URL: https://github.com/apache/apisix-dashboard/pull/454#discussion_r485994194
##
File path: .actions/ASF-Release.cfg
##
@@ -0,0 +1,105 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
contributor
+#
nic-chen commented on a change in pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#discussion_r486000136
##
File path: api/route/route_test.go
##
@@ -0,0 +1,51 @@
+package route
+
+import (
+ "github.com/apisix/manager-api/conf"
+ "github.
nic-chen commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689904687
api ci not pass
This is an automated message from the Apache Git Service.
To respond to the message, ple
nic-chen commented on pull request #460:
URL: https://github.com/apache/apisix-dashboard/pull/460#issuecomment-689920798
need test cases for changes.
This is an automated message from the Apache Git Service.
To respond to the
juzhiyuan commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689924915
> the reason for ci failed is the same with [#452
(comment)](https://github.com/apache/apisix-dashboard/issues/452#issuecomment-688666125)
, we updated the logic of rou
y934317258 opened a new issue #2201:
URL: https://github.com/apache/apisix/issues/2201
### Issue description
use core.log Output log, output to which file
### Environment
* apisix version (cmd: `apisix version`):
* OS:
---
membphis commented on a change in pull request #2163:
URL: https://github.com/apache/apisix/pull/2163#discussion_r486031725
##
File path: apisix/http/router/radixtree_sni.lua
##
@@ -30,14 +30,53 @@ local ngx_decode_base64 = ngx.decode_base64
local ssl_certificates
local radix
linsir commented on issue #2201:
URL: https://github.com/apache/apisix/issues/2201#issuecomment-689943545
`logs/error.log`
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
tokers commented on pull request #2163:
URL: https://github.com/apache/apisix/pull/2163#issuecomment-689945812
@membphis It's really a problem if the number of certs is large, I will
refactor it with the LRU cache.
This is a
membphis opened a new issue #2202:
URL: https://github.com/apache/apisix/issues/2202
Sometimes users create an empty service (managed through API).
When needed later, add specific attributes, such as plug-ins, upstream, etc.
just remove this limitation and add test case to chec
y934317258 commented on issue #2201:
URL: https://github.com/apache/apisix/issues/2201#issuecomment-689950811
> `logs/error.log`
No matter what level of log, is it output to error.log ?
This is an automated message from th
Firstsawyou opened a new pull request #461:
URL: https://github.com/apache/apisix-dashboard/pull/461
Please answer these questions before submitting a pull request
- Why submit this pull request?
- [ ] Bug fix
- [ ] New feature provided
- [ ] Improve performance
- Rela
Yiyiyimu commented on pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#issuecomment-689952623
The newly imported lib [base64](https://github.com/iskolbin/lbase64) holds
MIT License and only 200 lines. So I think we could directly use it.
Yiyiyimu commented on a change in pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#discussion_r486040363
##
File path: rockspec/apisix-master-0.rockspec
##
@@ -52,6 +52,7 @@ dependencies = {
"lua-resty-kafka = 0.07",
"lua-resty-logger-socket = 2.0-0
Yiyiyimu edited a comment on pull request #2036:
URL: https://github.com/apache/apisix/pull/2036#issuecomment-689952623
The newly imported lib [base64](https://github.com/iskolbin/lbase64) is used
in `init_etcd` step to encode k/v send to etcd. Lib `base64` holds MIT License
and only 200 l
Yiyiyimu commented on a change in pull request #2101:
URL: https://github.com/apache/apisix/pull/2101#discussion_r484328359
##
File path: bin/apisix
##
@@ -940,6 +940,22 @@ function _M.start(...)
end
end
+local argparse = require "argparse"
+local parser
dickens7 opened a new issue #2203:
URL: https://github.com/apache/apisix/issues/2203
Is there a detailed version plan and release plan?
This is an automated message from the Apache Git Service.
To respond to the message, plea
idbeta opened a new issue #2204:
URL: https://github.com/apache/apisix/issues/2204
### Issue description
set twice route in the same uri，the second response should not include
“preNode”
curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY:
edd1c9f034335f136f87ad84b625c8f1'
nic-chen opened a new pull request #462:
URL: https://github.com/apache/apisix-dashboard/pull/462
Please answer these questions before submitting a pull request
- Why submit this pull request?
remove unnecessary license file for lib `dag-to-lua` which just is a
dependency not
Yiyiyimu opened a new pull request #2205:
URL: https://github.com/apache/apisix/pull/2205
### What this PR does / why we need it:
fix #2195
### Pre-submission checklist:
* [x] Did you explain what problem does this PR solve? Or what new features
have been added?
* [ ] H
moonming commented on a change in pull request #462:
URL: https://github.com/apache/apisix-dashboard/pull/462#discussion_r486059913
##
File path: LICENSE
##
@@ -224,5 +224,5 @@ Apache 2.0 licenses
The following components are provided under the Apache 2.0 License. See
project
Yiyiyimu commented on pull request #2205:
URL: https://github.com/apache/apisix/pull/2205#issuecomment-689978415
Chinese doc of plugins format needs to be unified。
This is an automated message from the Apache Git Service.
To
nic-chen commented on pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#issuecomment-689980972
> > the reason for ci failed is the same with [#452
(comment)](https://github.com/apache/apisix-dashboard/issues/452#issuecomment-688666125)
, we updated the logic of ro
liuxiran commented on a change in pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#discussion_r486072028
##
File path: api/route/route.go
##
@@ -34,11 +34,100 @@ func AppendRoute(r *gin.Engine) *gin.Engine {
r.GET("/apisix/admin/routes/:rid",
Firstsawyou commented on issue #2202:
URL: https://github.com/apache/apisix/issues/2202#issuecomment-689991034
Assigned to me.
This is an automated message from the Apache Git Service.
To respond to the message, please log on
liuxiran commented on a change in pull request #450:
URL: https://github.com/apache/apisix-dashboard/pull/450#discussion_r486077100
##
File path: api/script/db/schema.sql
##
@@ -18,7 +18,7 @@ CREATE TABLE `routes` (
`update_time` bigint(20),
`route_group_id` varchar(64) N
nic-chen commented on a change in pull request #462:
URL: https://github.com/apache/apisix-dashboard/pull/462#discussion_r486080122
##
File path: LICENSE
##
@@ -224,5 +224,5 @@ Apache 2.0 licenses
The following components are provided under the Apache 2.0 License. See
project
tokers edited a comment on pull request #2163:
URL: https://github.com/apache/apisix/pull/2163#issuecomment-690011030
@membphis changed.
This is an automated message from the Apache Git Service.
To respond to the message, ple
tokers commented on pull request #2163:
URL: https://github.com/apache/apisix/pull/2163#issuecomment-690011030
@membphis way changed.
This is an automated message from the Apache Git Service.
To respond to the message, please
ChelinTsien closed pull request #1960:
URL: https://github.com/apache/apisix/pull/1960
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
membphis commented on a change in pull request #2205:
URL: https://github.com/apache/apisix/pull/2205#discussion_r486103749
##
File path: doc/plugins/syslog.md
##
@@ -47,6 +47,7 @@ This will provide the ability to send Log data requests as
JSON objects.
|max_retry_times|optio
membphis commented on a change in pull request #2205:
URL: https://github.com/apache/apisix/pull/2205#discussion_r486104143
##
File path: doc/zh-cn/plugins/http-logger.md
##
@@ -46,6 +46,7 @@
| buffer_duration |可选的|必须先处理批次中最旧条目的最长期限（以秒为单位），默认值为 5|
| max_retry_count |可选的|从处理管道
68 matches
Mail list logo
