Re: [I] help request: Issue with APISIX Route Not Working via Proxy Domain [apisix]

2025-06-03 Thread via GitHub
ajaysehwal closed issue #12260: help request: Issue with APISIX Route Not Working via Proxy Domain URL: https://github.com/apache/apisix/issues/12260 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: [I] bug: After the forward-auth verification is passed, the upstream cannot obtain the request body (payload) with 504 Timeout [apisix]

2025-06-03 Thread via GitHub
jmz67 commented on issue #11200: URL: https://github.com/apache/apisix/issues/11200#issuecomment-2938794789 i meet the same problem, after i sent the request to the third, the upstream service timed out -- This is an automated message from the Apache Git Service. To respond to the me

Re: [PR] feat: disable editing of IDs while editing [apisix-dashboard]

2025-06-03 Thread via GitHub
SkyeYoung commented on code in PR #3098: URL: https://github.com/apache/apisix-dashboard/pull/3098#discussion_r2125766692 ## src/components/form-slice/FormSectionGeneral.tsx: ## @@ -46,13 +46,14 @@ const FormItemID = () => { }; export type FormSectionGeneralProps = { + /**

Re: [PR] feat: disable editing of IDs while editing [apisix-dashboard]

2025-06-03 Thread via GitHub
Copilot commented on code in PR #3098: URL: https://github.com/apache/apisix-dashboard/pull/3098#discussion_r2125760500 ## src/components/form-slice/FormSectionGeneral.tsx: ## @@ -46,13 +46,14 @@ const FormItemID = () => { }; export type FormSectionGeneralProps = { + /** wi

Re: [PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.2 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] closed pull request #3097: chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.2 URL: https://github.com/apache/apisix-dashboard/pull/3097 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.80.2 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] opened a new pull request, #3099: URL: https://github.com/apache/apisix-dashboard/pull/3099 Bumps [@tanstack/react-query](https://github.com/TanStack/query/tree/HEAD/packages/react-query) from 5.74.4 to 5.80.2. Release notes Sourced from https://github.com/TanStack

(apisix-dashboard) branch dependabot/npm_and_yarn/tanstack/react-query-5.79.2 deleted (was 4dcad0ea2)

2025-06-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/tanstack/react-query-5.79.2 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git was 4dcad0ea2 chore(deps): bump @tanstack/react-query from 5.7

(apisix-dashboard) branch dependabot/npm_and_yarn/tanstack/react-query-5.80.2 created (now c623a9877)

2025-06-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/tanstack/react-query-5.80.2 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git at c623a9877 chore(deps): bump @tanstack/react-query from 5.7

Re: [PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.2 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] commented on PR #3097: URL: https://github.com/apache/apisix-dashboard/pull/3097#issuecomment-2938758435 Superseded by #3099. -- 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

Re: [I] help request: How to upgrade APISIX 3.5 to 3.12 [apisix]

2025-06-03 Thread via GitHub
juzhiyuan commented on issue #12280: URL: https://github.com/apache/apisix/issues/12280#issuecomment-2938668902 Hi @PlasticJonbenetDoll, 1. Check https://github.com/apache/apisix/blob/master/CHANGELOG.md for changes between 3.5 and 3.12. In the same major version (3.x), there should n

Re: [I] bug: zipkin plugin's traceId printed to accessLog are all escaped characters [apisix]

2025-06-03 Thread via GitHub
lchpersonal commented on issue #12271: URL: https://github.com/apache/apisix/issues/12271#issuecomment-2938496332 > Hi [@lchpersonal](https://github.com/lchpersonal), can you share your config.yaml ? [config.zip](https://github.com/user-attachments/files/20585037/config.zip) -- Thi

Re: [I] bug: zipkin plugin's traceId printed to accessLog are all escaped characters [apisix]

2025-06-03 Thread via GitHub
lchpersonal commented on issue #12271: URL: https://github.com/apache/apisix/issues/12271#issuecomment-2938495060 > Hi [@lchpersonal](https://github.com/lchpersonal), can you share your config.yaml ? --- graphql: max_size: 1048576 apisix: enable_resolv_search_opt: true

Re: [I] bug: zipkin plugin's traceId printed to accessLog are all escaped characters [apisix]

2025-06-03 Thread via GitHub
Baoyuantop commented on issue #12271: URL: https://github.com/apache/apisix/issues/12271#issuecomment-2938437161 Hi @lchpersonal, can you share your config.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

[I] bug: Control Plane fails to start when config.yaml is mounted via volumeMount [apisix]

2025-06-03 Thread via GitHub
vguddoda opened a new issue, #12281: URL: https://github.com/apache/apisix/issues/12281 ### Current Behavior When mounting a ConfigMap containing config.yaml into the APISIX Control Plane container using volumeMounts, APISIX startup fails with an error indicating that the file /usr/l

