[PR] chore(deps-dev): bump axios from 1.11.0 to 1.12.0 in /t [apisix]

2025-09-13 Thread via GitHub
dependabot[bot] opened a new pull request, #12613: URL: https://github.com/apache/apisix/pull/12613 Bumps [axios](https://github.com/axios/axios) from 1.11.0 to 1.12.0. Release notes Sourced from https://github.com/axios/axios/releases";>axios's releases. Release v1.12.0 R

[PR] chore(deps): bump axios from 1.11.0 to 1.12.0 [apisix-dashboard]

2025-09-13 Thread via GitHub
dependabot[bot] opened a new pull request, #3202: URL: https://github.com/apache/apisix-dashboard/pull/3202 Bumps [axios](https://github.com/axios/axios) from 1.11.0 to 1.12.0. Release notes Sourced from https://github.com/axios/axios/releases";>axios's releases. Release v1.1

Re: [PR] feat: Add proxy-chain plugin to chain multiple upstream service calls [apisix]

2025-09-13 Thread via GitHub
vahidaghazadeh commented on code in PR #12128: URL: https://github.com/apache/apisix/pull/12128#discussion_r2346318180 ## apisix/plugins/proxy-chain.lua: ## @@ -0,0 +1,479 @@ +#!/usr/bin/env perl +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contrib

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-13 Thread via GitHub
bzp2010 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2336329176 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- du

Re: [PR] feat(nacos): add metadata filtering support to nacos discovery [apisix]

2025-09-13 Thread via GitHub
Baoyuantop commented on code in PR #12445: URL: https://github.com/apache/apisix/pull/12445#discussion_r2331934894 ## docs/en/latest/discovery/nacos.md: ## @@ -278,3 +279,52 @@ The formatted response as below: } } ``` + + Metadata filtering + +APISIX supports filtering

Re: [PR] fix: use upstream id instead of inline upstream [apisix-ingress-controller]

2025-09-13 Thread via GitHub
Copilot commented on code in PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546#discussion_r2331872901 ## internal/adc/translator/httproute.go: ## @@ -522,16 +556,16 @@ func (t *Translator) TranslateHTTPRoute(tctx *provider.TranslateContext, httpRou

Re: [PR] docs: Improve `chaitin-waf` plugin docs and remove unintended highlights [apisix]

2025-09-13 Thread via GitHub
moonming merged PR #12608: URL: https://github.com/apache/apisix/pull/12608 -- 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...@ap

[I] help request: Failed to parse domain [apisix]

2025-09-12 Thread via GitHub
Kozatoo opened a new issue, #10505: URL: https://github.com/apache/apisix/issues/10505 ### Description Context: I'm trying to build an application with an Auth Service which should only validate the tokens, and admin service which sould be protected. I am running APIsix from d

Re: [I] help request: Plugins do not work propely [apisix-ingress-controller]

2025-09-12 Thread via GitHub
bzp2010 commented on issue #2539: URL: https://github.com/apache/apisix-ingress-controller/issues/2539#issuecomment-3287524771 @alvarobem Has the Ingress Controller reported any synchronization errors? --- If not, you need to inspect the plugin configuration stored in e

Re: [PR] feat: add support for named servicePort in ApisixRoute backend [apisix-ingress-controller]

2025-09-12 Thread via GitHub
bzp2010 commented on code in PR #2553: URL: https://github.com/apache/apisix-ingress-controller/pull/2553#discussion_r2345868342 ## internal/adc/translator/apisixroute.go: ## @@ -364,14 +389,26 @@ func (t *Translator) translateApisixRouteStreamBackendResolveGranularity(tctx *p

Re: [PR] feat: add support for named servicePort in ApisixRoute backend [apisix-ingress-controller]

2025-09-12 Thread via GitHub
bzp2010 commented on code in PR #2553: URL: https://github.com/apache/apisix-ingress-controller/pull/2553#discussion_r2345870942 ## internal/adc/translator/apisixroute.go: ## @@ -328,6 +329,26 @@ func (t *Translator) buildService(ar *apiv2.ApisixRoute, rule apiv2.ApisixRouteH

