Re: [I] help request: is there a way for authz-keycloak plugin to redirect when an invalid bearer token is provided? [apisix]

2024-09-18 Thread via GitHub
Xb990219 commented on issue #11588: URL: https://github.com/apache/apisix/issues/11588#issuecomment-2360107930 > > Is there a way to redirect to another uri when the bearer token is invalid, instead of throwing a 401 unauthorized error? > > If the bearer token is invalid, the keycloak

Re: [I] help request: is there a way for authz-keycloak plugin to redirect when an invalid bearer token is provided? [apisix]

2024-09-18 Thread via GitHub
Xb990219 closed issue #11588: help request: is there a way for authz-keycloak plugin to redirect when an invalid bearer token is provided? URL: https://github.com/apache/apisix/issues/11588 -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] docs: add Chinese language support to wasm documentation [apisix]

2024-09-18 Thread via GitHub
yzeng25 commented on code in PR #11572: URL: https://github.com/apache/apisix/pull/11572#discussion_r1766101123 ## docs/zh/latest/wasm.md: ## @@ -0,0 +1,84 @@ +# Wasm + +APISIX 支持使用 [Proxy Wasm SDK](https://github.com/proxy-wasm/spec#sdks) 编写的 Wasm 插件。 + +目前,仅实现了少数 API。请关注 [wa

