tzssangglass commented on issue #5723:
URL: https://github.com/apache/apisix/issues/5723#issuecomment-987934535
related to: https://github.com/apache/apisix/issues/5692
can you provide packet capture data for the network between apisix and etcd?
--
This is an automated message
tzssangglass commented on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987926989
> so maybe we are missing an http_server_location_configuration_snippet item.
Yes, that's what I mean about our lack of provision for users to set custom
variables. Would
juzhiyuan commented on pull request #5728:
URL: https://github.com/apache/apisix/pull/5728#issuecomment-987915927
@zuiyangqingzhou Hi, please also update this PR's description
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
juzhiyuan commented on a change in pull request #5728:
URL: https://github.com/apache/apisix/pull/5728#discussion_r763979941
##
File path: apisix/plugins/ua-restriction.lua
##
@@ -34,11 +35,11 @@ local schema = {
type = "boolean",
default = false,
tzssangglass commented on issue #5706:
URL: https://github.com/apache/apisix/issues/5706#issuecomment-987915215
Maybe we can enhance nacos discovery to make it work. Would anyone like to
give a detailed example? For example, from which interface of nacos do we get
information? What
juzhiyuan commented on a change in pull request #5727:
URL: https://github.com/apache/apisix/pull/5727#discussion_r763978097
##
File path: apisix/plugins/csrf.lua
##
@@ -0,0 +1,150 @@
+local core= require("apisix.core")
+local ngx = ngx
+local plugin_name =
juzhiyuan commented on pull request #5686:
URL: https://github.com/apache/apisix/pull/5686#issuecomment-987909316
> > IMO, we need to discuss in ML first, TeamCode is a commercial product
and requires user login to use
>
> Hi, I sent a mail to the address:
juzhiyuan commented on issue #5719:
URL: https://github.com/apache/apisix/issues/5719#issuecomment-987908278
Hi, could you please provide more steps so I could reproduce it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub
xuminwlt commented on issue #5706:
URL: https://github.com/apache/apisix/issues/5706#issuecomment-987902430
+1
need too.
--
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
xuminwlt opened a new issue #5729:
URL: https://github.com/apache/apisix/issues/5729
### Issue description
test-nginx project is clone like this
root/ is
tokers commented on issue #784:
URL:
https://github.com/apache/apisix-ingress-controller/issues/784#issuecomment-987894374
> ```
>
> apiVersion: apisix.apache.org/v2beta2
>
> kind: ApisixRoute
>
> metadata:
>
> name: frontend-route
>
> spec:
>
>
tokers commented on pull request #5718:
URL: https://github.com/apache/apisix/pull/5718#issuecomment-987891961
> > docs: refactor Installation Guide
>
>
>
> Hi @tzssangglass, I'm confused by this file Because this file's title is
`How to build Apache APISIX`, it's more
tokers commented on a change in pull request #774:
URL:
https://github.com/apache/apisix-ingress-controller/pull/774#discussion_r763953716
##
File path: tools.go
##
@@ -1,3 +1,4 @@
+//go:build tools
Review comment:
I have updated it but I don't have time to validate
tokers commented on a change in pull request #5728:
URL: https://github.com/apache/apisix/pull/5728#discussion_r763952504
##
File path: apisix/plugins/ua-restriction.lua
##
@@ -34,11 +35,11 @@ local schema = {
type = "boolean",
default = false,
juzhiyuan commented on pull request #5449:
URL: https://github.com/apache/apisix/pull/5449#issuecomment-987879068
> Author of the PR here, just to avoid confusion. I have deployed the above
fix already internally and its working there for me. From what I can see here
merging is currently
anderseknert commented on issue #5714:
URL: https://github.com/apache/apisix/issues/5714#issuecomment-987872952
Thanks for raising this @bzp2010 Let me know if there's any questions
along the way with regards to OPA, and I'd be happy to answer them :)
As for the suggested input
lxm commented on pull request #779:
URL:
https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-987855282
ping @tao12345666333 @gxthrj @lingsamuel
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
zlhgo edited a comment on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must be placed in the location block, so maybe we
are missing an
zlhgo edited a comment on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must be placed in the location block, so maybe we
are missing a
zlhgo edited a comment on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must be placed in the location section, so maybe
we are missing a
zlhgo edited a comment on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must place in the location section, so maybe we
are missing a
fbartels commented on pull request #5449:
URL: https://github.com/apache/apisix/pull/5449#issuecomment-987830406
Author of the PR here, just to avoid confusion. I have deployed the above
fix already internally and its working there for me. From what I can see here
merging is currently
codecov-commenter edited a comment on pull request #788:
URL:
https://github.com/apache/apisix-ingress-controller/pull/788#issuecomment-987664591
#
zlhgo edited a comment on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must place in the location section, so maybe we
are missing a
zlhgo commented on issue #5596:
URL: https://github.com/apache/apisix/issues/5596#issuecomment-987792571
i have written a plugin for this issue, but there is a problem that the `set
$upstream_name -;` directive must place in the location section, so maybe we
are missing a
juzhiyuan merged pull request #796:
URL: https://github.com/apache/apisix-website/pull/796
--
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:
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 aa81b30 docs: add Shivam committer
zuiyangqingzhou opened a new pull request #5728:
URL: https://github.com/apache/apisix/pull/5728
fix: [5724](https://github.com/apache/apisix/issues/5724)
### What this PR does / why we need it:
### Pre-submission checklist:
* [ ] Did you explain what
Baoyuantop opened a new pull request #5727:
URL: https://github.com/apache/apisix/pull/5727
### What this PR does / why we need it:
Add CSRF plugin for APISIX
Discuss in https://lists.apache.org/thread/ddj53svylg9s541vf2gpysxwgyrhwlzx
### Pre-submission checklist:
juzhiyuan edited a comment on pull request #5718:
URL: https://github.com/apache/apisix/pull/5718#issuecomment-987782053
Here has one issue about Make Test:
https://github.com/apache/apisix/issues/5726
--
This is an automated message from the Apache Git Service.
To respond to the
juzhiyuan commented on pull request #5718:
URL: https://github.com/apache/apisix/pull/5718#issuecomment-987782053
Here has one PR about Make Test: https://github.com/apache/apisix/issues/5726
--
This is an automated message from the Apache Git Service.
To respond to the message, please
tokers commented on issue #5723:
URL: https://github.com/apache/apisix/issues/5723#issuecomment-987776625
@zero-create Would you try to disable the `server-info` plugin to see what
happens.
--
This is an automated message from the Apache Git Service.
To respond to the message, please
tokers commented on issue #5723:
URL: https://github.com/apache/apisix/issues/5723#issuecomment-987776298
It seems that the lease grant requests cost a lot of time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
besich commented on issue #5726:
URL: https://github.com/apache/apisix/issues/5726#issuecomment-987775618
problems encountered in executing make test to run unit tests.
execute
```shell
make ci-env-up
make ci-env-up project_compose_ci=ci/pod/docker-compose.common.yml
make
besich opened a new issue #5726:
URL: https://github.com/apache/apisix/issues/5726
### Improved or typo fixed.
this document contains the relevant chapters “Step 4: Run Test Cases”
i suggest that this chapter be separated into a markdown file.
The reasons are as follows:
*
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
github-actions[bot] commented on pull request #2152:
URL: https://github.com/apache/apisix-dashboard/pull/2152#issuecomment-987766216
This pull request has been marked as stale due to 60 days of inactivity. It
will be closed in 4 weeks if no further activity occurs. If you think that's
github-actions[bot] commented on issue #2923:
URL: https://github.com/apache/apisix/issues/2923#issuecomment-987763713
This issue has been closed due to lack of activity. If you think that is
incorrect, or the issue requires additional review, you can revive the issue at
any time.
--
github-actions[bot] commented on pull request #4984:
URL: https://github.com/apache/apisix/pull/4984#issuecomment-987763764
This pull request/issue has been closed due to lack of activity. If you
think that is incorrect, or the pull request requires review, you can revive
the PR at any
github-actions[bot] closed pull request #4984:
URL: https://github.com/apache/apisix/pull/4984
--
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:
github-actions[bot] closed issue #2923:
URL: https://github.com/apache/apisix/issues/2923
--
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:
GhangZh closed issue #5708:
URL: https://github.com/apache/apisix/issues/5708
--
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:
GhangZh commented on issue #5708:
URL: https://github.com/apache/apisix/issues/5708#issuecomment-987752837
>
Thanks for the help, it was a problem with the environment variables that
didn't get this configuration
--
This is an automated message from the Apache Git Service.
To
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
okaybase commented on pull request #796:
URL: https://github.com/apache/apisix-website/pull/796#issuecomment-987736649
congratulation~
--
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
zuiyangqingzhou opened a new issue #5724:
URL: https://github.com/apache/apisix/issues/5724
### Issue description
The ngx.re.find method is used in the ua-restriction and uri-blocker
plug-ins, but there is no strict logical checking of the regex parameter, which
in some cases can
yzeng25 commented on a change in pull request #796:
URL: https://github.com/apache/apisix-website/pull/796#discussion_r763795682
##
File path:
website/i18n/zh/docusaurus-plugin-content-blog/2021/11/26/apache-apisix-committer-experience.md
##
@@ -0,0 +1,61 @@
+---
+title: "成为
spacewander closed issue #5417:
URL: https://github.com/apache/apisix/issues/5417
--
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:
spacewander merged pull request #5682:
URL: https://github.com/apache/apisix/pull/5682
--
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:
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 b85ebd4 fix(patch): add global
juzhiyuan commented on pull request #5718:
URL: https://github.com/apache/apisix/pull/5718#issuecomment-987723380
> docs: refactor Installation Guide
Hi @tzssangglass, I'm confused by this file Because this file's title is
`How to build Apache APISIX`, it's more like used in a
spacewander commented on issue #5708:
URL: https://github.com/apache/apisix/issues/5708#issuecomment-987721607
Can't reproduce on my side. My configuration is:
```
ext-plugin:
path_for_test: /tmp/runner.sock
#cmd: ["/home/lzx/git/apisix-go-plugin-runner/go-runner", "run",
shuaijinchao commented on a change in pull request #5715:
URL: https://github.com/apache/apisix/pull/5715#discussion_r763780260
##
File path: apisix/plugins/log-rotate.lua
##
@@ -194,18 +231,33 @@ local function rotate()
core.log.info("rotate interval:", interval)
shuaijinchao commented on a change in pull request #5715:
URL: https://github.com/apache/apisix/pull/5715#discussion_r763780260
##
File path: apisix/plugins/log-rotate.lua
##
@@ -194,18 +231,33 @@ local function rotate()
core.log.info("rotate interval:", interval)
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
lxm edited a comment on pull request #779:
URL:
https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-987688535
> re-run all jobs.
E2E tests failed with "check the ingress lb status is updated" this time,
maybe something wrong with test cases. I can pass this
lxm commented on pull request #779:
URL:
https://github.com/apache/apisix-ingress-controller/pull/779#issuecomment-987688535
> re-run all jobs.
E2E tests failed with "check the ingress lb status is updated" this time,
maybe something wrong with test cases. I can pass this case
yzeng25 commented on a change in pull request #796:
URL: https://github.com/apache/apisix-website/pull/796#discussion_r763743951
##
File path:
website/i18n/zh/docusaurus-plugin-content-blog/2021/11/26/apache-apisix-committer-experience.md
##
@@ -0,0 +1,61 @@
+---
+title: "成为
juzhiyuan commented on issue #698:
URL: https://github.com/apache/apisix-website/issues/698#issuecomment-987673394
> the master branch is correct
You're right, and let's lead users to use the Latest branch by default I
will close this issue soon.
--
This is an automated message
juzhiyuan closed issue #698:
URL: https://github.com/apache/apisix-website/issues/698
--
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:
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
juzhiyuan commented on a change in pull request #5718:
URL: https://github.com/apache/apisix/pull/5718#discussion_r763727963
##
File path: docs/en/latest/how-to-build.md
##
@@ -21,22 +21,29 @@ title: How to build Apache APISIX
#
-->
-## Step 1: Install dependencies
+##
codecov-commenter edited a comment on pull request #2200:
URL: https://github.com/apache/apisix-dashboard/pull/2200#issuecomment-963965563
#
codecov-commenter commented on pull request #788:
URL:
https://github.com/apache/apisix-ingress-controller/pull/788#issuecomment-987664591
#
101 - 165 of 165 matches
Mail list logo