Re: [PR] docs: update mqtt-proxy doc to enable stream mode [apisix]

2025-09-12 Thread via GitHub
github-actions[bot] commented on PR #12139: URL: https://github.com/apache/apisix/pull/12139#issuecomment-3284651852 This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. --

Re: [I] help request: Failed to parse domain [apisix]

2025-09-12 Thread via GitHub
perry2of5 commented on issue #10505: URL: https://github.com/apache/apisix/issues/10505#issuecomment-3286720883 I'm having this issue using the standard docker container. I finally just put in a script to run before apisix starts to expand the variables. ```bash #!/bin/bash

[I] Consumer labels do not show up when [apisix-dashboard]

2025-09-12 Thread via GitHub
mdnfiras opened a new issue, #3201: URL: https://github.com/apache/apisix-dashboard/issues/3201 ### Issue description After adding labels to a consumer, we can see them displayed on the consumer page. But as soon as I refresh the consumer page the labels do not show anymore. ##

Re: [I] Consumer labels do not show up in dashboard [apisix-dashboard]

2025-09-12 Thread via GitHub
mdnfiras commented on issue #3201: URL: https://github.com/apache/apisix-dashboard/issues/3201#issuecomment-3285274734 same for credentials by the way: https://github.com/user-attachments/assets/cf5a4ad4-06d5-4e9d-b715-737cf8d0cc5f"; /> -- This is an automated message from the Apa

Re: [I] request help: why the location in response header that returned by APISIX has the 9443 port? [apisix]

2025-09-12 Thread via GitHub
Eifoen commented on issue #6954: URL: https://github.com/apache/apisix/issues/6954#issuecomment-3285162447 +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

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-12 Thread via GitHub
bzp2010 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2343921386 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- du

Re: [PR] change(openid-connect): when `bearer_only` is `false`, require the user to fill in `session.secret` [apisix]

2025-09-12 Thread via GitHub
SkyeYoung merged PR #12609: URL: https://github.com/apache/apisix/pull/12609 -- 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...@a

[PR] docs: update notes for API-drive standalone mode [apisix]

2025-09-12 Thread via GitHub
kayx23 opened a new pull request, #12612: URL: https://github.com/apache/apisix/pull/12612 ### Description Update notes for API-drive standalone mode. Remove the experimental label for the next release (3.14.0). ### Checklist - [x] I have explained the need for this PR a

Re: [PR] feat: add adc debug server [apisix-ingress-controller]

2025-09-12 Thread via GitHub
Revolyssup commented on PR #2550: URL: https://github.com/apache/apisix-ingress-controller/pull/2550#issuecomment-3284537023 > > I don't think it should be a standalone server, rather than a part of the ingress controller API. > > **Suggestion: Move the debug API into the unified Ing

Re: [I] bug: dynamic upstream, one Eureka node is unavailable, half of the requests are lost after reloading [apisix]

2025-09-12 Thread via GitHub
Baoyuantop commented on issue #12610: URL: https://github.com/apache/apisix/issues/12610#issuecomment-3284436792 Hi @liquanzhou, thanks for your report. We have verified that this issue does exist and will schedule a fix. -- This is an automated message from the Apache Git Service. To res

Re: [PR] fix(grpc-transcode): correctly serialize empty repeated fields to [] [apisix]

2025-09-12 Thread via GitHub
bytelazy commented on PR #12391: URL: https://github.com/apache/apisix/pull/12391#issuecomment-3284352731 > Hi @bytelazy, could you please merge the latest master branch code? sure, I have merged the latest master branch -- This is an automated message from the Apache Git Service. T

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-12 Thread via GitHub
SkyeYoung commented on PR #12611: URL: https://github.com/apache/apisix/pull/12611#issuecomment-3284185391 > change @bzp2010 This is not very common. I usually refer to https://github.com/TrigenSoftware/simple-release/blob/main/GUIDE.md Since this feature did not exist before,

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-12 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2343252988 ## apisix/plugins/jwt-auth.lua: ## @@ -144,8 +142,9 @@ function _M.check_schema(conf, schema_type) return false, err end -if conf.algorithm ~= "R