[PR] feat: disable editing of IDs while editing [apisix-dashboard]

2025-06-03 Thread via GitHub
SkyeYoung opened a new pull request, #3098: URL: https://github.com/apache/apisix-dashboard/pull/3098 Please answer these questions before submitting a pull request, **or your PR will get closed**. **Why submit this pull request?** - [ ] Bugfix - [x] New feature provided

Re: [PR] feat: adds support for nginx variables in service_name param [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on code in PR #12187: URL: https://github.com/apache/apisix/pull/12187#discussion_r2125563602 ## apisix/upstream.lua: ## @@ -297,7 +297,12 @@ function _M.set_by_route(route, api_ctx) return 503, err end -local new_nodes, err = di

(apisix) branch master updated: feat: add embedded apisix dashboard ui (#12276)

2025-06-03 Thread bzp2010
This is an automated email from the ASF dual-hosted git repository. bzp2010 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 58066abc8 feat: add embedded apisix dashboard u

apisix.git: Error while running github feature from master:.asf.yaml

2025-06-03 Thread Apache Infrastructure
An error occurred while processing the github feature in .asf.yaml: Too many GitHub labels/topics - must be <= 20 items! --- With regards, ASF Infra. For further information, please see the .asf.yaml documentation at: https://github.com/apache/infrastructure-asfyaml/blob/main/README.md

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
bzp2010 merged PR #12276: URL: https://github.com/apache/apisix/pull/12276 -- 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...@api

Re: [I] feat: support to expose APISIX Dashboard UI within APISIX [apisix]

2025-06-03 Thread via GitHub
bzp2010 closed issue #12160: feat: support to expose APISIX Dashboard UI within APISIX URL: https://github.com/apache/apisix/issues/12160 -- 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

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on PR #12276: URL: https://github.com/apache/apisix/pull/12276#issuecomment-2938247847 @moonming @juzhiyuan > How should the Dashboard be integrated with APISIX? Should we embed the Dashboard's commit ID as part of the integration? > submodule (I prefer this

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
juzhiyuan commented on PR #12276: URL: https://github.com/apache/apisix/pull/12276#issuecomment-2938231300 > How should the Dashboard be integrated with APISIX? Should we embed the Dashboard's commit ID as part of the integration? submodule or follow https://github.com/apache/apisix/

Re: [PR] feat: add `authorization` attribute for `loki-logger` (#11420) [apisix]

2025-06-03 Thread via GitHub
membphis commented on code in PR #12243: URL: https://github.com/apache/apisix/pull/12243#discussion_r2125390915 ## apisix/plugins/loki-logger.lua: ## @@ -138,11 +147,12 @@ end local function send_http_data(conf, log) +local headers = conf.headers or {} Review Comment:

(apisix-website) branch yzeng25-patch-3 deleted (was dbaedbfbcaa)

2025-06-03 Thread yilialin
This is an automated email from the ASF dual-hosted git repository. yilialin pushed a change to branch yzeng25-patch-3 in repository https://gitbox.apache.org/repos/asf/apisix-website.git was dbaedbfbcaa fix: update commits to 39 to align with pictures and ZH-CN blog The revisions that we

Re: [PR] fix: update commits to 39 to align with pictures and ZH-CN blog [apisix-website]

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

(apisix-website) branch master updated: fix: update commits to 39 to align with pictures and ZH-CN blog (#1920)

2025-06-03 Thread yilialin
This is an automated email from the ASF dual-hosted git repository. yilialin 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 01d752bf63d fix: update commits to 39

Re: [PR] feat: adds support for nginx variables in service_name param [apisix]

2025-06-03 Thread via GitHub
undying commented on code in PR #12187: URL: https://github.com/apache/apisix/pull/12187#discussion_r2124667723 ## apisix/core/utils.lua: ## @@ -461,5 +461,35 @@ function _M.check_tls_bool(fields, conf, plugin_name) end end +--- +-- Checks if a string is an nginx variabl

Re: [I] feat: in standalone mode (without etcd), protobuf with import is not loaded [apisix]

2025-06-03 Thread via GitHub
davendu commented on issue #11486: URL: https://github.com/apache/apisix/issues/11486#issuecomment-2936482807 > Multiple proto files can be merged temporarily using an external tool. Waiting for others to contribute this feature. @Baoyuantop I'm not quite sure if I understand your ide

Re: [PR] feat: adds support for nginx variables in service_name param [apisix]

2025-06-03 Thread via GitHub
undying commented on code in PR #12187: URL: https://github.com/apache/apisix/pull/12187#discussion_r2124362620 ## apisix/upstream.lua: ## @@ -297,7 +297,12 @@ function _M.set_by_route(route, api_ctx) return 503, err end -local new_nodes, err = di

Re: [I] help request: luarocks installation dependency failed [apisix]

2025-06-03 Thread via GitHub
chucklean commented on issue #12274: URL: https://github.com/apache/apisix/issues/12274#issuecomment-2935705191 > Waiting for the upstream PR to be merged: [luarocks/luarocks#1798](https://github.com/luarocks/luarocks/pull/1798) Hi, once the upstream PR is merged and your PR is merged

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
mikyll commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934417056 @moonming @lxz999 Not sure if that's related, but I've noticed something curious: when setting the API key explicitly as a **string**, it works fine. But if the API-key is a *

[PR] fix: update commits to 39 to align with pictures and ZH-CN blog [apisix-website]

2025-06-03 Thread via GitHub
yzeng25 opened a new pull request, #1920: URL: https://github.com/apache/apisix-website/pull/1920 The current text '39' is not matching up with the pictures in EN and ZH-CN blog, or the texts in ZH-CN blog. Update to '38' to match up Fixes: #[Add issue number here] Chang

(apisix-website) 01/01: fix: update commits to 39 to align with pictures and ZH-CN blog

2025-06-03 Thread yilinzeng
This is an automated email from the ASF dual-hosted git repository. yilinzeng pushed a commit to branch yzeng25-patch-3 in repository https://gitbox.apache.org/repos/asf/apisix-website.git commit dbaedbfbcaa9b6b60c789b4fc179a22a6ba3e0ba Author: Yilin Zeng <36651058+yzen...@users.noreply.github.co

(apisix-website) branch yzeng25-patch-3 created (now dbaedbfbcaa)

2025-06-03 Thread yilinzeng
This is an automated email from the ASF dual-hosted git repository. yilinzeng pushed a change to branch yzeng25-patch-3 in repository https://gitbox.apache.org/repos/asf/apisix-website.git at dbaedbfbcaa fix: update commits to 39 to align with pictures and ZH-CN blog This branch includes

Re: [PR] feat: add `authorization` attribute for `loki-logger` (#11420) [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on PR #12243: URL: https://github.com/apache/apisix/pull/12243#issuecomment-2934683055 It looks much more reliable and once the tests pass I can approve the change. Then I'll call other reviewers. -- This is an automated message from the Apache Git Service. To respond

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on PR #12276: URL: https://github.com/apache/apisix/pull/12276#issuecomment-2934757435 @SkyeYoung > Perhaps you could try reusing the Dockerfile used to build the image in apisix-dashboard, and then copy out the final build artifacts. I would try to compile t

Re: [PR] feat: adds support for nginx variables in service_name param [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on code in PR #12187: URL: https://github.com/apache/apisix/pull/12187#discussion_r2123492465 ## apisix/upstream.lua: ## @@ -297,7 +297,12 @@ function _M.set_by_route(route, api_ctx) return 503, err end -local new_nodes, err = di

Re: [PR] feat: add support for multiple nacos instances and replace events library with shdict [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on code in PR #12263: URL: https://github.com/apache/apisix/pull/12263#discussion_r2123476536 ## apisix/discovery/nacos/init.lua: ## @@ -15,410 +15,196 @@ -- limitations under the License. -- -local require= require -local local_conf = r

Re: [PR] feat: add `authorization` attribute for `loki-logger` (#11420) [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on code in PR #12243: URL: https://github.com/apache/apisix/pull/12243#discussion_r2123446191 ## apisix/plugins/loki-logger.lua: ## @@ -138,11 +147,15 @@ end local function send_http_data(conf, log) +local headers = {} +if conf.headers ~= nil then

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
mikyll commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934554967 I understand that, but from the screenshot you provided, it shows you're sending the request to default HTTP port (80) 👀 Or at least that's what it seems, since I guess "com"

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
lxz999 commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934578194 > I understand that, but from the screenshot you provided, it shows you're sending the `curl` request to default HTTP port (80) 👀 > > Or at least that's what it seems, since I

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
lxz999 commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934539450 > Seems like you're trying to access the Admin API on the same port of the APISIX service (80) 🙂 > > Is the admin API exposed on the default port (9180)? Try to send the reque

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
lxz999 commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934472868 > [@moonming](https://github.com/moonming) [@lxz999](https://github.com/lxz999) > > Not sure if that's related, but I've noticed something curious: when setting the AP

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
mikyll commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2934505197 Is the admin API exposed on the default port (9180)? Try to send the request to: ``` http://:9180/apisix/admin/upstreams ``` -- This is an automated message from the A

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
SkyeYoung commented on PR #12276: URL: https://github.com/apache/apisix/pull/12276#issuecomment-2934264590 > We need to modify the APISIX build process to complete the front-end compilation in advance. Perhaps you could try reusing the Dockerfile used to build the image in `apisix-da

Re: [PR] docs: improve `plugin-develop` docs [apisix]

2025-06-03 Thread via GitHub
slow-groovin commented on code in PR #12242: URL: https://github.com/apache/apisix/pull/12242#discussion_r2123237839 ## docs/en/latest/plugin-develop.md: ## @@ -205,18 +129,25 @@ local schema = { The schema defines a non-negative number `i`, a string `s`, a non-empty array o

Re: [I] feat: test dashboard project [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on issue #12234: URL: https://github.com/apache/apisix/issues/12234#issuecomment-2934220585 Remarks: We need to request the ASF Infra team to add Docker Hub credentials for the APISIX repository. This repository is now not authorized to push images. -- This is an

[PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.2 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] opened a new pull request, #3097: URL: https://github.com/apache/apisix-dashboard/pull/3097 Bumps [@tanstack/react-query](https://github.com/TanStack/query/tree/HEAD/packages/react-query) from 5.74.4 to 5.79.2. Release notes Sourced from https://github.com/TanStack

Re: [I] feat: test dashboard project [apisix]

2025-06-03 Thread via GitHub
SkyeYoung commented on issue #12234: URL: https://github.com/apache/apisix/issues/12234#issuecomment-2934178666 I think the better workflow should be: For the first one: Each APISIX commit merged into master -> build and push the dev image -> dashboard periodically (currently 10 a.m.

Re: [PR] feat: add `authorization` attribute for `loki-logger` (#11420) [apisix]

2025-06-03 Thread via GitHub
slow-groovin commented on code in PR #12243: URL: https://github.com/apache/apisix/pull/12243#discussion_r2123083772 ## docs/en/latest/plugins/loki-logger.md: ## @@ -45,6 +45,7 @@ When enabled, the Plugin will serialize the request context information to [JSON | endpoint_addrs

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
moonming commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2933848193 @lxz999 did you restart apisix after modifying the admin-key? 403 is a client error, not the server side. -- This is an automated message from the Apache Git Service. To respond

(apisix-dashboard) branch dependabot/npm_and_yarn/tanstack/react-query-5.79.2 created (now 4dcad0ea2)

2025-06-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/tanstack/react-query-5.79.2 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git at 4dcad0ea2 chore(deps): bump @tanstack/react-query from 5.7

Re: [I] Apisix-Dashboard Helm Chart Should Not Require Secrets in Values for Users [apisix-helm-chart]

2025-06-03 Thread via GitHub
reza-hw commented on issue #815: URL: https://github.com/apache/apisix-helm-chart/issues/815#issuecomment-2933892004 +1 Agreed. This is a must-have feature in the helm charts. We can't be expected to just put the sensetive information in the git repo. -- This is an automated message f

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on PR #12276: URL: https://github.com/apache/apisix/pull/12276#issuecomment-2933930616 ### About the documentation This integration process is not yet complete. We need to modify the APISIX build process to complete the front-end compilation in advance. Therefore, the

Re: [PR] feat: add embedded apisix dashboard ui [apisix]

2025-06-03 Thread via GitHub
bzp2010 commented on code in PR #12276: URL: https://github.com/apache/apisix/pull/12276#discussion_r2122984348 ## apisix/cli/ngx_tpl.lua: ## @@ -646,6 +646,21 @@ http { apisix.http_admin() } } + +{% if enable_admin_ui then %} +

Re: [I] help request: After upgrading from version 3.2 to 3.12, calling the admin API returns a 403 error [apisix]

2025-06-03 Thread via GitHub
lxz999 commented on issue #12279: URL: https://github.com/apache/apisix/issues/12279#issuecomment-2933899770 > [@lxz999](https://github.com/lxz999) did you restart apisix after modifying the admin-key? 403 is a client error, not the server side. After restarting, changing the admin-ke

Re: [PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.0 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] closed pull request #3095: chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.0 URL: https://github.com/apache/apisix-dashboard/pull/3095 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

(apisix-dashboard) branch dependabot/npm_and_yarn/tanstack/react-query-5.79.0 deleted (was 28c3df269)

2025-06-03 Thread github-bot
This is an automated email from the ASF dual-hosted git repository. github-bot pushed a change to branch dependabot/npm_and_yarn/tanstack/react-query-5.79.0 in repository https://gitbox.apache.org/repos/asf/apisix-dashboard.git was 28c3df269 chore(deps): bump @tanstack/react-query from 5.7

Re: [PR] chore(deps): bump @tanstack/react-query from 5.74.4 to 5.79.0 [apisix-dashboard]

2025-06-03 Thread via GitHub
dependabot[bot] commented on PR #3095: URL: https://github.com/apache/apisix-dashboard/pull/3095#issuecomment-2933791607 Superseded by #3097. -- 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