[apisix-website] branch asf-site updated (0481897fcc1 -> 01ffc4c0ae7)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/apisix-website.git discard 0481897fcc1 deploy: bdd3e823ced08b417b9721400e21b1e22f776d36 new 01ffc4c0ae7 deploy: bdd3e823ced08b417b9721400e21b1e22f776d36 This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (0481897fcc1) \ N -- N -- N refs/heads/asf-site (01ffc4c0ae7) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omit" are not gone; other references still refer to them. Any revisions marked "discard" are gone forever. The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: 404.html | 4 ++-- .../index.html| 4 ++-- .../Apache-APISIX-From-OpenSource-Commercialization/index.html| 8 articles/Apache-APISIX-Incubator-Journey/index.html | 4 ++-- articles/Apache-APISIX-Kubernetes-Ingress/index.html | 4 ++-- articles/Apache-APISIX-in-China-Mobile-Cloud/index.html | 8 .../index.html| 4 ++-- .../index.html| 4 ++-- articles/Rendering-Community-Events-Using-ECharts/index.html | 4 ++-- articles/Speed-Limiting-With-Apache-APISIX/index.html | 4 ++-- articles/Test-Apache-APISIX-Resilience-With-Chaos-Mesh/index.html | 4 ++-- articles/The-Appeal-of-OpenSource/index.html | 4 ++-- articles/The-Evolution-of-Apache-APISIX/index.html| 4 ++-- .../index.html| 4 ++-- articles/archive/index.html | 6 +++--- articles/atom.xml | 6 +++--- articles/index.html | 6 +++--- articles/page/2/index.html| 4 ++-- articles/rss.xml | 6 +++--- assets/js/{09b17eb5.7e476228.js => 09b17eb5.a4ff8a82.js} | 2 +- .../js/0d0b38bc.8448a023.js => assets/js/0d0b38bc.5cc4d931.js | 2 +- assets/js/{507e1a4c.e6c0b754.js => 507e1a4c.a12cb330.js} | 2 +- assets/js/{8f700fa6.0864c2b9.js => 8f700fa6.b97506b0.js} | 2 +- assets/js/{90fc8a94.5ef0e13a.js => 90fc8a94.1df51fac.js} | 2 +- assets/js/{941960b1.d0d7ebce.js => 941960b1.dc39df25.js} | 2 +- assets/js/{d739b11f.530dc8f4.js => d739b11f.681b1303.js} | 2 +- assets/js/{runtime~main.92405fc2.js => runtime~main.e0732ff7.js} | 2 +- blog/page/10/index.html | 2 +- blog/page/11/index.html | 2 +- blog/page/12/index.html | 2 +- blog/page/13/index.html | 2 +- blog/page/14/index.html | 2 +- blog/page/15/index.html | 2 +- blog/page/16/index.html | 2 +- blog/page/17/index.html | 2 +- blog/page/18/index.html | 2 +- blog/page/2/index.html| 2 +- blog/page/3/index.html| 2 +- blog/page/4/index.html| 2 +- blog/page/5/index.html| 2 +- blog/page/6/index.html| 2 +- blog/page/7/index.html| 2 +- blog/page/8/index.html| 2 +- blog/page/9/index.html| 2 +- contribute/index.html | 6 +++--- docs/general/blog/index.html | 4 ++-- docs/general/committer-guide/index.html | 4 ++-- docs/general/contributor-guide/index.html | 4 ++-- docs/general/documentation-style-guide/index.html | 4 ++--
[GitHub] [apisix] sangnguyen7 commented on issue #7661: bug: [Plugin] OpenID-Connect had an issue or not supported when using "use_jwks=true" and use "discovery" as the JWKS endpoint
sangnguyen7 commented on issue #7661: URL: https://github.com/apache/apisix/issues/7661#issuecomment-1213649126 Here is the link to the code snippet that checks the jwks_uri in the "resty.openidc" package. https://github.com/zmartzone/lua-resty-openidc/blob/b07330120ffe54dd3fbeac247726b76d0f9dc793/lib/resty/openidc.lua#L893 ``` jwks, err = openidc_jwks(opts.discovery.jwks_uri, force, opts.ssl_verify, opts.keepalive, opts.timeout, opts.jwk_expires_in, opts.proxy_opts, ``` As above, it requires the "discovery" as an object which contains "jwks_uri" -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] sangnguyen7 commented on issue #7661: bug: [Plugin] OpenID-Connect had an issue or not supported when using "use_jwks=true" and use "discovery" as the JWKS endpoint
sangnguyen7 commented on issue #7661: URL: https://github.com/apache/apisix/issues/7661#issuecomment-1213647274 @Hazel6869 Thanks for your comment. Sorry, I did not know that I had entered incorrectly in the Step To Reproduce above. It should be like this: ``` { "uri": "/verify", "plugins": { "openid-connect": { "client_id": "", "client_secret": "", "disable": false, "discovery": "http://host:port/.well-known/jwks.json;, "use_jwks": true } }, "upstream":{ "type": "roundrobin", "nodes":{ "httpbin.org:443":1 } } }' ``` Yes, I did use "discovery" for my JWKS endpoint and I got the error in this issue. "pts.discovery.jwks_uri is not present or not a string" -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] pixeldin commented on issue #2586: Why is there still this problem,grafana cannot be displayed
pixeldin commented on issue #2586: URL: https://github.com/apache/apisix-dashboard/issues/2586#issuecomment-1213644268 Hi, @yigeqiji, this maybe help, update the `allow_embedding` config item as `true` in your Grafana configuration file and restart Grafana server. ``` # security # set to true if you want to allow browsers to render Grafana in a , , or . default is false. allow_embedding = true ``` -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] Hazel6869 commented on issue #7660: docs: Dead link
Hazel6869 commented on issue #7660: URL: https://github.com/apache/apisix/issues/7660#issuecomment-1213641766 @stillfox-lee Hi, thanks for your suggestion, we will fix it soon -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] Hazel6869 commented on issue #7661: bug: [Plugin] OpenID-Connect had an issue or not supported when using "use_jwks=true" and use "discovery" as the JWKS endpoint
Hazel6869 commented on issue #7661: URL: https://github.com/apache/apisix/issues/7661#issuecomment-1213639793 @sangnguyen7 Hi, you can try use "discovery" to replace "discovery.jwks_uri". The response returned by the value of "discovery" needs to contain "jwks_uri", you can see https://openid.net/specs/openid-connect-discovery-1_0.html. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] github-actions[bot] closed issue #719: request help: Does apisix-ingress-controller service account require coordination.k8s.io in clusterRule
github-actions[bot] closed issue #719: request help: Does apisix-ingress-controller service account require coordination.k8s.io in clusterRule URL: https://github.com/apache/apisix-ingress-controller/issues/719 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] github-actions[bot] commented on issue #719: request help: Does apisix-ingress-controller service account require coordination.k8s.io in clusterRule
github-actions[bot] commented on issue #719: URL: https://github.com/apache/apisix-ingress-controller/issues/719#issuecomment-1213627362 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. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] nfrankel commented on pull request #1285: docs: Add Backend-for-Frontend: the demo blog post
nfrankel commented on PR #1285: URL: https://github.com/apache/apisix-website/pull/1285#issuecomment-1213366488 **Please do not merge before Aug. 17th** -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] netlify[bot] commented on pull request #1285: docs: Add Backend-for-Frontend: the demo blog post
netlify[bot] commented on PR #1285: URL: https://github.com/apache/apisix-website/pull/1285#issuecomment-1213365387 ### Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| | Latest commit | 6ecad0854f33abaed16f04b0dc44a81c719a7d5d | | Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62f6928fe716f50008f7ddcf | -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] nfrankel opened a new pull request, #1285: docs: Add Backend-for-Frontend: the demo blog post
nfrankel opened a new pull request, #1285: URL: https://github.com/apache/apisix-website/pull/1285 Changes: New blog post: "Backend-for-Frontend: the demo" Screenshots of the change: https://user-images.githubusercontent.com/752258/184414934-5bd8648e-26eb-475a-88d1-5da1a0cbe376.png;> -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #6418: bug: apisix failed to verify the validity of the server certificate
tzssangglass commented on issue #6418: URL: https://github.com/apache/apisix/issues/6418#issuecomment-1213313531 > 我了解,但这样的话只能配置一个ca证书,如果有多个上游,并且多个上游使用不同的ca证书进行签发,这里只能满足配置一个。 We can use this way: https://github.com/apache/apisix/blob/master/docs/zh/latest/certificate.md#%E8%AE%BE%E7%BD%AE%E5%A4%9A%E4%B8%AA-ca-%E8%AF%81%E4%B9%A6 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] sangnguyen7 opened a new issue, #7661: bug: [Plugin] OpenID-Connect had an issue or supported when using "use_jwks=true" and use "discovery" as the JWKS endpoint
sangnguyen7 opened a new issue, #7661: URL: https://github.com/apache/apisix/issues/7661 ### Current Behavior Get 401 Response when introspecting a JWK token: WWW-Authenticate="Bearer realm="apisix", error="invalid_token", error_description="opts.discovery.jwks_uri is not present or not a string" I have checked the openid-connect plugin code, it's using "resty.openidc" which supports "discovery" as an object that has a property "jwks_uri". However, the current openid-conect plugin required "discovery" as a String. ### Expected Behavior Should make "discovery" as an object, instead of String, so that we can config as "discovery.jwks_uri" ### Error Logs _No response_ ### Steps to Reproduce 1. Setup a route ("/verify") to an upstream with an openid-connect plugin setting as follow: ``` { "uri": "/verify", "plugins": { "openid-connect": { "client_id": "", "client_secret": "", "disable": false, "discovery.jwks_uri": "http://host:port/.well-known/jwks.json;, "use_jwks": true } }, "upstream":{ "type": "roundrobin", "nodes":{ "httpbin.org:443":1 } } }' ``` 2. Make a call ``` curl -i -X GET http://127.0.0.1:9080/verify -H "Host: httpbin.org" -H "Authorization: Bearer {JWT_TOKEN}" ``` ### Environment - APISIX version (run `apisix version`): 2.15 - Operating system (run `uname -a`): Kubernetes - OpenResty / Nginx version (run `openresty -V` or `nginx -V`): - etcd version, if relevant (run `curl http://127.0.0.1:9090/v1/server_info`): - APISIX Dashboard version, if relevant: - Plugin runner version, for issues related to plugin runners: - LuaRocks version, for installation issues (run `luarocks --version`): -- 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...@apisix.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] vitcou opened a new issue, #2588: "Refused to load the image" , which build by deploy-with-docker.md
vitcou opened a new issue, #2588: URL: https://github.com/apache/apisix-dashboard/issues/2588 ### Issue description i used the "deploy-with-docker.md" to build the docker image with git id 0273865d54eb60a5812c403a887398fe74d30a65() the image can run, but i found the plugin image can not load right. https://user-images.githubusercontent.com/20715740/184387033-ebe41ea9-9bd5-4b3b-ab07-5c7706590d71.png;> ### Expected behavior plugin image can load right. ### How to Reproduce 1.using "deploy-with-docker.md" build image by master(0273865d54eb60a5812c403a887398fe74d30a65) 2.run the docker container 3.add route 4.the plugin page can not load default plugin images right ### Screenshots _No response_ ### Environment - apisix version (cmd: `apisix version`): - OS (cmd: `uname -a`): - OpenResty / Nginx version (cmd: `nginx -V` or `openresty -V`): - etcd version, if have (cmd: run `etcd --version`): - apisix-dashboard version, if have: - Browser version, if have: ### Additional context _No response_ -- 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...@apisix.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] ccxhwmy commented on pull request #7099: init elasticsearch-logger.lua
ccxhwmy commented on PR #7099: URL: https://github.com/apache/apisix/pull/7099#issuecomment-1213119807 > > > @spacewander @tzssangglass #7643 for this issue,It's up to you > > > > > > Thanks for your contribution, let's do it. > > @ccxhwmy As mentioned above, you should use the elasticsearch bulk API (endpoint/index/_bulk)..but I noticed you didn't. Yes, I will implement this suggestion, Thank you. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] FangSen9000 commented on pull request #2584: Chore: move yarn to pnpm(try1.5)
FangSen9000 commented on PR #2584: URL: https://github.com/apache/apisix-dashboard/pull/2584#issuecomment-1213084793 To put it simply, there are two GitHub workflows, one is to package and generate rpm, and the other is to package and generate docker image. It turned out that I used yarn. When i migrated to pnpm, I saw that neither the dockerfile nor the makefile mentioned yarn. I didn't move these two files. Now there is a problem with these two files, saying that pnpm can't be found How can I install a pnpm when generating docker, or install a pnpm when packaging RPM? How can I use Yum? apt? -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] ZhangShangyu commented on issue #7651: help request: init.lua:349: store_token(): ext-plugin:set valid items forcibly overwritten
ZhangShangyu commented on issue #7651: URL: https://github.com/apache/apisix/issues/7651#issuecomment-1213052559 > thanks your explanation. so this log just a tips, not related to a performance or business issue? -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] FangSen9000 commented on pull request #2584: Chore: move yarn to pnpm(try1.5)
FangSen9000 commented on PR #2584: URL: https://github.com/apache/apisix-dashboard/pull/2584#issuecomment-1213039293 @bzp2010 At present, I have encountered two problems: 1. After running the command in cypress, the server does not know whether it is running well. This is a strange question. Maybe something is missing. I'm reading the cypress document 2. The rpm and the generation of docker images cannot be carried out because of the lack of pnpm. Why did yarn work before, perhaps because the workflow comes with yarn? I am looking for commands to install pnpm in the rpm and docker generation files. It is also possible to try to build RPM packages and images myself. I am learning. I'd like you to take a look. Maybe you with rich experience can see it. Whether there are mistakes in “makefile” and “dockerfile”, and whether there are obvious missing items. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] mhh12121 commented on issue #1235: request help: Does apisixroute v2beta3 support multiple plugins?
mhh12121 commented on issue #1235: URL: https://github.com/apache/apisix-ingress-controller/issues/1235#issuecomment-1213031306 @tao12345666333 Thank you. But I did look through the admin's log; With the apisixroute mentioned above, we could see **/backend_docs** was forced to use forward-auth: ``` 2022/08/12 11:44:50 [warn] 44#44: *3203743 [lua] plugin.lua:750: run_plugin(): forward-auth exits with http status code 400, client: 10.23.4.212, server: _, request: "GET /backend_docs HTTP/1.1", host: "10.23.4.212:31684" 10.23.4.212 - - [12/Aug/2022:11:44:50 +] 10.23.4.212:31684 "GET /backend_docs HTTP/1.1" 400 60 0.002 "-" "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/100.0.4896.127 Safari/537.36" - - - "http://10.23.4.212:31684/backend_docs; ``` -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] jaysonsantos commented on issue #2426: bug:enable openid-connect plugin without redirect_uri got 500 error
jaysonsantos commented on issue #2426: URL: https://github.com/apache/apisix/issues/2426#issuecomment-1212988261 For those still struggling with that, by default resty.session will try to use `aes` as the encryption algorithm for the cookie that holds the session, but without setting `session_secret` `session.start` will fail with the reason `EVP_DecryptFinal_ex`, to avoid that you have to set it with a 32 bytes random value. If using the helm chart to spin up apisix, this should do the trick ```yaml configurationSnippet: httpSrv: | set $session_secret 8044c47e83b5ac9bb7c868eb8b202e93; ``` to generate that secret: ``` openssl rand 16 -hex ``` -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-helm-chart] Iced-Sun opened a new issue, #325: proxy protocol customization
Iced-Sun opened a new issue, #325: URL: https://github.com/apache/apisix-helm-chart/issues/325 Currently, the proxy-protocol related options are not exposed in value.yaml. -- 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...@apisix.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] quanbisen commented on pull request #2385: fix: The bug of missing secret field and algorithm when creating jwt-auth has been solved
quanbisen commented on PR #2385: URL: https://github.com/apache/apisix-dashboard/pull/2385#issuecomment-1212962389 Hey, man. Can you resolve this merge conflict? -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] stillfox-lee commented on issue #7660: docs: Dead link
stillfox-lee commented on issue #7660: URL: https://github.com/apache/apisix/issues/7660#issuecomment-1212961112 https://github.com/apache/apisix/blob/5bdcd4412025877b15ea71510be13ebfca52d220/docs/en/latest/stream-proxy.md?plain=1#L76 I found English version doc link to the right place. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] stillfox-lee opened a new issue, #7660: docs: Dead link
stillfox-lee opened a new issue, #7660: URL: https://github.com/apache/apisix/issues/7660 ### Current State https://github.com/apache/apisix/blob/5bdcd4412025877b15ea71510be13ebfca52d220/docs/zh/latest/stream-proxy.md?plain=1#L74 [test case] link 404. ### Desired State Link to the right test case URL. -- 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...@apisix.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] github-actions[bot] closed issue #4129: request help: oidc problem
github-actions[bot] closed issue #4129: request help: oidc problem URL: https://github.com/apache/apisix/issues/4129 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] github-actions[bot] commented on issue #4129: request help: oidc problem
github-actions[bot] commented on issue #4129: URL: https://github.com/apache/apisix/issues/4129#issuecomment-1212944123 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. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] quanbisen opened a new pull request, #2587: support conf varible override
quanbisen opened a new pull request, #2587: URL: https://github.com/apache/apisix-dashboard/pull/2587 Please answer these questions before submitting a pull request, **or your PR will get closed**. **Why submit this pull request?** add conf varible `APISIX_CONF_IN_JSON` reading in json format. **What changes will this PR take into?** add PathExist utils and modity conf loading logic. **Related issues** fix/resolve #2514 **Checklist:** - [ ] Did you explain what problem does this PR solve? Or what new features have been added? - [ ] Have you added corresponding test cases? - [ ] Have you modified the corresponding document? - [ ] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] quanbisen closed pull request #2582: support conf varible override
quanbisen closed pull request #2582: support conf varible override URL: https://github.com/apache/apisix-dashboard/pull/2582 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] hyt05 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
hyt05 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212927734 Solved, my k8s cluster node network is not connected, and the health check fails. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] hyt05 closed issue #1242: request help: After removing apisix-ingress-controller it will not start
hyt05 closed issue #1242: request help: After removing apisix-ingress-controller it will not start URL: https://github.com/apache/apisix-ingress-controller/issues/1242 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on pull request #7099: init elasticsearch-logger.lua
tzssangglass commented on PR #7099: URL: https://github.com/apache/apisix/pull/7099#issuecomment-1212920351 > As mentioned above, you should use the elasticsearch bulk API (endpoint/index/_bulk)..but I noticed you didn't. Yes, we need to implement this. > Why not recommend testing on my basis or use my plugin directly. It's not a decision I made. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] whioue commented on issue #6418: bug: apisix failed to verify the validity of the server certificate
whioue commented on issue #6418: URL: https://github.com/apache/apisix/issues/6418#issuecomment-1212890082 > > 在upstream中只配置了client_cert和client_key,却没有client_ca字段, > > IMO, The CA certificate used for the mTLS connection between APISIX and upstream is > > ```yaml > apisix: > ssl: > ssl_trusted_certificate: /path/to/certs/ca-certificates.crt > ``` 我了解,但这样的话只能配置一个ca证书,如果有多个上游,并且多个上游使用不同的ca证书进行签发,这里只能满足配置一个。 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944258178 ## apisix/plugins/grpc-transcode.lua: ## @@ -72,6 +72,17 @@ local schema = { "disable_hooks", } }, +show_status_in_body = { +description = "show decoded grpc-status-details-bin in response body", +type= "boolean", +default = false +}, +-- https://github.com/googleapis/googleapis/blob/master/google/rpc/status.proto#L46 Review Comment: Do you mean link after api7? -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #6418: bug: apisix failed to verify the validity of the server certificate
tzssangglass commented on issue #6418: URL: https://github.com/apache/apisix/issues/6418#issuecomment-1212885278 > 在upstream中只配置了client_cert和client_key,却没有client_ca字段, IMO, The CA certificate used for the mTLS connection between APISIX and upstream is ```yaml apisix: ssl: ssl_trusted_certificate: /path/to/certs/ca-certificates.crt ``` -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944254388 ## apisix/plugins/grpc-transcode/response.lua: ## @@ -14,18 +14,77 @@ -- See the License for the specific language governing permissions and -- limitations under the License. -- -local util = require("apisix.plugins.grpc-transcode.util") +local util= require("apisix.plugins.grpc-transcode.util") +local grpc_proto = require("apisix.plugins.grpc-transcode.proto") local core = require("apisix.core") local pb = require("pb") local ngx= ngx local string = string +local ngx_decode_base64 = ngx.decode_base64 +local ipairs = ipairs +local pcall = pcall -return function(ctx, proto, service, method, pb_option) + +local function handle_error_response(status_detail_type) +local headers = ngx.resp.get_headers() +local grpc_status = headers["grpc-status-details-bin"] Review Comment: 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 go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #3783: request help: SSL: error:1408F10B:SSL routines:ssl3_get_record:wrong version number
tzssangglass commented on issue #3783: URL: https://github.com/apache/apisix/issues/3783#issuecomment-1212867950 > 我理解的重定向插件是客户端使用http访问apisix,会重定向为客户端使用https访问apisix,这与apisix与后端的ssl握手有关联吗? I mean, this problem was solved by this last user with the `redirect` plugin. If your question is different, please submit a new issue. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
tzssangglass commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1212859774 > yes, but I'm on 4 5g per hour, unless it's only reserved for an hour, the cache will still be high Very heavy usage, in which case I suggest you turn off access log (if feasible) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch master updated: docs: introduce the paging and filtering (#7647)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git The following commit(s) were added to refs/heads/master by this push: new 5bdcd4412 docs: introduce the paging and filtering (#7647) 5bdcd4412 is described below commit 5bdcd4412025877b15ea71510be13ebfca52d220 Author: tzssangglass AuthorDate: Fri Aug 12 15:54:33 2022 +0800 docs: introduce the paging and filtering (#7647) --- docs/en/latest/admin-api.md | 86 + docs/zh/latest/admin-api.md | 85 2 files changed, 171 insertions(+) diff --git a/docs/en/latest/admin-api.md b/docs/en/latest/admin-api.md index 9889c1be9..8193f6152 100644 --- a/docs/en/latest/admin-api.md +++ b/docs/en/latest/admin-api.md @@ -27,6 +27,92 @@ By default, the Admin API listens to port `9080` (`9443` for HTTPS) when APISIX **Note**: Mentions of `X-API-KEY` in this document refers to `apisix.admin_key.key`—the access token for Admin API—in your configuration file. +## V3 + +The Admin API has made some breaking changes in V3 version, as well as supporting additional features. + +### Support new response body format + +1. Remove `action` field in response body; +2. Adjust the response body structure when fetching the list of resources, the new response body structure like: + +```json +{ +"count":2, +"list":[ +{ +... +}, +{ +... +} +] +} +``` + +### Support paging query + +Paging query is supported when getting the resource list, paging parameters include: + +| parameter | Default | Valid range | Description | +| - | -- | --- | | +| page | 1 | [1, ...]| Number of pages | +| page_size | | [10, 500] | Number of resources per page | + +The example is as follows: + +```shell +$ curl http://127.0.0.1:9080/apisix/admin/routes?page=1_size=10 \ +-H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -i -d ' +{ + "count": 1, + "list": [ +{ + ... +} + ] +} +``` + +Resources that support paging queries: + +- Consumer +- Global Rules +- Plugin Config +- Proto +- Route +- Service +- SSL +- Stream Route +- Upstream + +### Support filtering query + +When getting a list of resources, it supports filtering resources based on `name`, `label`, `uri`. + +| parameter | parameter| +| - | | +| name | Query resource by their `name`, which will not appear in the query results if the resource itself does not have `name`. | +| label | Query resource by their `label`, which will not appear in the query results if the resource itself does not have `label`. | +| uri | Supported on Route resources only. If the `uri` of a Route is equal to the uri of the query or if the `uris` contains the uri of the query, the Route resource appears in the query results. | + +When multiple filter parameters are enabled, use the intersection of the query results for different filter parameters. + +The following example will return a list of routes, and all routes in the list satisfy: the `name` of the route contains the string "test", the `uri` contains the string "foo", and there is no restriction on the `label` of the route, since the label of the query is the empty string. + +```shell +$ curl http://127.0.0.1:9080/apisix/admin/routes?name=test=foo= \ +-H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -i -d ' +{ + "count": 1, + "list": [ +{ + ... +} + ] +} +``` + ## Route **API**: /apisix/admin/routes/{id}?ttl=0 diff --git a/docs/zh/latest/admin-api.md b/docs/zh/latest/admin-api.md index 7f7668e23..a5a2dffaa 100644 --- a/docs/zh/latest/admin-api.md +++ b/docs/zh/latest/admin-api.md @@ -29,6 +29,91 @@ Admin API 是为 Apache APISIX 服务的一组 API,我们可以将参数传递 在下面出现的 `X-API-KEY` 指的是 `conf/config.yaml` 文件中的 `apisix.admin_key.key`,它是 Admin API 的访问 token。 +## V3 + +Admin API 在 V3 版本中做了一些不向下兼容的调整,以及支持更多特性。 + +### 支持新的响应体格式 + +1. 移除响应体中的 `action` 字段; +2. 调整获取资源列表时的响应体结构,新的响应体结构示例如下: + +```json +{ +"count":2, +"list":[ +{ +... +}, +{ +... +} +] +} +``` + +### 支持分页查询 + +获取资源列表时支持分页查询,分页参数包括: + +| 参数 | 默认值 | 范围 | 说明 | +| - | -- | | | +| page | 1 | [1, ...] | 页数 | +| page_size || [10, 500]| 每页资源数量 | + +示例如下: + +```shell +$ curl http://127.0.0.1:9080/apisix/admin/routes?page=1_size=10 \ +-H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -i -d ' +{ + "count": 1, + "list": [ +{ + ... +} + ] +} +``` + +目前支持分页查询的资源如下: + +- Consumer +- Global Rules +- Plugin Config +- Proto +- Route +- Service +-
[GitHub] [apisix] juzhiyuan merged pull request #7647: docs: introduce the paging and filtering
juzhiyuan merged PR #7647: URL: https://github.com/apache/apisix/pull/7647 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Error while running github feature from .asf.yaml in apisix!
An error occurred while running github feature in .asf.yaml!: 422 {"message": "Validation Failed", "errors": ["A repository cannot have more than 20 topics."], "documentation_url": "https://docs.github.com/rest/reference/repos#replace-all-repository-topics"}
[GitHub] [apisix-ingress-controller] hyt05 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
hyt05 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212832235 > 我看到pod状态是CrashLoopBackOff,在这种情况下,你需要查看它的日志并调试它。 > > 此外,在使用 APISIX Ingress 时,不建议您使用 APISIX Dashboard 进行数据操作。 ![image](https://user-images.githubusercontent.com/48778924/184309465-46277520-5a70-465b-896e-b5699d593a1f.png) Its log is as follows, I don't have upstreams appearing in the log, and it doesn't help after I redeploy, how can I fix it -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] membphis closed pull request #7659: test: make a try, do not merge me!!
membphis closed pull request #7659: test: make a try, do not merge me!! URL: https://github.com/apache/apisix/pull/7659 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
tao12345666333 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212829897 I saw the pod status is CrashLoopBackOff, In this scenario, you need to view its log and debug it. In addition, when using APISIX Ingress, it is not recommended that you use APISIX Dashboard for data manipulation. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch test_reuse_ctx updated (1c9cf7671 -> 2a9e476b8)
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch test_reuse_ctx in repository https://gitbox.apache.org/repos/asf/apisix.git from 1c9cf7671 test: make a try add 2a9e476b8 no message No new revisions were added by this update. Summary of changes: apisix/init.lua | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
[GitHub] [apisix] membphis opened a new pull request, #7659: test: make a try, do not merge me!!
membphis opened a new pull request, #7659: URL: https://github.com/apache/apisix/pull/7659 ### 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 features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [ ] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch test_reuse_ctx created (now 1c9cf7671)
This is an automated email from the ASF dual-hosted git repository. membphis pushed a change to branch test_reuse_ctx in repository https://gitbox.apache.org/repos/asf/apisix.git at 1c9cf7671 test: make a try This branch includes the following new commits: new 1c9cf7671 test: make a try The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference.
[apisix] 01/01: test: make a try
This is an automated email from the ASF dual-hosted git repository. membphis pushed a commit to branch test_reuse_ctx in repository https://gitbox.apache.org/repos/asf/apisix.git commit 1c9cf7671cb4f42c5b77da0d49dff2b5015407fd Author: YuanSheng Wang AuthorDate: Fri Aug 12 15:36:26 2022 +0800 test: make a try --- apisix/init.lua | 4 1 file changed, 4 insertions(+) diff --git a/apisix/init.lua b/apisix/init.lua index 17f1740ef..eb097c1e1 100644 --- a/apisix/init.lua +++ b/apisix/init.lua @@ -322,6 +322,10 @@ function _M.http_access_phase() -- always fetch table from the table pool, we don't need a reused api_ctx local api_ctx = core.tablepool.fetch("api_ctx", 0, 32) +if nkeys(api_ctx) != 0 then +error("hit the wrong case") +api_ctx = {} +end ngx_ctx.api_ctx = api_ctx if not verify_tls_client(api_ctx) then
[GitHub] [apisix] monkeyDluffy6017 opened a new pull request, #7658: docs: add the lost part to openwhish doc
monkeyDluffy6017 opened a new pull request, #7658: URL: https://github.com/apache/apisix/pull/7658 ### Description add the lost part to openwhish doc ### Checklist - [x] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new features added to this PR - [ ] I have added tests corresponding to this change - [x] I have updated the documentation to reflect this change - [x] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix] branch master updated: chore: upgrade lua-resty-expr (#7655)
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 e91e67497 chore: upgrade lua-resty-expr (#7655) e91e67497 is described below commit e91e674971d36b288a507987d1f29181ff4eccf4 Author: tzssangglass AuthorDate: Fri Aug 12 15:30:33 2022 +0800 chore: upgrade lua-resty-expr (#7655) --- rockspec/apisix-master-0.rockspec | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rockspec/apisix-master-0.rockspec b/rockspec/apisix-master-0.rockspec index 2eb266d2c..7949735b2 100644 --- a/rockspec/apisix-master-0.rockspec +++ b/rockspec/apisix-master-0.rockspec @@ -60,7 +60,7 @@ dependencies = { "binaryheap = 0.4", "api7-dkjson = 0.1.1", "resty-redis-cluster = 1.02-4", -"lua-resty-expr = 1.3.1", +"lua-resty-expr = 1.3.2", "graphql = 0.0.2", "argparse = 0.7.1-1", "luasocket = 3.0rc1-2",
Error while running github feature from .asf.yaml in apisix!
An error occurred while running github feature in .asf.yaml!: 422 {"message": "Validation Failed", "errors": ["A repository cannot have more than 20 topics."], "documentation_url": "https://docs.github.com/rest/reference/repos#replace-all-repository-topics"}
[GitHub] [apisix] spacewander merged pull request #7655: chore: upgrade lua-resty-expr
spacewander merged PR #7655: URL: https://github.com/apache/apisix/pull/7655 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander opened a new pull request, #7657: feat: backport commits to 2.13
spacewander opened a new pull request, #7657: URL: https://github.com/apache/apisix/pull/7657 Thanks for the report of @redynasc Signed-off-by: spacewander ### 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 features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [ ] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] hyt05 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
hyt05 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212812990 I did an uninstall and deleted the data in etcd and redeploy also couldn't get it to work -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] hyt05 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
hyt05 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212809867 > please show me your `kubectl get pods apisix-ingress-controller` command's result. thanks. ![image](https://user-images.githubusercontent.com/48778924/184305409-32269ffc-416c-4b75-bd93-7cf25fc1e9e0.png) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944181561 ## apisix/plugins/grpc-transcode/response.lua: ## @@ -14,18 +14,77 @@ -- See the License for the specific language governing permissions and -- limitations under the License. -- -local util = require("apisix.plugins.grpc-transcode.util") +local util= require("apisix.plugins.grpc-transcode.util") +local grpc_proto = require("apisix.plugins.grpc-transcode.proto") local core = require("apisix.core") local pb = require("pb") local ngx= ngx local string = string +local ngx_decode_base64 = ngx.decode_base64 +local ipairs = ipairs +local pcall = pcall -return function(ctx, proto, service, method, pb_option) + +local function handle_error_response(status_detail_type) +local headers = ngx.resp.get_headers() +local grpc_status = headers["grpc-status-details-bin"] +if grpc_status then +grpc_status = ngx_decode_base64(grpc_status) +if grpc_status == nil then +ngx.arg[1] = "grpc-status-details-bin is not base64 format" +return "grpc-status-details-bin is not base64 format" +end + +local status_pb_state = grpc_proto.fetch_status_pb_state() +local old_pb_state = pb.state(status_pb_state) + +local decoded_grpc_status = pb.decode("grpc.status.ErrorStatus", grpc_status) Review Comment: 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 go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] jackkkkklee commented on a diff in pull request #7634: feat: add openfunction plugin
jacklee commented on code in PR #7634: URL: https://github.com/apache/apisix/pull/7634#discussion_r944180771 ## apisix/plugins/openfunction.lua: ## @@ -0,0 +1,36 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor license agreements. See the NOTICE file distributed with +-- this work for additional information regarding copyright ownership. +-- The ASF licenses this file to You under the Apache License, Version 2.0 +-- (the "License"); you may not use this file except in compliance with +-- the License. You may obtain a copy of the License at +-- +-- http://www.apache.org/licenses/LICENSE-2.0 +-- +-- Unless required by applicable law or agreed to in writing, software +-- distributed under the License is distributed on an "AS IS" BASIS, +-- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. +-- See the License for the specific language governing permissions and +-- limitations under the License. +-- +local ngx_encode_base64 = ngx.encode_base64 +local plugin_name, plugin_version, priority = "openfunction", 0.1, -1902 + +local openfunction_authz_schema = { +service_token = {type = "string"} +} + +local function request_processor(conf, ctx, params) +local headers = params.headers or {} +-- setting authorization headers if not already set +if not headers["Authorization"] and conf.authorization Review Comment: Thanks for the reminding. Lower-case for headers is needed if you mean the local param 'headers', or you mean the header of docs above? Test case added. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1235: request help: Does apisixroute v2beta3 support multiple plugins?
tao12345666333 commented on issue #1235: URL: https://github.com/apache/apisix-ingress-controller/issues/1235#issuecomment-1212795915 From the results of the above Admin API, it does not include the plugin, as you expect. In the situation you are currently experiencing, the client request is actually failing, right? This requires you to make the request again and check the APISIX logs -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944170147 ## t/plugin/grpc-transcode3.t: ## @@ -122,3 +122,447 @@ POST /grpctest Content-Type: application/json --- response_body chomp {"message":"Hello world, name: Joe, age: 1, name: Jake, age: 2"} + + + +=== TEST 3: set proto (id: 1, get error response from rpc) +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test + +local code, body = t('/apisix/admin/proto/1', + ngx.HTTP_PUT, + [[{ +"content" : "syntax = \"proto3\"; + package helloworld; + service Greeter { + rpc GetErrResp (HelloRequest) returns (HelloReply) {} + } + message HelloRequest { + string name = 1; + repeated string items = 2; + } + message HelloReply { + string message = 1; + repeated string items = 2; + } + message ErrorDetail { + int64 code = 1; + string message = 2; + string type = 3; + }" + }]] +) + +if code >= 300 then +ngx.status = code +end +ngx.say(body) +} +} +--- request Review Comment: What do you mean? section 3 should be merge into section 1, or section 3 should be after section 1? -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] tao12345666333 commented on issue #1242: request help: After removing apisix-ingress-controller it will not start
tao12345666333 commented on issue #1242: URL: https://github.com/apache/apisix-ingress-controller/issues/1242#issuecomment-1212793580 please show me your `kubectl get pods apisix-ingress-controller` command's result. thanks. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944167885 ## t/plugin/grpc-transcode3.t: ## @@ -122,3 +122,447 @@ POST /grpctest Content-Type: application/json --- response_body chomp {"message":"Hello world, name: Joe, age: 1, name: Jake, age: 2"} + + + +=== TEST 3: set proto (id: 1, get error response from rpc) +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test + +local code, body = t('/apisix/admin/proto/1', + ngx.HTTP_PUT, + [[{ +"content" : "syntax = \"proto3\"; + package helloworld; + service Greeter { + rpc GetErrResp (HelloRequest) returns (HelloReply) {} + } + message HelloRequest { + string name = 1; + repeated string items = 2; + } + message HelloReply { + string message = 1; + repeated string items = 2; + } + message ErrorDetail { + int64 code = 1; + string message = 2; + string type = 3; + }" + }]] +) + +if code >= 300 then +ngx.status = code +end +ngx.say(body) +} +} +--- request +GET /t +--- response_body +passed +--- no_error_log +[error] + + + +=== TEST 4: set routes (id: 1, get error response from rpc) Review Comment: 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 go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-java-plugin-runner] branch main updated: feat: release 0.3.0 (#183)
This is an automated email from the ASF dual-hosted git repository. tzssangglass pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/apisix-java-plugin-runner.git The following commit(s) were added to refs/heads/main by this push: new 5f9a418 feat: release 0.3.0 (#183) 5f9a418 is described below commit 5f9a418ba0c56feb437d6194be54bba1b954677a Author: tzssangglass AuthorDate: Fri Aug 12 14:50:51 2022 +0800 feat: release 0.3.0 (#183) --- CHANGELOG.md | 14 ++ Makefile | 2 +- pom.xml| 2 +- runner-core/pom.xml| 4 ++-- runner-dist/apisix-runner-bin-dist/pom.xml | 2 +- runner-dist/apisix-runner-src-dist/pom.xml | 2 +- runner-dist/pom.xml| 4 ++-- runner-plugin-sdk/pom.xml | 2 +- runner-plugin/pom.xml | 4 ++-- runner-starter/pom.xml | 4 ++-- sample/pom.xml | 4 ++-- 11 files changed, 29 insertions(+), 15 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 430164e..941ba3b 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -23,9 +23,23 @@ title: Changelog ## Table of Contents +- [0.3.0](#030) - [0.2.0](#020) - [0.1.0](#010) +## 0.3.0 + +This release mainly provides the ability to get headers from upstream, and support download the project from Maven Center. + +### Change + +- rename the name of the function that gets all the headers. [132](https://github.com/apache/apisix-java-plugin-runner/pull/132) + +### Core + +- support filter upstream response headers. [164](https://github.com/apache/apisix-java-plugin-runner/pull/164) +- support hot reload of plugin filters. [158](https://github.com/apache/apisix-java-plugin-runner/pull/158) + ## 0.2.0 This release mainly provides the ability to get variables and request body. diff --git a/Makefile b/Makefile index 6388a61..d5766b6 100644 --- a/Makefile +++ b/Makefile @@ -17,7 +17,7 @@ SHELL := /bin/bash -o pipefail -VERSION ?= 0.3.0-SNAPSHOT +VERSION ?= 0.3.0 RELEASE_SRC = apisix-java-plugin-runner-${VERSION}-src .PHONY: release-src diff --git a/pom.xml b/pom.xml index 50d97a8..845fa0b 100644 --- a/pom.xml +++ b/pom.xml @@ -22,7 +22,7 @@ org.apache.apisix apisix-plugin-runner -0.3.0-SNAPSHOT +0.3.0 org.apache diff --git a/runner-core/pom.xml b/runner-core/pom.xml index aaeaede..41bd98e 100644 --- a/runner-core/pom.xml +++ b/runner-core/pom.xml @@ -24,7 +24,7 @@ org.apache.apisix apisix-plugin-runner -0.3.0-SNAPSHOT +0.3.0 apisix-runner-core @@ -36,7 +36,7 @@ org.apache.apisix apisix-runner-plugin-sdk -0.3.0-SNAPSHOT +0.3.0 org.springframework.boot diff --git a/runner-dist/apisix-runner-bin-dist/pom.xml b/runner-dist/apisix-runner-bin-dist/pom.xml index 48043e3..1c0f0c5 100644 --- a/runner-dist/apisix-runner-bin-dist/pom.xml +++ b/runner-dist/apisix-runner-bin-dist/pom.xml @@ -23,7 +23,7 @@ org.apache.apisix apisix-runner-dist -0.3.0-SNAPSHOT +0.3.0 apisix-runner-bin-dist diff --git a/runner-dist/apisix-runner-src-dist/pom.xml b/runner-dist/apisix-runner-src-dist/pom.xml index ab817be..d3c2d91 100644 --- a/runner-dist/apisix-runner-src-dist/pom.xml +++ b/runner-dist/apisix-runner-src-dist/pom.xml @@ -23,7 +23,7 @@ org.apache.apisix apisix-runner-dist -0.3.0-SNAPSHOT +0.3.0 apisix-runner-src-dist diff --git a/runner-dist/pom.xml b/runner-dist/pom.xml index f2e9bdf..a092b1e 100644 --- a/runner-dist/pom.xml +++ b/runner-dist/pom.xml @@ -23,7 +23,7 @@ org.apache.apisix apisix-plugin-runner -0.3.0-SNAPSHOT +0.3.0 apisix-runner-dist @@ -35,7 +35,7 @@ org.apache.apisix apisix-runner-starter -0.3.0-SNAPSHOT +0.3.0 diff --git a/runner-plugin-sdk/pom.xml b/runner-plugin-sdk/pom.xml index b52133f..7554648 100644 --- a/runner-plugin-sdk/pom.xml +++ b/runner-plugin-sdk/pom.xml @@ -24,7 +24,7 @@ org.apache.apisix apisix-plugin-runner -0.3.0-SNAPSHOT +0.3.0 apisix-runner-plugin-sdk diff --git a/runner-plugin/pom.xml b/runner-plugin/pom.xml index 9bffaf2..f0fba48 100644 --- a/runner-plugin/pom.xml +++ b/runner-plugin/pom.xml @@ -23,7 +23,7 @@ org.apache.apisix apisix-plugin-runner -0.3.0-SNAPSHOT +0.3.0 apisix-runner-plugin @@ -35,7 +35,7 @@ org.apache.apisix apisix-runner-plugin-sdk -0.3.0-SNAPSHOT +0.3.0 com.google.code.gson diff --git
[GitHub] [apisix-java-plugin-runner] tzssangglass merged pull request #183: feat: release 0.3.0
tzssangglass merged PR #183: URL: https://github.com/apache/apisix-java-plugin-runner/pull/183 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944161456 ## docs/en/latest/plugins/grpc-transcode.md: ## @@ -44,6 +44,9 @@ APISIX takes in an HTTP request, transcodes it and forwards it to a gRPC service | method| string | True | | Method name of the gRPC service. | | deadline | number | False | 0 | Deadline for the gRPC service in ms. | | pb_option | array[string([pb_option_def](#options-for-pb_option))] | False | | protobuf options.| +| show_status_in_body | boolean | False | False | Whether to display the parsed `grpc-status-details-bin` in the response body | Review Comment: 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 go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] nfrankel opened a new issue, #7656: bug: HTTP methods cannot be configured in stand-alone mode
nfrankel opened a new issue, #7656: URL: https://github.com/apache/apisix/issues/7656 ### Current Behavior I'm using Apache APISIX 2.14.1. I've configured stand-alone mode: ``` apisix: enable_admin: false config_center: yaml ``` I've configured a simple route: ``` routes: - uri: / methods: [GET, PUT, POST] upstream: type: roundrobin nodes: "httpbin.org": 1 #END ``` When I execute a PUT request, I get a 405: ``` >curl --head -X POST localhost:9080/ HTTP/1.1 405 METHOD NOT ALLOWED Content-Type: text/html; charset=utf-8 Content-Length: 178 Connection: keep-alive Date: Fri, 12 Aug 2022 06:23:36 GMT Allow: GET, OPTIONS, HEAD Access-Control-Allow-Origin: * Access-Control-Allow-Credentials: true Server: APISIX/2.14.1 ``` ### Expected Behavior Stand-alone mode is *the* way to run Apache APISIX in production while following GitOps patterns. I'd expect you can configure HTTP methods the same way as for standard mode. ### Error Logs _No response_ ### Steps to Reproduce I've described the steps in the first section ### Environment - APISIX version (run `apisix version`): ``` /usr/local/openresty/luajit/bin/luajit ./apisix/cli/apisix.lua version 2.14.1 ``` - Operating system (run `uname -a`): ``` Linux 6cd28012c29c 5.10.104-linuxkit #1 SMP PREEMPT Thu Mar 17 17:05:54 UTC 2022 aarch64 Linux ``` - OpenResty / Nginx version (run `openresty -V` or `nginx -V`): ``` nginx version: openresty/1.19.9.1 built by gcc 10.3.1 20210424 (Alpine 10.3.1_git20210424) built with OpenSSL 1.1.1g 21 Apr 2020 TLS SNI support enabled configure arguments: --prefix=/usr/local/openresty/nginx --with-cc-opt='-O2 -DAPISIX_BASE_VER=1.19.9.1.6 -DNGX_LUA_ABORT_AT_PANIC -I/usr/local/openresty/zlib/include -I/usr/local/openresty/pcre/include -I/usr/local/openresty/openssl111/include' --add-module=../ngx_devel_kit-0.3.1 --add-module=../echo-nginx-module-0.62 --add-module=../xss-nginx-module-0.06 --add-module=../ngx_coolkit-0.2 --add-module=../set-misc-nginx-module-0.32 --add-module=../form-input-nginx-module-0.12 --add-module=../encrypted-session-nginx-module-0.08 --add-module=../srcache-nginx-module-0.32 --add-module=../ngx_lua-0.10.20 --add-module=../ngx_lua_upstream-0.07 --add-module=../headers-more-nginx-module-0.33 --add-module=../array-var-nginx-module-0.05 --add-module=../memc-nginx-module-0.19 --add-module=../redis2-nginx-module-0.15 --add-module=../redis-nginx-module-0.3.7 --add-module=../ngx_stream_lua-0.0.10 --with-ld-opt='-Wl,-rpath,/usr/local/openresty/luajit/lib -Wl,-rpath,/usr/local/openresty/wasmtime- c-api/lib -L/usr/local/openresty/zlib/lib -L/usr/local/openresty/pcre/lib -L/usr/local/openresty/openssl111/lib -Wl,-rpath,/usr/local/openresty/zlib/lib:/usr/local/openresty/pcre/lib:/usr/local/openresty/openssl111/lib' --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../mod_dubbo --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../ngx_multi_upstream_module --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../apisix-nginx-module --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../apisix-nginx-module/src/stream --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../wasm-nginx-module --add-module=/tmp/tmp.oYTC5eok64/openresty-1.19.9.1/../lua-var-nginx-module --with-poll_module --with-pcre-jit --with-stream --with-stream_ssl_module --with-stream_ssl_preread_module --with-http_v2_module --without-mail_pop3_module --without-mail_imap_module --without-mail_smtp_module --with-http_stub_status_module --with-http_realip_module --with-http_addition_module --with-http_auth_request_m odule --with-http_secure_link_module --with-http_random_index_module --with-http_gzip_static_module --with-http_sub_module --with-http_dav_module --with-http_flv_module --with-http_mp4_module --with-http_gunzip_module --with-threads --with-compat --with-stream --with-http_ssl_module ``` - etcd version, if relevant (run `curl http://127.0.0.1:9090/v1/server_info`): - - APISIX Dashboard version, if relevant: - - Plugin runner version, for issues related to plugin runners: - - LuaRocks version, for installation issues (run `luarocks --version`): ``` /usr/local/bin/luarocks 3.8.0 LuaRocks main command-line interface ``` -- 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...@apisix.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass opened a new pull request, #7655: chore: upgrade lua-resty-expr
tzssangglass opened a new pull request, #7655: URL: https://github.com/apache/apisix/pull/7655 ### Description Fixes # (issue) ### Checklist - [x] I have explained the need for this PR and the problem it solves - [x] I have explained the changes or the new features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [ ] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] fanchangjifen commented on pull request #7099: init elasticsearch-logger.lua
fanchangjifen commented on PR #7099: URL: https://github.com/apache/apisix/pull/7099#issuecomment-1212774744 > > @spacewander @tzssangglass #7643 for this issue,It's up to you > > Thanks for your contribution, let's do it. @ccxhwmy As mentioned above, you should use the elasticsearch bulk API (endpoint/index/_bulk)..but I noticed you didn't. @tzssangglass Why not recommend testing on my basis or use my plugin directly. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on a diff in pull request #7634: feat: add openfunction plugin
spacewander commented on code in PR #7634: URL: https://github.com/apache/apisix/pull/7634#discussion_r944148528 ## docs/en/latest/plugins/openfunction.md: ## @@ -0,0 +1,122 @@ +--- +title: openfunction +keywords: + - APISIX + - Plugin + - OpenFunction + - openfunction +description: This document contains information about the CNCF OpenFunction Plugin. +--- + + + +## Description + +The `openfunction` Plugin is used to integrate APISIX with [CNCF OpenFunction](https://openfunction.dev/) serverless platform. + +This Plugin can be configured on a Route and requests will be send to the configured OpenWhish API endpoint as the upstream. + +## Attributes + +| Name | Type| Required | Default | Valid values | Description | +| - | --- | | --- | | -- | +| function_uri | string | True | | | function uri. For example, `https://localhost:30858/default/function-sample`. | +| ssl_verify| boolean | False| true| | When set to `true` verifies the SSL certificate. | +| service_token | string | False| | | The token format is 'xx:xx' which support basic auth for ingress controller, . | +| timeout | integer | False| 6ms | [1, 6]ms | OpenFunction action and HTTP call timeout in ms. | +| keepalive | boolean | False| true| | When set to `true` keeps the connection alive for reuse. | +| keepalive_timeout | integer | False| 6ms | [1000,...]ms | Time is ms for connection to remain idle without closing. | +| keepalive_pool| integer | False| 5 | [1,...] | Maximum number of requests that can be sent on this connection before closing it. | + +:::note + +The `timeout` attribute sets the time taken by the OpenFunction to execute, and the timeout for the HTTP client in APISIX. OpenFunction calls may take time to pull the runtime image and start the container. So, if the value is set too small, it may cause a large number of requests to fail. + +::: + +## Enabling the Plugin + +Before configuring the Plugin, you need to have OpenFunction running. For details, please refer to [Installation](https://openfunction.dev/docs/getting-started/installation/). Review Comment: ```suggestion Before configuring the plugin, you need to have OpenFunction running. For details, please refer to [Installation](https://openfunction.dev/docs/getting-started/installation/). ``` ## docs/en/latest/plugins/openfunction.md: ## @@ -0,0 +1,122 @@ +--- +title: openfunction +keywords: + - APISIX + - Plugin + - OpenFunction + - openfunction +description: This document contains information about the CNCF OpenFunction Plugin. +--- + + + +## Description + +The `openfunction` Plugin is used to integrate APISIX with [CNCF OpenFunction](https://openfunction.dev/) serverless platform. + +This Plugin can be configured on a Route and requests will be send to the configured OpenWhish API endpoint as the upstream. + +## Attributes + +| Name | Type| Required | Default | Valid values | Description | +| - | --- | | --- | | -- | +| function_uri | string | True | | | function uri. For example, `https://localhost:30858/default/function-sample`. | +| ssl_verify| boolean | False| true| | When set to `true` verifies the SSL certificate. | +| service_token | string | False| | | The token format is 'xx:xx' which support basic auth for ingress controller, . | +| timeout | integer | False| 6ms | [1, 6]ms | OpenFunction action and HTTP call timeout in ms. | +| keepalive | boolean | False| true| | When set to `true` keeps the connection alive for reuse. | +| keepalive_timeout | integer | False| 6ms | [1000,...]ms | Time is ms for
[GitHub] [apisix] monkeyDluffy6017 commented on pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on PR #7639: URL: https://github.com/apache/apisix/pull/7639#issuecomment-1212771200 > Please make the CI pass, thanks! I'm working on it. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] monkeyDluffy6017 commented on a diff in pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
monkeyDluffy6017 commented on code in PR #7639: URL: https://github.com/apache/apisix/pull/7639#discussion_r944148237 ## t/plugin/grpc-transcode3.t: ## @@ -122,3 +122,447 @@ POST /grpctest Content-Type: application/json --- response_body chomp {"message":"Hello world, name: Joe, age: 1, name: Jake, age: 2"} + + + +=== TEST 3: set proto (id: 1, get error response from rpc) +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test + +local code, body = t('/apisix/admin/proto/1', + ngx.HTTP_PUT, + [[{ +"content" : "syntax = \"proto3\"; + package helloworld; + service Greeter { + rpc GetErrResp (HelloRequest) returns (HelloReply) {} + } + message HelloRequest { + string name = 1; + repeated string items = 2; + } + message HelloReply { + string message = 1; + repeated string items = 2; + } + message ErrorDetail { + int64 code = 1; + string message = 2; + string type = 3; + }" + }]] +) + +if code >= 300 then +ngx.status = code +end +ngx.say(body) +} +} +--- request +GET /t +--- response_body +passed +--- no_error_log +[error] + + + +=== TEST 4: set routes (id: 1, get error response from rpc) +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test + +local code, body = t('/apisix/admin/routes/1', +ngx.HTTP_PUT, +[[{ +"methods": ["GET", "POST"], +"uri": "/grpctest", +"plugins": { +"grpc-transcode": { +"proto_id": "1", +"service": "helloworld.Greeter", +"method": "GetErrResp" +} +}, +"upstream": { +"scheme": "grpc", +"type": "roundrobin", +"nodes": { +"127.0.0.1:50051": 1 +} +} +}]] +) + +if code >= 300 then +ngx.status = code +end +ngx.say(body) +} +} +--- request +GET /t +--- response_body +passed +--- no_error_log +[error] + + + +=== TEST 5: hit route (error response in header) +--- config +location /t { +content_by_lua_block { +local json = require("toolkit.json") +local t = require("lib.test_admin").test + +local code, body, headers = t('/grpctest?name=world', +ngx.HTTP_GET +) + +ngx.status = code + +ngx.header['grpc-status'] = headers['grpc-status'] +ngx.header['grpc-message'] = headers['grpc-message'] +ngx.header['grpc-status-details-bin'] = headers['grpc-status-details-bin'] + +body = json.encode(body) +ngx.say(body) +} +} +--- request +GET /t +--- response_headers +grpc-status: 14 +grpc-message: Out of service +grpc-status-details-bin: CA4SDk91dCBvZiBzZXJ2aWNlGlcKKnR5cGUuZ29vZ2xlYXBpcy5jb20vaGVsbG93b3JsZC5FcnJvckRldGFpbBIpCAESHFRoZSBzZXJ2ZXIgaXMgb3V0IG9mIHNlcnZpY2UaB3NlcnZpY2U +--- response_body_unlike eval +qr/error/ +--- no_error_log +[error] +--- error_code: 503 + + + +=== TEST 6: set routes (id: 1, show error response in body) +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test + +local code, body = t('/apisix/admin/routes/1', +ngx.HTTP_PUT, +[[{ +"methods": ["GET", "POST"], +"uri": "/grpctest", +"plugins": { +"grpc-transcode": { +"proto_id": "1", +"service": "helloworld.Greeter", +"method": "GetErrResp", +"show_status_in_body": true +} +}, +"upstream": { +"scheme": "grpc", +"type": "roundrobin", +"nodes": { +"127.0.0.1:50051": 1 +} +} +}]] +) + +if code >= 300 then +ngx.status = code +end
[GitHub] [apisix] spacewander commented on pull request #7639: feat: convert grpc-status-details-bin in header to HTTP response body…
spacewander commented on PR #7639: URL: https://github.com/apache/apisix/pull/7639#issuecomment-1212768801 Please make the CI pass, thanks! -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] codecov-commenter commented on pull request #1243: feat: update secret referenced by ingress
codecov-commenter commented on PR #1243: URL: https://github.com/apache/apisix-ingress-controller/pull/1243#issuecomment-1212761045 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) Report > Merging [#1243](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) (f8ae028) into [master](https://codecov.io/gh/apache/apisix-ingress-controller/commit/fcfa1882957a1d111c616c1ef646b98a0fb6a70f?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) (fcfa188) will **decrease** coverage by `0.70%`. > The diff coverage is `0.00%`. ```diff @@Coverage Diff @@ ## master#1243 +/- ## == - Coverage 42.78% 42.08% -0.71% == Files 73 73 Lines6477 6585 +108 == Hits 2771 2771 - Misses 3409 3517 +108 Partials 297 297 ``` | [Impacted Files](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) | Coverage Δ | | |---|---|---| | [pkg/providers/apisix/translation/apisix\_ssl.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy9hcGlzaXgvdHJhbnNsYXRpb24vYXBpc2l4X3NzbC5nbw==) | `0.00% <0.00%> (ø)` | | | [pkg/providers/ingress/ingress.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy9pbmdyZXNzL2luZ3Jlc3MuZ28=) | `4.09% <0.00%> (-1.13%)` | :arrow_down: | | [pkg/providers/ingress/provider.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy9pbmdyZXNzL3Byb3ZpZGVyLmdv) | `0.00% <0.00%> (ø)` | | | [...providers/ingress/translation/annotations/types.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy9pbmdyZXNzL3RyYW5zbGF0aW9uL2Fubm90YXRpb25zL3R5cGVzLmdv) | `100.00% <ø> (ø)` | | | [pkg/providers/translation/util.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy90cmFuc2xhdGlvbi91dGlsLmdv) | `15.00% <0.00%> (-3.19%)` | :arrow_down: | | [pkg/providers/utils/ingress\_status.go](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1243/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-cGtnL3Byb3ZpZGVycy91dGlscy9pbmdyZXNzX3N0YXR1cy5nbw==) | `0.00% <0.00%> (ø)` | | :mega: We’re building smart automated test selection to slash your CI/CD build times. [Learn more](https://about.codecov.io/iterative-testing/?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation) -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] SkyeYoung closed issue #1110: [Site][Discussion]: maintain a VSCode config dir
SkyeYoung closed issue #1110: [Site][Discussion]: maintain a VSCode config dir URL: https://github.com/apache/apisix-website/issues/1110 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch asf-site updated (b26fde9ec27 -> 0481897fcc1)
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch asf-site in repository https://gitbox.apache.org/repos/asf/apisix-website.git discard b26fde9ec27 deploy: c40429a02a23f1a7eb5daa559c6700897fff8a84 new 0481897fcc1 deploy: bdd3e823ced08b417b9721400e21b1e22f776d36 This update added new revisions after undoing existing revisions. That is to say, some revisions that were in the old version of the branch are not in the new version. This situation occurs when a user --force pushes a change and generates a repository containing something like this: * -- * -- B -- O -- O -- O (b26fde9ec27) \ N -- N -- N refs/heads/asf-site (0481897fcc1) You should already have received notification emails for all of the O revisions, and so the following emails describe only the N revisions from the common base, B. Any revisions marked "omit" are not gone; other references still refer to them. Any revisions marked "discard" are gone forever. The 1 revisions listed above as "new" are entirely new to this repository and will be described in separate emails. The revisions listed as "add" were already present in the repository and have only been added to this reference. Summary of changes: 404.html | 4 ++-- .../index.html| 4 ++-- .../Apache-APISIX-From-OpenSource-Commercialization/index.html| 8 articles/Apache-APISIX-Incubator-Journey/index.html | 4 ++-- articles/Apache-APISIX-Kubernetes-Ingress/index.html | 4 ++-- articles/Apache-APISIX-in-China-Mobile-Cloud/index.html | 8 .../index.html| 4 ++-- .../index.html| 4 ++-- articles/Rendering-Community-Events-Using-ECharts/index.html | 4 ++-- articles/Speed-Limiting-With-Apache-APISIX/index.html | 4 ++-- articles/Test-Apache-APISIX-Resilience-With-Chaos-Mesh/index.html | 4 ++-- articles/The-Appeal-of-OpenSource/index.html | 4 ++-- articles/The-Evolution-of-Apache-APISIX/index.html| 4 ++-- .../index.html| 4 ++-- articles/archive/index.html | 4 ++-- articles/atom.xml | 6 +++--- articles/index.html | 6 +++--- articles/page/2/index.html| 4 ++-- articles/rss.xml | 6 +++--- assets/js/03b491a1.58f31295.js| 1 + assets/js/03b491a1.95091bf1.js| 1 - assets/js/{09b17eb5.fcc723b3.js => 09b17eb5.7e476228.js} | 2 +- assets/js/{0d0b38bc.0e5b9d91.js => 0d0b38bc.8448a023.js} | 2 +- assets/js/{507e1a4c.5f46bb1b.js => 507e1a4c.e6c0b754.js} | 2 +- assets/js/{66fbb9c2.09ee8a7d.js => 66fbb9c2.7571b96a.js} | 2 +- assets/js/7af3052c.ee6fe00d.js| 1 + assets/js/7af3052c.fee870a4.js| 1 - assets/js/8e784bf3.26c4f8aa.js| 1 + assets/js/8e784bf3.4c75e021.js| 1 - assets/js/{8f700fa6.66daaf20.js => 8f700fa6.0864c2b9.js} | 2 +- assets/js/{90fc8a94.5a99e83b.js => 90fc8a94.5ef0e13a.js} | 2 +- assets/js/{941960b1.720b31f2.js => 941960b1.d0d7ebce.js} | 2 +- assets/js/a056998a.9237aade.js| 1 + assets/js/a056998a.ae708590.js| 1 - assets/js/adbb350b.589667cb.js| 1 + assets/js/adbb350b.706a9c56.js| 1 - assets/js/b1fc098c.d4661a3e.js| 1 - assets/js/b1fc098c.f5bb2155.js| 1 + assets/js/c6ebe4e5.e810e51d.js| 1 + assets/js/c6ebe4e5.ec270492.js| 1 - assets/js/{d739b11f.14f084ac.js => d739b11f.530dc8f4.js} | 2 +- assets/js/{ef816cbf.228e4fff.js => ef816cbf.dc45de06.js} | 2 +- assets/js/runtime~main.92405fc2.js| 1 + assets/js/runtime~main.ddae6623.js| 1 - blog/page/10/index.html | 2 +- blog/page/11/index.html | 2 +- blog/page/12/index.html | 2 +- blog/page/13/index.html | 2 +- blog/page/14/index.html | 2 +- blog/page/15/index.html
[GitHub] [apisix-ingress-controller] lingsamuel opened a new pull request, #1243: feat: update secret referenced by ingress
lingsamuel opened a new pull request, #1243: URL: https://github.com/apache/apisix-ingress-controller/pull/1243 Signed-off-by: Ling Samuel ### Type of change: - [x] Bugfix ### What this PR does / why we need it: -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] wolgod commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
wolgod commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1212758886 > > but the rest of the split access-log will also occupy the cache, unless which logs are manually deleted > > we can use `max_kept` in `log-rotate` plugins to delete old logs files. ref: https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/log-rotate.md yes, but I'm on 4 5g per hour, unless it's only reserved for an hour, the cache will still be high -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: feat: devcontainer (#1283)
This is an automated email from the ASF dual-hosted git repository. guoqi 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 bdd3e823ced feat: devcontainer (#1283) bdd3e823ced is described below commit bdd3e823ced08b417b9721400e21b1e22f776d36 Author: Young AuthorDate: Fri Aug 12 14:02:02 2022 +0800 feat: devcontainer (#1283) --- .devcontainer/Dockerfile| 14 +++ .devcontainer/base.Dockerfile | 55 + .devcontainer/devcontainer.json | 51 ++ README.md | 50 - 4 files changed, 158 insertions(+), 12 deletions(-) diff --git a/.devcontainer/Dockerfile b/.devcontainer/Dockerfile new file mode 100644 index 000..2b28681164d --- /dev/null +++ b/.devcontainer/Dockerfile @@ -0,0 +1,14 @@ +# [Choice] Node.js version (use -bullseye variants on local arm64/Apple Silicon): 18, 16, 14, 18-bullseye, 16-bullseye, 14-bullseye, 18-buster, 16-buster, 14-buster +ARG VARIANT=16-bullseye +FROM mcr.microsoft.com/vscode/devcontainers/javascript-node:0-${VARIANT} + +# [Optional] Uncomment this section to install additional OS packages. +# RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \ +# && apt-get -y install --no-install-recommends + +# [Optional] Uncomment if you want to install an additional version of node using nvm +# ARG EXTRA_NODE_VERSION=10 +# RUN su node -c "source /usr/local/share/nvm/nvm.sh && nvm install ${EXTRA_NODE_VERSION}" + +# [Optional] Uncomment if you want to install more global node modules +# RUN su node -c "npm install -g " diff --git a/.devcontainer/base.Dockerfile b/.devcontainer/base.Dockerfile new file mode 100644 index 000..a3e066fd912 --- /dev/null +++ b/.devcontainer/base.Dockerfile @@ -0,0 +1,55 @@ +# [Choice] Node.js version (use -bullseye variants on local arm64/Apple Silicon): 18-bullseye, 16-bullseye, 14-bullseye, 18-buster, 16-buster, 14-buster +ARG VARIANT=16-bullseye +FROM node:${VARIANT} + +# [Option] Install zsh +ARG INSTALL_ZSH="true" +# [Option] Upgrade OS packages to their latest versions +ARG UPGRADE_PACKAGES="true" + +# Install needed packages, yarn, nvm and setup non-root user. Use a separate RUN statement to add your own dependencies. +ARG USERNAME=node +ARG USER_UID=1000 +ARG USER_GID=$USER_UID +ARG NPM_GLOBAL=/usr/local/share/npm-global +ENV NVM_DIR=/usr/local/share/nvm +ENV NVM_SYMLINK_CURRENT=true \ +PATH=${NPM_GLOBAL}/bin:${NVM_DIR}/current/bin:${PATH} +COPY library-scripts/*.sh library-scripts/*.env /tmp/library-scripts/ +RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \ +# Remove imagemagick due to https://security-tracker.debian.org/tracker/CVE-2019-10131 +&& apt-get purge -y imagemagick imagemagick-6-common \ +# Install common packages, non-root user, update yarn and install nvm +&& bash /tmp/library-scripts/common-debian.sh "${INSTALL_ZSH}" "${USERNAME}" "${USER_UID}" "${USER_GID}" "${UPGRADE_PACKAGES}" "true" "true" \ +# Install yarn, nvm +&& rm -rf /opt/yarn-* /usr/local/bin/yarn /usr/local/bin/yarnpkg \ +&& bash /tmp/library-scripts/node-debian.sh "${NVM_DIR}" "none" "${USERNAME}" \ +# Configure global npm install location, use group to adapt to UID/GID changes +&& if ! cat /etc/group | grep -e "^npm:" > /dev/null 2>&1; then groupadd -r npm; fi \ +&& usermod -a -G npm ${USERNAME} \ +&& umask 0002 \ +&& mkdir -p ${NPM_GLOBAL} \ +&& touch /usr/local/etc/npmrc \ +&& chown ${USERNAME}:npm ${NPM_GLOBAL} /usr/local/etc/npmrc \ +&& chmod g+s ${NPM_GLOBAL} \ +&& npm config -g set prefix ${NPM_GLOBAL} \ +&& sudo -u ${USERNAME} npm config -g set prefix ${NPM_GLOBAL} \ +# Install eslint +&& su ${USERNAME} -c "umask 0002 && npm install -g eslint" \ +&& npm cache clean --force > /dev/null 2>&1 \ +# Install python-is-python3 on bullseye to prevent node-gyp regressions +&& . /etc/os-release \ +&& if [ "${VERSION_CODENAME}" = "bullseye" ]; then apt-get -y install --no-install-recommends python-is-python3; fi \ +# Clean up +&& apt-get autoremove -y && apt-get clean -y && rm -rf /var/lib/apt/lists/* /root/.gnupg /tmp/library-scripts + +# [Optional] Uncomment this section to install additional OS packages. +# RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \ +# && apt-get -y install --no-install-recommends + +# [Optional] Uncomment if you want to install an additional version of node using nvm +# ARG EXTRA_NODE_VERSION=10 +# RUN su node -c "source /usr/local/share/nvm/nvm.sh && nvm install ${EXTRA_NODE_VERSION}" + +# [Optional] Uncomment if you want to install more global node modules +# RUN su node -c "npm install -g "" \ No newline at end of file diff --git a/.devcontainer/devcontainer.json
[GitHub] [apisix-website] guoqqqi merged pull request #1283: feat: devcontainer
guoqqqi merged PR #1283: URL: https://github.com/apache/apisix-website/pull/1283 -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on a diff in pull request #7647: docs: introduce the paging and filtering
tzssangglass commented on code in PR #7647: URL: https://github.com/apache/apisix/pull/7647#discussion_r944137129 ## docs/zh/latest/admin-api.md: ## @@ -29,6 +29,89 @@ Admin API 是为 Apache APISIX 服务的一组 API,我们可以将参数传递 在下面出现的 `X-API-KEY` 指的是 `conf/config.yaml` 文件中的 `apisix.admin_key.key`,它是 Admin API 的访问 token。 +## V3 + +Admin API 在 V3 版本中做了一些不向下兼容的调整,以及支持更多特性。 + +### 响应体格式 + +1. 移除响应体中的 `action` 字段; +2. 调整获取资源列表时的响应体结构,新的响应体结构示例: + +```json +{ +"count":2, +"list":[ +{ +... +}, +{ +... +} +] +} +``` + +### 分页 Review Comment: done, and others are also. -- 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...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org