Re: [PR] fix: use upstream id instead of inline upstream in traffic-split plugin [apisix-ingress-controller]

2025-09-11 Thread via GitHub
AlinsRan commented on code in PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546#discussion_r2332332521 ## internal/adc/translator/httproute.go: ## @@ -508,11 +531,22 @@ func (t *Translator) TranslateHTTPRoute(tctx *provider.TranslateContext, httpRou

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
bzp2010 commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2343058255 ## t/plugin/jwt-auth.t: ## @@ -1222,3 +1222,71 @@ hello world --- error_code: 400 --- response_body {"error_msg":"invalid plugins configuration: failed to check the

Re: [PR] chore(deps): bump apache/skywalking-eyes from 9f0a5c0571ed1a0c13a16808cd8f59bc22f03883 to 07a607ff5b0759f5ed47306c865aac50fe9b3985 [apisix-dashboard]

2025-09-11 Thread via GitHub
dependabot[bot] commented on PR #3148: URL: https://github.com/apache/apisix-dashboard/pull/3148#issuecomment-3283827944 Superseded by #3200. -- 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

[PR] chore(deps): bump apache/skywalking-eyes from 9f0a5c0571ed1a0c13a16808cd8f59bc22f03883 to 3655e78854be6a56ae72a0c9a847aedd133c3b74 [apisix-dashboard]

2025-09-11 Thread via GitHub
dependabot[bot] opened a new pull request, #3200: URL: https://github.com/apache/apisix-dashboard/pull/3200 Bumps [apache/skywalking-eyes](https://github.com/apache/skywalking-eyes) from 9f0a5c0571ed1a0c13a16808cd8f59bc22f03883 to 3655e78854be6a56ae72a0c9a847aedd133c3b74. Changelog

Re: [PR] chore(deps): bump apache/skywalking-eyes from 9f0a5c0571ed1a0c13a16808cd8f59bc22f03883 to 07a607ff5b0759f5ed47306c865aac50fe9b3985 [apisix-dashboard]

2025-09-11 Thread via GitHub
dependabot[bot] closed pull request #3148: chore(deps): bump apache/skywalking-eyes from 9f0a5c0571ed1a0c13a16808cd8f59bc22f03883 to 07a607ff5b0759f5ed47306c865aac50fe9b3985 URL: https://github.com/apache/apisix-dashboard/pull/3148 -- This is an automated message from the Apache Git Service.

Re: [PR] chore: add skip_mtls_uril_regex test for ApisixTLS [apisix-ingress-controller]

2025-09-11 Thread via GitHub
Revolyssup merged PR #2555: URL: https://github.com/apache/apisix-ingress-controller/pull/2555 -- 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: notificat

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342866858 ## apisix/plugins/jwt-auth.lua: ## @@ -144,8 +142,9 @@ function _M.check_schema(conf, schema_type) return false, err end -if conf.algorithm ~= "R

Re: [PR] feat: add adc debug server [apisix-ingress-controller]

2025-09-11 Thread via GitHub
AlinsRan commented on PR #2550: URL: https://github.com/apache/apisix-ingress-controller/pull/2550#issuecomment-3283422930 > I don't think it should be a standalone server, rather than a part of the ingress controller API. **Suggestion: Move the debug API into the unified Ingress Ser

Re: [PR] fix: use upstream id instead of inline upstream in traffic-split plugin [apisix-ingress-controller]

2025-09-11 Thread via GitHub
ronething commented on code in PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546#discussion_r2332386878 ## internal/adc/translator/httproute.go: ## @@ -508,11 +531,22 @@ func (t *Translator) TranslateHTTPRoute(tctx *provider.TranslateContext, httpRou

