Re: [PR] change: remove self-contained info fields from ssl definition [apisix]

2023-10-12 Thread via GitHub
jiangfucheng commented on PR #10323: URL: https://github.com/apache/apisix/pull/10323#issuecomment-1761005147 > @jiangfucheng > > I have not recive your reponse email yet. you can click the email list for confirm. > > apache/apisix is the core project. to remove this field from

Re: [I] help request: Is the radix tree rebuilt every time any route is updated? [apisix]

2023-10-12 Thread via GitHub
wklken commented on issue #10268: URL: https://github.com/apache/apisix/issues/10268#issuecomment-1761002361 After read the source code and do a benchmark, It seems we can't fix it by change the `timeout` or `timer` keep calling `PUT /routes/1` to update the route, then - if there

Re: [PR] change: remove self-contained info fields from ssl definition [apisix]

2023-10-12 Thread via GitHub
Sn0rt commented on PR #10323: URL: https://github.com/apache/apisix/pull/10323#issuecomment-1760996477 @jiangfucheng I have not recive your reponse email yet. you can click the email list for confirm. apache/apisix is the core project. to remove this field from apisix, we can

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
lingsamuel commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1357833902 ## apisix/init.lua: ## @@ -1021,6 +1021,7 @@ function _M.stream_init_worker() plugin.init_worker() xrpc.init_worker() router.stream_init_worker() +

Re: [I] bug: dashbord config not effect [apisix]

2023-10-12 Thread via GitHub
beginnerWJC commented on issue #10327: URL: https://github.com/apache/apisix/issues/10327#issuecomment-1760990845 I've got the same problem. -- 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 sp

Re: [PR] feat: release 3.6.1 [apisix]

2023-10-12 Thread via GitHub
Revolyssup commented on PR #10328: URL: https://github.com/apache/apisix/pull/10328#issuecomment-1760988101 This commit to be added https://github.com/apache/apisix/pull/10312 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[apisix] branch master updated: docs: update upstream resource API in admin-api.md (#8867)

2023-10-12 Thread monkeydluffy
This is an automated email from the ASF dual-hosted git repository. monkeydluffy 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 2650551ff docs: update upstream resource A