Re: [I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
zhoujiexiong commented on issue #11587: URL: https://github.com/apache/apisix/issues/11587#issuecomment-2359858962 @bpasson `docker exec -it apisix-dev-env make deps` Did this step run successfully? And what is the output of `docker exec -it apisix-dev-env which openrest

Re: [I] help request: how to close keepalive [apisix]

2024-09-18 Thread via GitHub
zzl-code closed issue #11571: help request: how to close keepalive URL: https://github.com/apache/apisix/issues/11571 -- 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 unsubsc

Re: [I] help request: how to close keepalive [apisix]

2024-09-18 Thread via GitHub
zzl-code commented on issue #11571: URL: https://github.com/apache/apisix/issues/11571#issuecomment-2359783726 > > 我注意到代码中有一个 enable_keepalive;应如何配置它以使其值为 false? > > 读完代码后,我可以得出结论,这是无法配置的。但是,您可以修改代码并测试它是否解决了您的问题。然后您可以建议配置该`keepalive`字段。 > > [您可以使用https://github.com/shreemaan-ab

Re: [I] request help: missing user key in JWT token [apisix-ingress-controller]

2024-09-18 Thread via GitHub
github-actions[bot] commented on issue #2250: URL: https://github.com/apache/apisix-ingress-controller/issues/2250#issuecomment-2359731353 This issue has been marked as stale due to 90 days of inactivity. It will be closed in 30 days if no further activity occurs. If this issue is still re

Re: [I] feat: run and report conformance of Gateway API [apisix-ingress-controller]

2024-09-18 Thread via GitHub
github-actions[bot] commented on issue #2088: URL: https://github.com/apache/apisix-ingress-controller/issues/2088#issuecomment-2359731305 This issue has been marked as stale due to 90 days of inactivity. It will be closed in 30 days if no further activity occurs. If this issue is still re

Re: [I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
bpasson commented on issue #11587: URL: https://github.com/apache/apisix/issues/11587#issuecomment-2358837917 @zhoujiexiong I followed there [reference](https://apisix.apache.org/docs/apisix/build-apisix-dev-environment-on-mac/) and got stuk at `docker exec -it apisix-dev-env make run` whic

Re: [I] help request: How consumer work with forward-auth? [apisix]

2024-09-18 Thread via GitHub
zhoujiexiong commented on issue #11567: URL: https://github.com/apache/apisix/issues/11567#issuecomment-2358680968 > However, I'm unsure how to get the consumer_name as the forward-auth plugin doesn't seem to offer the necessary key configuration. @sky-dawn More details about this

[I] Online Playground cannot be accessed [apisix-dashboard]

2024-09-18 Thread via GitHub
andyyin opened a new issue, #2966: URL: https://github.com/apache/apisix-dashboard/issues/2966 ### Issue description https://apisix-dashboard.apiseven.com/ cannot be accessed ### Expected behavior - ### How to Reproduce - ### Screenshots _No re

Re: [I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
zhoujiexiong commented on issue #11587: URL: https://github.com/apache/apisix/issues/11587#issuecomment-2358651076 > Is there a dev-container I can use for a local build? @bpasson Try this [reference](https://apisix.apache.org/docs/apisix/build-apisix-dev-environment-on-mac/) first?

Re: [PR] feat: support gcp secret manager [apisix]

2024-09-18 Thread via GitHub
HuanXin-Chen commented on code in PR #11436: URL: https://github.com/apache/apisix/pull/11436#discussion_r1765101790 ## t/secret/gcp.t: ## @@ -0,0 +1,737 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTIC

Re: [I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
bpasson commented on issue #11587: URL: https://github.com/apache/apisix/issues/11587#issuecomment-2358354764 @zhoujiexiong I have very little knowledge about Lua language, but I will try I suppose this is not much more than swapping some lines. -- This is an automated message from the Ap

Re: [I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
zhoujiexiong commented on issue #11587: URL: https://github.com/apache/apisix/issues/11587#issuecomment-2358317117 @bpasson The [Header Priority](https://apisix.apache.org/docs/apisix/plugins/proxy-rewrite/#header-priority) mentioned in the document should refer to the priority of t

Re: [I] When will the docker image for APISIX 3.10 release? [apisix-docker]

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

Re: [PR] feat: support gcp secret manager [apisix]

2024-09-18 Thread via GitHub
membphis commented on code in PR #11436: URL: https://github.com/apache/apisix/pull/11436#discussion_r1757871158 ## t/secret/gcp.t: ## @@ -0,0 +1,737 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one or more +# contributor license agreements. See the NOTICE fi

Re: [PR] feat: rewrite hmac-auth plugin for usability [apisix]

2024-09-18 Thread via GitHub
membphis commented on code in PR #11581: URL: https://github.com/apache/apisix/pull/11581#discussion_r1764662315 ## apisix/plugins/hmac-auth.lua: ## @@ -354,86 +283,62 @@ local function validate(ctx, params) end -local function get_params(ctx) -local params = {} -lo

Re: [PR] feat: rewrite hmac-auth plugin for usability [apisix]

2024-09-18 Thread via GitHub
membphis commented on code in PR #11581: URL: https://github.com/apache/apisix/pull/11581#discussion_r1764650470 ## apisix/plugins/hmac-auth.lua: ## @@ -148,204 +118,163 @@ function _M.check_schema(conf, schema_type) end -local function get_consumer(access_key) -if not

Re: [I] bug: cors plugin does not allow setting ** if allow_credential is true [apisix]

2024-09-18 Thread via GitHub
rajan123456 commented on issue #11586: URL: https://github.com/apache/apisix/issues/11586#issuecomment-2357878300 Great! Thanks for pointing out! I was able to apply my config by additionally specifying ``` expose_headers: '**' ``` Do we need to update the docs? As the docs

[I] help request: is there a way for authz-keycloak plugin to redirect when an invalid bearer token is provided? [apisix]

2024-09-18 Thread via GitHub
Xb990219 opened a new issue, #11588: URL: https://github.com/apache/apisix/issues/11588 ### Description I have search for the current issues and cannot find a solution. I know there is an `access_denied_redirect_uri` attribute in authz-keycloak plugin to redirect the user when

Re: [PR] feat: content-moderation plugin [apisix]

2024-09-18 Thread via GitHub
shreemaan-abhishek commented on code in PR #11541: URL: https://github.com/apache/apisix/pull/11541#discussion_r1751581472 ## apisix/plugins/ai/openai.lua: ## @@ -0,0 +1,33 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor license agre

[I] bug: proxy-rewrite header priority wrong [apisix]

2024-09-18 Thread via GitHub
bpasson opened a new issue, #11587: URL: https://github.com/apache/apisix/issues/11587 ### Current Behavior The current priority for header operations is: `add > remove > set`. This makes it impossible to handle the use-case where you want to add multiple headers with the same name b

Re: [PR] feat: content-moderation plugin [apisix]

2024-09-18 Thread via GitHub
nic-6443 commented on code in PR #11541: URL: https://github.com/apache/apisix/pull/11541#discussion_r1764593201 ## apisix/plugins/content-moderation.lua: ## @@ -0,0 +1,155 @@ +-- +-- Licensed to the Apache Software Foundation (ASF) under one or more +-- contributor license agre

Re: [PR] docs: update config.yaml.example about upstream's keepalive. [apisix]

2024-09-18 Thread via GitHub
kayx23 commented on code in PR #11585: URL: https://github.com/apache/apisix/pull/11585#discussion_r1764574484 ## conf/config.yaml.example: ## @@ -238,12 +238,12 @@ nginx_config: # Config for render the template to generate n

[I] bug: cors plugin does not allow setting ** if allow_credential is true [apisix]

2024-09-18 Thread via GitHub
rajan123456 opened a new issue, #11586: URL: https://github.com/apache/apisix/issues/11586 ### Current Behavior I am trying to define an APISIX route using CRD. I am trying to enable cors plugin but facing an issue with specifying * ``` apiVersion: apisix.apache.org/v2 ki

(apisix) branch moonming-patch-2 updated (3490ebabf -> 8aeeace92)

2024-09-18 Thread wenming
This is an automated email from the ASF dual-hosted git repository. wenming pushed a change to branch moonming-patch-2 in repository https://gitbox.apache.org/repos/asf/apisix.git from 3490ebabf docs: update config.yaml.example about upstream's keepalive. add 8aeeace92 f No new revisio