Re: [I] bug: parse_domain_for_nodes issue [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on issue #12512: URL: https://github.com/apache/apisix/issues/12512#issuecomment-3272946043 @Baoyuantop pls have a look -- 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 spe

Re: [I] request help: Could not find keepalive_pool in UpstreamStruct [apisix-ingress-controller]

2025-09-11 Thread via GitHub
Baoyuantop commented on issue #2545: URL: https://github.com/apache/apisix-ingress-controller/issues/2545#issuecomment-3265611493 Duplicate of #1883 -- 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

Re: [PR] fix: release checker when nodes are nil [apisix]

2025-09-11 Thread via GitHub
nic-6443 commented on PR #12149: URL: https://github.com/apache/apisix/pull/12149#issuecomment-3272927395 @Baoyuantop we can close it now, we've completely refactored the health checker module. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-11 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335818425 ## docs/en/latest/reference/api-reference.md: ## @@ -1194,7 +1194,7 @@ ApisixRouteStream defines the configuration for a Layer 4 (TCP/UDP) route. C

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342861298 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342861298 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342861298 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342844480 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
nic-6443 commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342843574 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in `secret` [apisix]

2025-09-11 Thread via GitHub
nic-6443 commented on code in PR #12611: URL: https://github.com/apache/apisix/pull/12611#discussion_r2342843574 ## apisix/plugins/jwt-auth.lua: ## @@ -145,7 +143,8 @@ function _M.check_schema(conf, schema_type) end if conf.algorithm ~= "RS256" and conf.algorithm ~=

Re: [PR] fix: typos in k8s discovery code [apisix]

2025-09-11 Thread via GitHub
ChuanFF commented on PR #12288: URL: https://github.com/apache/apisix/pull/12288#issuecomment-3283454063 > Hi @ChuanFF, before continuing to complete the code, could you please describe your current modification plan in the description? This will help other maintainers review this PR.

Re: [PR] feat(openid-connect): when `bearer_only` is `false`, require the user to fill in `session.secret` [apisix]

2025-09-11 Thread via GitHub
SkyeYoung commented on code in PR #12609: URL: https://github.com/apache/apisix/pull/12609#discussion_r2342768314 ## t/plugin/security-warning2.t: ## @@ -64,6 +64,7 @@ Using opa host with no TLS is a security risk === TEST 3: openid-connect with no TLS +--- ONLY Review Com

[PR] feat(jwt-auth): when algorithm is not `RS256` or `ES256`, require the user to fill in secret [apisix]

2025-09-11 Thread via GitHub
SkyeYoung opened a new pull request, #12611: URL: https://github.com/apache/apisix/pull/12611 ### Description Which issue(s) this PR fixes: Fixes # ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have

[PR] chore: add skip_mtls_uril_regex test for ApisixTLS [apisix-ingress-controller]

2025-09-11 Thread via GitHub
Revolyssup opened a new pull request, #2555: URL: https://github.com/apache/apisix-ingress-controller/pull/2555 Part of test migaration from v1.8 - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches - [ ] Documentation - [ ] Refactor -

[PR] chore: migrate e2e test for secretRef in ApisixRoute.plugins [apisix-ingress-controller]

2025-09-11 Thread via GitHub
Revolyssup opened a new pull request, #2556: URL: https://github.com/apache/apisix-ingress-controller/pull/2556 Part of test migaration from v1.8 - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches - [ ] Documentation - [ ] Refactor -

[I] test: abstract gateway proxy configuration acquisition [apisix-ingress-controller]

2025-09-11 Thread via GitHub
ronething opened a new issue, #2554: URL: https://github.com/apache/apisix-ingress-controller/issues/2554 Currently, the generation of gateway proxy configurations is distributed across different files, but there is quite a lot of repetition. We can consider using a unified method to obtain

[PR] docs: Improve `chaitin-waf` plugin docs and remove unintended highlights [apisix]

2025-09-11 Thread via GitHub
kayx23 opened a new pull request, #12608: URL: https://github.com/apache/apisix/pull/12608 ### Description This PR improve `chaitin-waf` plugin docs and remove unintended highlights (that are not rendered) introduced in previous plugin doc improvement PRs. Which issue(s) t

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-11 Thread via GitHub
nic-6443 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2339439485 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- d

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

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

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

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

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

2025-09-10 Thread via GitHub
dependabot[bot] commented on PR #3197: URL: https://github.com/apache/apisix-dashboard/pull/3197#issuecomment-3278241678 Superseded by #3199. -- 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: [PR] chore(deps-dev): bump @tanstack/react-query-devtools from 5.74.4 to 5.87.3 [apisix-dashboard]

2025-09-10 Thread via GitHub
dependabot[bot] closed pull request #3197: chore(deps-dev): bump @tanstack/react-query-devtools from 5.74.4 to 5.87.3 URL: https://github.com/apache/apisix-dashboard/pull/3197 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

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

2025-09-10 Thread via GitHub
dependabot[bot] closed pull request #3195: chore(deps): bump @tanstack/react-query from 5.74.4 to 5.87.1 URL: https://github.com/apache/apisix-dashboard/pull/3195 -- 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

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

2025-09-10 Thread via GitHub
dependabot[bot] commented on PR #3195: URL: https://github.com/apache/apisix-dashboard/pull/3195#issuecomment-3278214267 Superseded by #3198. -- 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

[PR] fix: healthcheck manager missing runtime information [apisix]

2025-09-10 Thread via GitHub
Revolyssup opened a new pull request, #12607: URL: https://github.com/apache/apisix/pull/12607 - In case of service discovery, the node_ver information is not attached to the service received from etcd (due to deepcopy) therefore it's lost and healthchecker is not recreated. - In case of

Re: [PR] fix(docs/blog): inline image width syntax [apisix-website]

2025-09-10 Thread via GitHub
moonming merged PR #1953: URL: https://github.com/apache/apisix-website/pull/1953 -- 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

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
bzp2010 commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2336139957 ## api/adc/types.go: ## @@ -195,15 +195,14 @@ type Timeout struct { // +k8s:deepcopy-gen=true type StreamRoute struct { - Description strin

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
AlinsRan merged PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551 -- 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: notificatio

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-10 Thread via GitHub
bzp2010 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2336329176 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- du

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-10 Thread via GitHub
bzp2010 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2336287389 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- du

Re: [I] help request: Is there any way to bypass mTLS using the domain:port method? [apisix]

2025-09-10 Thread via GitHub
Baoyuantop commented on issue #12606: URL: https://github.com/apache/apisix/issues/12606#issuecomment-3274142575 ‌‌‌Hi @jojo578578, can you provide more detailed requirements information? Including your request chain. -- This is an automated message from the Apache Git Service. To res

Re: [PR] fix(limit-req): Make Redis path atomic via EVAL + use hash key with TTL [apisix]

2025-09-10 Thread via GitHub
Baoyuantop commented on PR #12605: URL: https://github.com/apache/apisix/pull/12605#issuecomment-3274152214 ‌Hi @falvaradorodriguez, we need to add the test case for this fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
bzp2010 commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2336139957 ## api/adc/types.go: ## @@ -195,15 +195,14 @@ type Timeout struct { // +k8s:deepcopy-gen=true type StreamRoute struct { - Description strin

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335818425 ## docs/en/latest/reference/api-reference.md: ## @@ -1194,7 +1194,7 @@ ApisixRouteStream defines the configuration for a Layer 4 (TCP/UDP) route. C

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335856106 ## api/adc/types.go: ## @@ -195,15 +195,14 @@ type Timeout struct { // +k8s:deepcopy-gen=true type StreamRoute struct { - Description stri

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335856106 ## api/adc/types.go: ## @@ -195,15 +195,14 @@ type Timeout struct { // +k8s:deepcopy-gen=true type StreamRoute struct { - Description stri

[I] help request: Is there any way to bypass mTLS using the domain:port method? [apisix]

2025-09-10 Thread via GitHub
jojo578578 opened a new issue, #12606: URL: https://github.com/apache/apisix/issues/12606 ### Description I achieved this by forwarding different pages using the same domain name but different ports. Now I have a requirement that some ports require two-way authentication, and some po

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-10 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335812649 ## docs/en/latest/reference/api-reference.md: ## @@ -1194,7 +1194,7 @@ ApisixRouteStream defines the configuration for a Layer 4 (TCP/UDP) route. C

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
bzp2010 commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335633930 ## api/adc/types.go: ## @@ -195,15 +195,14 @@ type Timeout struct { // +k8s:deepcopy-gen=true type StreamRoute struct { - Description strin

Re: [PR] docs: specify namespace in metadata explicitly [apisix-ingress-controller]

2025-09-09 Thread via GitHub
kayx23 merged PR #2549: URL: https://github.com/apache/apisix-ingress-controller/pull/2549 -- 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

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
ronething commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335287543 ## test/e2e/crds/v2/streamroute.go: ## @@ -0,0 +1,222 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor lic

[PR] docs: add configuration reference doc and update config.yaml inline comments [apisix-ingress-controller]

2025-09-09 Thread via GitHub
kayx23 opened a new pull request, #2552: URL: https://github.com/apache/apisix-ingress-controller/pull/2552 ### Type of change: - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches - [x] Documentation - [ ] Refa

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335461109 ## api/v2/shared_types.go: ## @@ -45,6 +45,11 @@ const ( DefaultWeight = 100 ) +const ( + ResolveGranularityService = "service" +

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335461109 ## api/v2/shared_types.go: ## @@ -45,6 +45,11 @@ const ( DefaultWeight = 100 ) +const ( + ResolveGranularityService = "service" +

[PR] fix inline image width format [apisix-website]

2025-09-09 Thread via GitHub
Yilialinn opened a new pull request, #1953: URL: https://github.com/apache/apisix-website/pull/1953 Fix the unsupported style image format ![img_v3_02q0_e046a3b4-cdbd-4553-ac98-69225613061g](https://github.com/user-attachments/assets/cc53cdc8-3672-4002-9a12-0ef31366aaa2) -- Thi

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
AlinsRan commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335314448 ## test/e2e/crds/v2/streamroute.go: ## @@ -0,0 +1,222 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor lice

Re: [PR] fix: release checker when nodes are nil [apisix]

2025-09-09 Thread via GitHub
Baoyuantop closed pull request #12149: fix: release checker when nodes are nil URL: https://github.com/apache/apisix/pull/12149 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-09 Thread via GitHub
nic-6443 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2335290695 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- d

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-09 Thread via GitHub
nic-6443 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2335290695 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- d

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
ronething commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335271919 ## api/v2/shared_types.go: ## @@ -45,6 +45,11 @@ const ( DefaultWeight = 100 ) +const ( + ResolveGranularityService = "service" +

Re: [PR] feat: add adc debug server [apisix-ingress-controller]

2025-09-09 Thread via GitHub
AlinsRan commented on PR #2550: URL: https://github.com/apache/apisix-ingress-controller/pull/2550#issuecomment-3272853508 I don't think it should be a standalone server, rather than a part of the ingress API. -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-09 Thread via GitHub
AlinsRan commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2335251386 ## apisix/admin/standalone.lua: ## @@ -420,6 +430,29 @@ function _M.init_worker() end events:register(update_config, EVENT_UPDATE, EVENT_UPDATE) +-- d

Re: [PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
Copilot commented on code in PR #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551#discussion_r2335197670 ## test/e2e/scaffold/scaffold.go: ## @@ -63,23 +63,41 @@ type Scaffold struct { dataplaneService *corev1.Service httpbinService *co

[PR] feat: support stream_route for ApisixRoute [apisix-ingress-controller]

2025-09-09 Thread via GitHub
AlinsRan opened a new pull request, #2551: URL: https://github.com/apache/apisix-ingress-controller/pull/2551 ### Type of change: - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches - [ ] Documentation - [ ] Re

Re: [PR] fix: use upstream id instead of inline upstream in traffic-split plugin [apisix-ingress-controller]

2025-09-09 Thread via GitHub
ronething commented on PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546#issuecomment-3271280815 > Are our tests based on the final results? Do they not need to be modified to verify the changes in the pull request? The current test should be sufficient. If

[PR] fix: typo in ai-proxy-multi [apisix]

2025-09-09 Thread via GitHub
Yilialinn opened a new pull request, #12601: URL: https://github.com/apache/apisix/pull/12601 ### Fixes the deployment failure ![img_v3_02pv_bb5f952f-7cff-4a08-9159-397d334b770g](https://github.com/user-attachments/assets/07fc1daf-2e20-49b9-92f9-e23364c377be) -- This is an automated m

Re: [PR] fix(standalone): support stream route in admin api mode [apisix]

2025-09-09 Thread via GitHub
nic-6443 commented on code in PR #12604: URL: https://github.com/apache/apisix/pull/12604#discussion_r2333254818 ## t/admin/standalone.t: ## @@ -91,7 +94,7 @@ X-Digest: t2 === TEST 3: get config --- config -location /t { +location /config { Review Comment: I modi

[PR] feat: add adc debug server [apisix-ingress-controller]

2025-09-09 Thread via GitHub
Revolyssup opened a new pull request, #2550: URL: https://github.com/apache/apisix-ingress-controller/pull/2550 https://github.com/user-attachments/assets/775e9427-4d30-494e-bf25-26c78b8e68c0 - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ]

Re: [PR] feat: add adc debug server [apisix-ingress-controller]

2025-09-09 Thread via GitHub
Revolyssup commented on code in PR #2550: URL: https://github.com/apache/apisix-ingress-controller/pull/2550#discussion_r2334439623 ## internal/provider/apisix/provider.go: ## @@ -80,12 +81,14 @@ func New(updater status.Updater, readier readiness.ReadinessManager, opts ...pro

Re: [PR] fix: use upstream id instead of inline upstream in traffic-split plugin [apisix-ingress-controller]

2025-09-09 Thread via GitHub
ronething merged PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546 -- 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: notificati

Re: [I] bug: limit-req plugin does not work correctly when configuring redis or redis-cluster [apisix]

2025-09-09 Thread via GitHub
luarx commented on issue #12592: URL: https://github.com/apache/apisix/issues/12592#issuecomment-327259 This is the result when using `policy: local`, so expected result 🟢 Using the same configuration as in the `policy: redis` test ([here](https://github.com/apache/apisix/issues/

Re: [I] help request: Plugins do not work propely [apisix-ingress-controller]

2025-09-09 Thread via GitHub
alvarobem commented on issue #2539: URL: https://github.com/apache/apisix-ingress-controller/issues/2539#issuecomment-3270882003 Hello @bzp2010. I am using etcd deployment mode and the ingress controller version is 2.0.0-rc3 -- This is an automated message from the Ap

Re: [I] 503 Service Temporarily Unavailable [apisix]

2025-09-09 Thread via GitHub
medmo commented on issue #4054: URL: https://github.com/apache/apisix/issues/4054#issuecomment-3270222900 503 Service Temporarily Unavailable may be caused by proxies. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use th

Re: [PR] feat: add support for CORS httproutefilter [apisix-ingress-controller]

2025-09-09 Thread via GitHub
Revolyssup merged PR #2548: URL: https://github.com/apache/apisix-ingress-controller/pull/2548 -- 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: notificat

[PR] docs: specify namespace in metadata explicitly [apisix-ingress-controller]

2025-09-09 Thread via GitHub
kayx23 opened a new pull request, #2549: URL: https://github.com/apache/apisix-ingress-controller/pull/2549 ### Type of change: - [ ] Bugfix - [ ] New feature provided - [ ] Improve performance - [ ] Backport patches - [x] Documentation - [ ] Refa

Re: [I] bug: parse_domain_for_nodes issue [apisix]

2025-09-09 Thread via GitHub
Mairu commented on issue #12512: URL: https://github.com/apache/apisix/issues/12512#issuecomment-3269882790 @SkyeYoung As you said you were not able to reproduce the issue I created a small reproduction repository: https://github.com/Mairu/apisix-issue-12512-repro -- This is an automated

Re: [PR] fix: use upstream id instead of inline upstream in traffic-split plugin [apisix-ingress-controller]

2025-09-09 Thread via GitHub
bzp2010 commented on code in PR #2546: URL: https://github.com/apache/apisix-ingress-controller/pull/2546#discussion_r2332799315 ## internal/adc/translator/httproute.go: ## @@ -508,11 +531,22 @@ func (t *Translator) TranslateHTTPRoute(tctx *provider.TranslateContext, httpRou

  1   2   3   4   5   6   7   8   9   10   >