Re: [PR] docs: update Upstream`s Request Body Parameters add lost Type colum a… [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 merged PR #8867: URL: https://github.com/apache/apisix/pull/8867 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr

[PR] feat: release 3.6.1 [apisix]

2023-10-12 Thread via GitHub
Revolyssup opened a new pull request, #10328: URL: https://github.com/apache/apisix/pull/10328 ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new

Re: [PR] change: remove self-contained info fields from ssl definition [apisix]

2023-10-12 Thread via GitHub
jiangfucheng commented on PR #10323: URL: https://github.com/apache/apisix/pull/10323#issuecomment-1760967742 @Sn0rt Hi, I replied to your email yesterday, but I am not sure if you received it, so, I am pasting the email content now. > I basically agree with you idea. Now APISIX not

Re: [PR] test: correct the customized-config-path name in test_cmd.sh [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10326: URL: https://github.com/apache/apisix/pull/10326#issuecomment-1760967141 @jiangfucheng please help to check -- 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 t

[I] bug: dashbord config not effect [apisix]

2023-10-12 Thread via GitHub
qlql489 opened a new issue, #10327: URL: https://github.com/apache/apisix/issues/10327 ### Current Behavior dashboard edit upstream port from 8081 to 80,query apisix admin api /apisix/admin/routes it work,port change to 80 ![image](https://github.com/apache/apisix/assets/107675

[PR] test: correct the customized-config-path name in test_cmd.sh [apisix]

2023-10-12 Thread via GitHub
xuruidong opened a new pull request, #10326: URL: https://github.com/apache/apisix/pull/10326 ### Description correct the customized-config-path name in test_cmd.sh Fixes # (issue) ### Checklist - [x] I have explained the need for this PR and the problem it solves

[I] help request: SSL_do_handshake() failed [apisix]

2023-10-12 Thread via GitHub
PlasticJonbenetDoll opened a new issue, #10325: URL: https://github.com/apache/apisix/issues/10325 ### Description Hello. I receive this error when trying to send request: `SSL_do_handshake() failed (SSL: error:14094410:SSL routines:ssl3_read_bytes:sslv3 alert handshake failure:

[PR] chore: remove docker image scanning [apisix-docker]

2023-10-12 Thread via GitHub
shreemaan-abhishek opened a new pull request, #515: URL: https://github.com/apache/apisix-docker/pull/515 Following is the dependency graph for APISIX: ![image](https://github.com/apache/apisix-docker/assets/61597896/21d6cb71-cf68-4c30-a227-ea73cad8425a) Most CVEs are from 3rd

Re: [PR] feat: support secret fetching in authz-keycloak [apisix]

2023-10-12 Thread via GitHub
Revolyssup commented on code in PR #10291: URL: https://github.com/apache/apisix/pull/10291#discussion_r1357726826 ## docs/en/latest/plugins/authz-keycloak.md: ## @@ -48,7 +48,7 @@ Refer to [Authorization Services Guide](https://www.keycloak.org/docs/latest/aut | token_endpoin

Re: [PR] fix: remove openldap dependencies from apisix [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10176: URL: https://github.com/apache/apisix/pull/10176#issuecomment-1760702512 @kingluo Could you help check if there is any remaining work pending? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[apisix-docker] branch master updated: chore: remove centos based docker images (#512)

2023-10-12 Thread monkeydluffy
This is an automated email from the ASF dual-hosted git repository. monkeydluffy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-docker.git The following commit(s) were added to refs/heads/master by this push: new abab1e8 chore: remove centos based

Re: [PR] chore: remove centos based docker images [apisix-docker]

2023-10-12 Thread via GitHub
monkeyDluffy6017 merged PR #512: URL: https://github.com/apache/apisix-docker/pull/512 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-uns

Re: [PR] doc: Documenting the possibility to specify nodePort for HTTP and TLS [apisix-helm-chart]

2023-10-12 Thread via GitHub
lingsamuel commented on PR #491: URL: https://github.com/apache/apisix-helm-chart/pull/491#issuecomment-1760678915 The doc is auto generated. Comments should be placed in values.yaml -- This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[apisix-helm-chart] branch dev updated: feat: allow deploying additional kubernetes resources (#636)

2023-10-12 Thread lingsamuel
This is an automated email from the ASF dual-hosted git repository. lingsamuel pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/dev by this push: new e299679 feat: allow deploying addition

Re: [PR] feat: allow deploying additional kubernetes resources [apisix-helm-chart]

2023-10-12 Thread via GitHub
lingsamuel merged PR #636: URL: https://github.com/apache/apisix-helm-chart/pull/636 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[apisix-helm-chart] branch master updated: Enable promethus plugins for service monitoring (#632)

2023-10-12 Thread lingsamuel
This is an automated email from the ASF dual-hosted git repository. lingsamuel pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/master by this push: new a54c769 Enable promethus plugins

Re: [PR] feat: Enable prometheus plugins for ingress's composite mode service monitoring [apisix-helm-chart]

2023-10-12 Thread via GitHub
lingsamuel merged PR #632: URL: https://github.com/apache/apisix-helm-chart/pull/632 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[apisix-helm-chart] branch dev updated: feat: allow changing service port in the Ingress manifest (#634)

2023-10-12 Thread lingsamuel
This is an automated email from the ASF dual-hosted git repository. lingsamuel pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/dev by this push: new 561107b feat: allow changing service p

Re: [PR] feat: allow changing service port in the Ingress manifest [apisix-helm-chart]

2023-10-12 Thread via GitHub
lingsamuel merged PR #634: URL: https://github.com/apache/apisix-helm-chart/pull/634 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

[apisix-helm-chart] branch dev updated: feat: allow adjusting nginx keepalive timeout (#637)

2023-10-12 Thread lingsamuel
This is an automated email from the ASF dual-hosted git repository. lingsamuel pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/dev by this push: new 981f82d feat: allow adjusting nginx ke

Re: [PR] feat: allow adjusting nginx keepalive timeout [apisix-helm-chart]

2023-10-12 Thread via GitHub
lingsamuel merged PR #637: URL: https://github.com/apache/apisix-helm-chart/pull/637 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsub

Re: [PR] chore: divide dockerfile `RUN` command [apisix-docker]

2023-10-12 Thread via GitHub
lingsamuel commented on PR #510: URL: https://github.com/apache/apisix-docker/pull/510#issuecomment-1760661665 It will definitely cause apt cache to remain in the generated image. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] feat: support secret fetching in authz-keycloak [apisix]

2023-10-12 Thread via GitHub
shreemaan-abhishek commented on code in PR #10291: URL: https://github.com/apache/apisix/pull/10291#discussion_r1357661692 ## docs/en/latest/plugins/authz-keycloak.md: ## @@ -48,7 +48,7 @@ Refer to [Authorization Services Guide](https://www.keycloak.org/docs/latest/aut | token

Re: [PR] fix: remove openldap dependencies from apisix [apisix]

2023-10-12 Thread via GitHub
Zhenye-Na commented on PR #10176: URL: https://github.com/apache/apisix/pull/10176#issuecomment-1760634303 > @Zhenye-Na What else does this pr need in order to be merged? From my point of view, this PR and https://github.com/api7/apisix-build-tools/pull/337 need to be merged together

[apisix-website] branch master updated: blog: Add bi-weekly report (0925-1008) (#1707)

2023-10-12 Thread young
This is an automated email from the ASF dual-hosted git repository. young 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 eaab568a277 blog: Add bi-weekly report (0

Re: [PR] blog: Add bi-weekly report (0925-1008) [apisix-website]

2023-10-12 Thread via GitHub
SkyeYoung merged PR #1707: URL: https://github.com/apache/apisix-website/pull/1707 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubsc

Re: [PR] fix: remove openldap dependencies from apisix [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10176: URL: https://github.com/apache/apisix/pull/10176#issuecomment-1760625788 @Zhenye-Na What else does this pr need in order to be merged? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] docs: architecture-design-fix [apisix]

2023-10-12 Thread via GitHub
kayx23 commented on code in PR #9017: URL: https://github.com/apache/apisix/pull/9017#discussion_r1357612305 ## docs/en/latest/architecture-design/apisix.md: ## @@ -38,14 +38,60 @@ The APISIX core handles the important functions like matching Routes, load balan APISIX also h

Re: [PR] feat(k8s-discovery): support mTLS [apisix]

2023-10-12 Thread via GitHub
moonming commented on PR #8699: URL: https://github.com/apache/apisix/pull/8699#issuecomment-1760546599 > > @e1ijah1 do you have time to fix comments and make CI happy? thanks > > > > I would like to try to fix the CI this weekend😊 great 👍 -- This is an automated mess

Re: [PR] feat(k8s-discovery): support mTLS [apisix]

2023-10-12 Thread via GitHub
e1ijah1 commented on PR #8699: URL: https://github.com/apache/apisix/pull/8699#issuecomment-1760543643 > @e1ijah1 do you have time to fix comments and make CI happy? thanks I would like to try to fix the CI this weekend😊 -- This is an automated message from the Apache Git Service. T

Re: [I] help request: route not found [apisix]

2023-10-12 Thread via GitHub
Revolyssup commented on issue #10322: URL: https://github.com/apache/apisix/issues/10322#issuecomment-1760341767 @xialp Can you share the route configuration and reproduction steps? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to Gi

[I] data format compatibility issues [apisix]

2023-10-12 Thread via GitHub
chiliast123 opened a new issue, #10324: URL: https://github.com/apache/apisix/issues/10324 ### Description Will there be data format compatibility issues with routes and upstream created by yaml files when apisix is upgraded (from 2.13 to 3.2.2)? ### Environment - APISIX

Re: [PR] fix: corrected minimum in healthcheck schema [apisix]

2023-10-12 Thread via GitHub
Neilblaze closed pull request #8949: fix: corrected minimum in healthcheck schema URL: https://github.com/apache/apisix/pull/8949 -- 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

Re: [PR] feat: support secret fetching in authz-keycloak [apisix]

2023-10-12 Thread via GitHub
Revolyssup commented on code in PR #10291: URL: https://github.com/apache/apisix/pull/10291#discussion_r1357015605 ## apisix/plugins/authz-keycloak.lua: ## @@ -16,11 +16,13 @@ -- local core = require("apisix.core") local http = require "resty.http" +local secret

Re: [PR] feat: support secret fetching in authz-keycloak [apisix]

2023-10-12 Thread via GitHub
Revolyssup commented on PR #10291: URL: https://github.com/apache/apisix/pull/10291#issuecomment-1759867865 > pls add user document for guide Updated the auth-keycloak plugin doc -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] feat: add jwe decrypt plugin [apisix]

2023-10-12 Thread via GitHub
fishioon commented on PR #10252: URL: https://github.com/apache/apisix/pull/10252#issuecomment-1759749063 > @fishioon please make the ci pass done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to

Re: [PR] feat: allow deploying additional kubernetes resources [apisix-helm-chart]

2023-10-12 Thread via GitHub
indrekj commented on PR #636: URL: https://github.com/apache/apisix-helm-chart/pull/636#issuecomment-1759672943 Rebased against dev to fix merge conflict. Workflow needs to be approved again. -- This is an automated message from the Apache Git Service. To respond to the message, please l

[apisix-helm-chart] branch dev updated: feat: allow adding extra containers to apisix deployment (#635)

2023-10-12 Thread zhangjintao
This is an automated email from the ASF dual-hosted git repository. zhangjintao pushed a commit to branch dev in repository https://gitbox.apache.org/repos/asf/apisix-helm-chart.git The following commit(s) were added to refs/heads/dev by this push: new d30ab3e feat: allow adding extra cont

Re: [PR] feat: allow adding extra containers to apisix deployment [apisix-helm-chart]

2023-10-12 Thread via GitHub
tao12345666333 merged PR #635: URL: https://github.com/apache/apisix-helm-chart/pull/635 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-u

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1356815739 ## apisix/init.lua: ## @@ -1078,6 +1079,34 @@ function _M.stream_preread_phase() api_ctx.matched_upstream = upstream +elseif matched_route.va

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1356791649 ## t/stream-node/sanity-with-service.t: ## @@ -0,0 +1,367 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license a

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1356782535 ## t/stream-node/sanity-with-service.t: ## @@ -0,0 +1,367 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license a

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1356778306 ## apisix/plugin.lua: ## @@ -636,6 +639,46 @@ function _M.merge_service_route(service_conf, route_conf) service_conf, route_conf) en

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10298: URL: https://github.com/apache/apisix/pull/10298#discussion_r1356769164 ## apisix/init.lua: ## @@ -1021,6 +1021,7 @@ function _M.stream_init_worker() plugin.init_worker() xrpc.init_worker() router.stream_init_worker

Re: [PR] fix: corrected minimum in healthcheck schema [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #8949: URL: https://github.com/apache/apisix/pull/8949#issuecomment-1759535250 hi @Neilblaze, It's designed on purpose, when the configuration is set to 0, it will never report healthy status -- This is an automated message from the Apache Git Service. To

Re: [PR] feat: support config stream_route upstream in service [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10298: URL: https://github.com/apache/apisix/pull/10298#issuecomment-1759504148 Please update the related doc: https://github.com/apache/apisix/blob/master/docs/en/latest/admin-api.md -- This is an automated message from the Apache Git Service. To respond

[PR] change: remove self-contained info fields from ssl definition [apisix]

2023-10-12 Thread via GitHub
Sn0rt opened a new pull request, #10323: URL: https://github.com/apache/apisix/pull/10323 ### Description Fixes https://github.com/apache/apisix/issues/10094 email: https://lists.apache.org/thread/8l4h8f6wcv482s0b7vt17do5z3g1y3o3 ### Checklist - [ ] I h

Re: [PR] fix: keep healthcheck target state when upstream changes [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10312: URL: https://github.com/apache/apisix/pull/10312#issuecomment-1759315265 @shreemaan-abhishek I don't know how you get this error, it has nothing to do with my changes. -- This is an automated message from the Apache Git Service. To respond to the m

Re: [I] bug: http://test.com:443 is not supported [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] closed issue #8082: bug: http://test.com:443 is not supported URL: https://github.com/apache/apisix/issues/8082 -- 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 comme

Re: [I] How to install offline? [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] closed issue #8034: How to install offline? URL: https://github.com/apache/apisix/issues/8034 -- 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

Re: [I] help request: whether apisix can be supported communication between grpc_client and grpc_server? [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #7973: URL: https://github.com/apache/apisix/issues/7973#issuecomment-1759312389 This issue has been marked as stale due to 350 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please simp

Re: [I] bug: APISIX 3.5.0 standalone is having etcd dependency. 3.4.0 did not have the issue. [apisix-docker]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #499: URL: https://github.com/apache/apisix-docker/issues/499#issuecomment-1759311716 This issue has been marked as stale due to 30 days of inactivity. It will be closed in 2 weeks if no further activity occurs. If this issue is still relevant, please

Re: [I] help request: Health check [apisix-docker]

2023-10-12 Thread via GitHub
github-actions[bot] closed issue #488: help request: Health check URL: https://github.com/apache/apisix-docker/issues/488 -- 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 uns

Re: [I] bug: http://test.com:443 is not supported [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #8082: URL: https://github.com/apache/apisix/issues/8082#issuecomment-1759312556 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. -- Th

Re: [I] How to install offline? [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #8034: URL: https://github.com/apache/apisix/issues/8034#issuecomment-1759312471 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. -- Th

Re: [I] plugin.lua:109: load_plugin(): failed to load plugin [auth-hook] err: /usr/local/apisix/apisix/init.lua:21: changing jit stack size is not allowed when some regexs have already been compiled a

2023-10-12 Thread via GitHub
github-actions[bot] closed issue #6404: plugin.lua:109: load_plugin(): failed to load plugin [auth-hook] err: /usr/local/apisix/apisix/init.lua:21: changing jit stack size is not allowed when some regexs have already been compiled and cached, context: init_worker_by_lua* URL: https://github.co

Re: [I] plugin.lua:109: load_plugin(): failed to load plugin [auth-hook] err: /usr/local/apisix/apisix/init.lua:21: changing jit stack size is not allowed when some regexs have already been compiled a

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #6404: URL: https://github.com/apache/apisix/issues/6404#issuecomment-1759312322 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. -- Th

Re: [I] help request: Health check [apisix-docker]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #488: URL: https://github.com/apache/apisix-docker/issues/488#issuecomment-1759311657 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. -

Re: [I] Use HTTP/2 to access api, but it was finally converted to HTTP/1.1 version by apisix [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #10168: URL: https://github.com/apache/apisix/issues/10168#issuecomment-1759308363 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still r

Re: [I] help request: Just getting started, got Header or Cookie Too large Error. [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #10078: URL: https://github.com/apache/apisix/issues/10078#issuecomment-1759308223 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still r

Re: [I] Eureka no valid upstream node [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #10071: URL: https://github.com/apache/apisix/issues/10071#issuecomment-1759308151 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still r

Re: [I] help request: Issue with yum install -y https://repos.apiseven.com/packages/centos/apache-apisix-repo-1.0-1.noarch.rpm [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #10123: URL: https://github.com/apache/apisix/issues/10123#issuecomment-1759308293 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still r

Re: [I] help request: Configure ssl certs for upstream health check [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] closed issue #9993: help request: Configure ssl certs for upstream health check URL: https://github.com/apache/apisix/issues/9993 -- 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

Re: [I] bug: does apisix support sse at now [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #10030: URL: https://github.com/apache/apisix/issues/10030#issuecomment-1759308082 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still r

Re: [I] help request: Configure ssl certs for upstream health check [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #9993: URL: https://github.com/apache/apisix/issues/9993#issuecomment-1759308027 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. -- Th

Re: [I] SSL_read() failed (SSL: error:14191044:SSL routines:tls1_enc:internal error) while waiting for request [apisix]

2023-10-12 Thread via GitHub
github-actions[bot] commented on issue #9950: URL: https://github.com/apache/apisix/issues/9950#issuecomment-1759307913 Due to lack of the reporter's response this issue has been labeled with "no response". It will be close in 3 days if no further activity occurs. If this issue is still rel

Re: [PR] fix: keep healthcheck target state when upstream changes [apisix]

2023-10-12 Thread via GitHub
shreemaan-abhishek commented on PR #10312: URL: https://github.com/apache/apisix/pull/10312#issuecomment-1759297689 I got the following error, when I ran the commands you mentioned: ``` 2023/10/12 15:37:06 [error] 65019#1620486: *4429371 lua entry thread aborted: runtime error: ..

[apisix] branch dependabot/go_modules/ci/pod/openfunction/function-example/test-body/golang.org/x/net-0.17.0 deleted (was 3dc4a9e6b)

2023-10-12 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/ci/pod/openfunction/function-example/test-body/golang.org/x/net-0.17.0 in repository https://gitbox.apache.org/repos/asf/apisix.git was 3dc4a9e6b chore(deps): bump

[apisix] branch dependabot/go_modules/ci/pod/openfunction/function-example/test-uri/golang.org/x/net-0.17.0 deleted (was 5e07a566c)

2023-10-12 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/ci/pod/openfunction/function-example/test-uri/golang.org/x/net-0.17.0 in repository https://gitbox.apache.org/repos/asf/apisix.git was 5e07a566c chore(deps): bump

Re: [PR] chore(deps): bump golang.org/x/net from 0.0.0-20211015210444-4f30a5c0130f to 0.17.0 in /ci/pod/openfunction/function-example/test-body [apisix]

2023-10-12 Thread via GitHub
dependabot[bot] commented on PR #10316: URL: https://github.com/apache/apisix/pull/10316#issuecomment-1759273742 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, le

Re: [PR] chore(deps): bump golang.org/x/net from 0.0.0-20211015210444-4f30a5c0130f to 0.17.0 in /ci/pod/openfunction/function-example/test-body [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 closed pull request #10316: chore(deps): bump golang.org/x/net from 0.0.0-20211015210444-4f30a5c0130f to 0.17.0 in /ci/pod/openfunction/function-example/test-body URL: https://github.com/apache/apisix/pull/10316 -- This is an automated message from the Apache Git Service. To

[apisix] branch dependabot/go_modules/t/grpc_server_example/golang.org/x/net-0.17.0 deleted (was 48ea1291c)

2023-10-12 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/go_modules/t/grpc_server_example/golang.org/x/net-0.17.0 in repository https://gitbox.apache.org/repos/asf/apisix.git was 48ea1291c chore(deps): bump golang.org/x/net in /t/g

Re: [PR] chore(deps): bump golang.org/x/net from 0.0.0-20220621193019-9d032be2e588 to 0.17.0 in /ci/pod/openfunction/function-example/test-uri [apisix]

2023-10-12 Thread via GitHub
dependabot[bot] commented on PR #10317: URL: https://github.com/apache/apisix/pull/10317#issuecomment-1759273532 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, le

Re: [PR] chore(deps): bump golang.org/x/net from 0.0.0-20220621193019-9d032be2e588 to 0.17.0 in /ci/pod/openfunction/function-example/test-uri [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 closed pull request #10317: chore(deps): bump golang.org/x/net from 0.0.0-20220621193019-9d032be2e588 to 0.17.0 in /ci/pod/openfunction/function-example/test-uri URL: https://github.com/apache/apisix/pull/10317 -- This is an automated message from the Apache Git Service. To

Re: [PR] chore(deps): bump golang.org/x/net from 0.7.0 to 0.17.0 in /t/grpc_server_example [apisix]

2023-10-12 Thread via GitHub
dependabot[bot] commented on PR #10318: URL: https://github.com/apache/apisix/pull/10318#issuecomment-1759273242 OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, le

Re: [PR] chore(deps): bump golang.org/x/net from 0.7.0 to 0.17.0 in /t/grpc_server_example [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 closed pull request #10318: chore(deps): bump golang.org/x/net from 0.7.0 to 0.17.0 in /t/grpc_server_example URL: https://github.com/apache/apisix/pull/10318 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

Re: [PR] feat: add jwe decrypt plugin [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10252: URL: https://github.com/apache/apisix/pull/10252#issuecomment-1759269971 @fishioon please make the ci pass -- 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] branch master updated: fix: fix and optimize tls in upstream_schema (#10269)

2023-10-12 Thread monkeydluffy
This is an automated email from the ASF dual-hosted git repository. monkeydluffy 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 11ee894e4 fix: fix and optimize tls in ups

Re: [I] bug: wrong json schema configuration [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 closed issue #10260: bug: wrong json schema configuration URL: https://github.com/apache/apisix/issues/10260 -- 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

Re: [PR] fix: fix and optimize tls in upstream_schema [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 merged PR #10269: URL: https://github.com/apache/apisix/pull/10269 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubs

Re: [PR] feat: allow changing service port in the Ingress manifest [apisix-helm-chart]

2023-10-12 Thread via GitHub
Revolyssup commented on PR #634: URL: https://github.com/apache/apisix-helm-chart/pull/634#issuecomment-1759215523 > @Revolyssup Could you elaborate on what exactly you mean? Do you mean using `.Values.service.tls.servicePort | default .Values.service.http.servicePort` in the ingress? I th

Re: [PR] feat: allow changing service port in the Ingress manifest [apisix-helm-chart]

2023-10-12 Thread via GitHub
indrekj commented on PR #634: URL: https://github.com/apache/apisix-helm-chart/pull/634#issuecomment-1759212500 @Revolyssup Could you elaborate on what exactly you mean? Do you mean using `.Values.service.tls.servicePort | default .Values.service.http.servicePort` in the ingress? I think t

Re: [PR] feat: allow changing service port in the Ingress manifest [apisix-helm-chart]

2023-10-12 Thread via GitHub
Revolyssup commented on PR #634: URL: https://github.com/apache/apisix-helm-chart/pull/634#issuecomment-1759195703 @indrekj If the use case is to allow usage with https then why not do something like `service.tls.servicePort | service.http.servicePort` ? -- This is an automated message f

[I] help request:路由未找到 [apisix]

2023-10-12 Thread via GitHub
xialp opened a new issue, #10322: URL: https://github.com/apache/apisix/issues/10322 ### Description 1.26版本k8s集群,helm chart 安装apisix apache/apisix:3.5.0-debian版本,dashbord配置路由,做好转发,但是报错找不到路由,其他版本的也尝试都不可以,但是换到低版本的K8s则是可以的 ### Environment - APISIX version (run `apisix ver

Re: [PR] feat: support secret fetching in authz-keycloak [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on code in PR #10291: URL: https://github.com/apache/apisix/pull/10291#discussion_r1356481413 ## apisix/plugins/authz-keycloak.lua: ## @@ -16,11 +16,13 @@ -- local core = require("apisix.core") local http = require "resty.http" +local secr

Re: [I] request help: When i create a Ingress, Why is an empty certificate generated? [apisix-ingress-controller]

2023-10-12 Thread via GitHub
jelly320 commented on issue #2002: URL: https://github.com/apache/apisix-ingress-controller/issues/2002#issuecomment-1759170572 @tao12345666333 -- 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 t

Re: [I] request help: When i create a Ingress, Why is an empty certificate generated? [apisix-ingress-controller]

2023-10-12 Thread via GitHub
jelly320 commented on issue #2002: URL: https://github.com/apache/apisix-ingress-controller/issues/2002#issuecomment-1759169897 > APISIX Ingress controller will synchronize SSL resources for APISIX according to your configuration. > > Please directly query the specific information of

[apisix] branch master updated: fix: add name field in plugin_config schema for consistenct (#10315)

2023-10-12 Thread monkeydluffy
This is an automated email from the ASF dual-hosted git repository. monkeydluffy 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 d753bfc93 fix: add name field in plugin_co

Re: [I] Compatibility issue: unable to create plugin config with APISIX v3.6 [apisix-ingress-controller]

2023-10-12 Thread via GitHub
monkeyDluffy6017 closed issue #2003: Compatibility issue: unable to create plugin config with APISIX v3.6 URL: https://github.com/apache/apisix-ingress-controller/issues/2003 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us

Re: [PR] fix: add name field in plugin_config schema for consistency [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 merged PR #10315: URL: https://github.com/apache/apisix/pull/10315 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubs

Re: [PR] fix: keep healthcheck target state when upstream changes [apisix]

2023-10-12 Thread via GitHub
monkeyDluffy6017 commented on PR #10312: URL: https://github.com/apache/apisix/pull/10312#issuecomment-1759107068 @shreemaan-abhishek You need to use wrk2 or remove the `-R` option -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] bug: https route does not use TLS [apisix]

2023-10-12 Thread via GitHub
GuyT2002 commented on issue #10293: URL: https://github.com/apache/apisix/issues/10293#issuecomment-1759096420 Well this is not written in the helm docs, as well as I can't set it to work - even editing the apisix configmap like so ``` ssl: enable: true ``` Manua