[GitHub] [apisix] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-20 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1398607800

   Is it becuase my upstream is consul?
   
   
![image](https://user-images.githubusercontent.com/21145779/213747301-fcd0ddfa-dc1a-4d49-a859-fbb6a5b6603d.png)
   I am very sure that it is reachable.
   I can curl if my route is **/*** but not if **/food/***


-- 
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] hndoss commented on issue #1608: bug: Block request method with Apisix Ingress Controller annotation

2023-01-20 Thread GitBox


hndoss commented on issue #1608:
URL: 
https://github.com/apache/apisix-ingress-controller/issues/1608#issuecomment-1398539218

   > You mean the request should not be sent upstream, right?
   
   Yes.


-- 
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] MirtoBusico commented on pull request #1468: Add blog post "Accessing Apisix-dashboard from everywhere with Keycloak authentication"

2023-01-20 Thread GitBox


MirtoBusico commented on PR #1468:
URL: https://github.com/apache/apisix-website/pull/1468#issuecomment-1398183444

   Yes the images are at
   
   https://github.com/MirtoBusico/assets-for-blogs
   
   
   On 20/01/23 10:50, Yilia wrote:
   >
   > @MirtoBusico  Hello, the picture is 
   > missing. Could you send me another available link?
   >
   > —
   > Reply to this email directly, view it on GitHub 
   > 
, 
   > or unsubscribe 
   > 
.
   > You are receiving this because you were mentioned.Message ID: 
   > ***@***.***>
   >


-- 
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 #6217: request help: does apisix authentication plugin support multiple authentication methods at the same time

2023-01-20 Thread GitBox


github-actions[bot] commented on issue #6217:
URL: https://github.com/apache/apisix/issues/6217#issuecomment-1398165168

   This issue has been marked as stale due to 350 days of inactivity. It will 
be closed in 2 weeks if no further activity occurs. If this issue is still 
relevant, please simply write any comment. Even if closed, you can still revive 
the issue at any time or discuss it on the d...@apisix.apache.org list. Thank 
you for your contributions.


-- 
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] Baoyuantop merged pull request #1483: chore: use CDN to speed up blog images

2023-01-20 Thread GitBox


Baoyuantop merged PR #1483:
URL: https://github.com/apache/apisix-website/pull/1483


-- 
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] Yilialinn commented on pull request #1468: Add blog post "Accessing Apisix-dashboard from everywhere with Keycloak authentication"

2023-01-20 Thread GitBox


Yilialinn commented on PR #1468:
URL: https://github.com/apache/apisix-website/pull/1468#issuecomment-1398149950

   @MirtoBusico Hello, the picture is missing. Could you send me another 
available link?


-- 
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 #1483: Update pictures

2023-01-20 Thread GitBox


netlify[bot] commented on PR #1483:
URL: https://github.com/apache/apisix-website/pull/1483#issuecomment-1398132686

   ###  Deploy Preview for *apache-apisix* 
processing.
   
   
   |  Name | Link |
   |-||
   | Latest commit | 
f168f225bcd2bcb12ee3239505c19b094309298e |
   | Latest deploy log | 
https://app.netlify.com/sites/apache-apisix/deploys/63ca5fedb66729000828559d |


-- 
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] Yilialinn opened a new pull request, #1483: Update pictures

2023-01-20 Thread GitBox


Yilialinn opened a new pull request, #1483:
URL: https://github.com/apache/apisix-website/pull/1483

   Fixes: #[Add issue number here]
   
   Changes:
   
   
   
   Screenshots of the change:
   
   
   


-- 
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] lifeblood opened a new issue, #8717: help request: access logs can not delete

2023-01-20 Thread GitBox


lifeblood opened a new issue, #8717:
URL: https://github.com/apache/apisix/issues/8717

   ### Description
   
   plugin_attr:
 log-rotate:
 interval: 76400# rotate interval (unit: second)
 max_kept: 24 # max number of log files will be kept
 max_size: -1  # max size of log files will be kept
 enable_compression: false# enable log file compression(gzip) or 
not, default false
   
   access logs can not delete
   
   ### Environment
   
   - APISIX version (run `apisix version`):
   - Operating system (run `uname -a`):
   - 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] codecov-commenter commented on pull request #2719: feat: release 3.0.0

2023-01-20 Thread GitBox


codecov-commenter commented on PR #2719:
URL: 
https://github.com/apache/apisix-dashboard/pull/2719#issuecomment-1398103050

   # 
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#2719](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (7d26f15) into 
[master](https://codecov.io/gh/apache/apisix-dashboard/commit/688afa2099b1782bb401083085b2384f27c9cc6a?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (688afa2) will **decrease** coverage by `22.14%`.
   > The diff coverage is `n/a`.
   
   ```diff
   @@ Coverage Diff @@
   ##   master#2719   +/-   ##
   ===
   - Coverage   76.39%   54.26%   -22.14% 
   ===
 Files 139   86   -53 
 Lines3754 4458  +704 
 Branches  874  268  -606 
   ===
   - Hits 2868 2419  -449 
   - Misses886 1836  +950 
   - Partials0  203  +203 
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | backend-unit-test | `50.08% <ø> (?)` | |
   | frontend-e2e-test | `67.98% <ø> (-8.42%)` | :arrow_down: |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | [Impacted 
Files](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[web/src/helpers.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9oZWxwZXJzLnRzeA==)
 | `19.67% <0.00%> (-57.38%)` | :arrow_down: |
   | 
[web/src/components/RightContent/AvatarDropdown.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1JpZ2h0Q29udGVudC9BdmF0YXJEcm9wZG93bi50c3g=)
 | `50.00% <0.00%> (-32.15%)` | :arrow_down: |
   | 
[web/src/components/PanelSection/index.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BhbmVsU2VjdGlvbi9pbmRleC50c3g=)
 | `75.00% <0.00%> (-25.00%)` | :arrow_down: |
   | 
[...b/src/components/Plugin/UI/referer-restriction.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9yZWZlcmVyLXJlc3RyaWN0aW9uLnRzeA==)
 | `69.69% <0.00%> (-21.22%)` | :arrow_down: |
   | 
[web/src/pages/Consumer/List.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9wYWdlcy9Db25zdW1lci9MaXN0LnRzeA==)
 | `73.17% <0.00%> (-19.52%)` | :arrow_down: |
   | 
[web/src/components/Plugin/PluginPage.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5QYWdlLnRzeA==)
 | `82.00% <0.00%> (-17.00%)` | :arrow_down: |
   | 
[web/src/components/ActionBar/ActionBar.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL0FjdGlvbkJhci9BY3Rpb25CYXIudHN4)
 | `69.23% <0.00%> (-15.39%)` | :arrow_down: |
   | 
[web/src/app.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9hcHAudHN4)
 | `87.87% <0.00%> (-12.13%)` | :arrow_down: |
   | 
[web/src/components/Plugin/UI/cors.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2719?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9jb3JzLnRzeA==)
 | `71.11% <0.00%> (-11.12%)` | :arrow_down: |
   | 

[GitHub] [apisix-dashboard] Baoyuantop opened a new pull request, #2719: feat: release 3.0.0

2023-01-20 Thread GitBox


Baoyuantop opened a new pull request, #2719:
URL: https://github.com/apache/apisix-dashboard/pull/2719

   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
   - [ ] Improve performance
   - [ ] Backport patches
   
   **What changes will this PR take into?**
   
   Release APISIX Dashboard 3.0.0
   
   **Related issues**
   
   fix/resolve #0001
   
   **Checklist:**
   
   - [x] Did you explain what problem does this PR solve? Or what new features 
have been added?
   - [ ] Have you added corresponding test cases?
   - [x] Have you modified the corresponding document?
   - [x] 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-ingress-controller] navendu-pottekkat commented on a diff in pull request #1590: chore: update issue templates

2023-01-20 Thread GitBox


navendu-pottekkat commented on code in PR #1590:
URL: 
https://github.com/apache/apisix-ingress-controller/pull/1590#discussion_r1082241622


##
.github/ISSUE_TEMPLATE/improve_docs.yml:
##
@@ -0,0 +1,33 @@
+name: "Documentation Issue"
+description: Issues related to documentation.
+title: "docs: "
+labels: [doc]
+body:
+  - type: markdown
+attributes:
+  value: |
+_The more information you share, the faster we can help you._
+
+Prior to opening the issue, please make sure that you:
+
+- Use English to communicate.
+- Search the [open 
issues](https://github.com/apache/apisix-ingress-controller/issues) to avoid 
duplicating the issue.
+
+  - type: textarea
+id: current-state
+attributes:
+  label: Current State
+  description: Describe the current state of the documentation.
+  placeholder: |
+The documentation for the API in this page (url) is missing ...

Review Comment:
   Do you mean to replace "documentation for the API" with "documentation for 
the function"? It is not clear to me.



-- 
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] MarkCupitt closed issue #8663: bug: failed to find any SSL certificate by SNI:

2023-01-20 Thread GitBox


MarkCupitt closed issue #8663: bug: failed to find any SSL certificate by SNI:
URL: https://github.com/apache/apisix/issues/8663


-- 
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] Baoyuantop commented on issue #2661: The node field is lost after the apisix-dashboard submission

2023-01-20 Thread GitBox


Baoyuantop commented on issue #2661:
URL: 
https://github.com/apache/apisix-dashboard/issues/2661#issuecomment-1398054411

   Hi @Bigwen-1, please provide information about the version you are using.


-- 
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] Baoyuantop closed issue #2707: ip限制插件配置UI展示异常

2023-01-20 Thread GitBox


Baoyuantop closed issue #2707: ip限制插件配置UI展示异常
URL: https://github.com/apache/apisix-dashboard/issues/2707


-- 
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] Baoyuantop merged pull request #2718: fix: plugin configuration lost in editor

2023-01-20 Thread GitBox


Baoyuantop merged PR #2718:
URL: https://github.com/apache/apisix-dashboard/pull/2718


-- 
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] Baoyuantop closed issue #2696: 2.15.0版本client-control配置后显示问题

2023-01-20 Thread GitBox


Baoyuantop closed issue #2696: 2.15.0版本client-control配置后显示问题
URL: https://github.com/apache/apisix-dashboard/issues/2696


-- 
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] Baoyuantop commented on issue #2664: SSL certificates upload by dashboard not working

2023-01-20 Thread GitBox


Baoyuantop commented on issue #2664:
URL: 
https://github.com/apache/apisix-dashboard/issues/2664#issuecomment-1398047584

   fixed in https://github.com/apache/apisix-dashboard/pull/2693


-- 
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] Baoyuantop closed issue #2664: SSL certificates upload by dashboard not working

2023-01-20 Thread GitBox


Baoyuantop closed issue #2664: SSL certificates upload by dashboard not working
URL: https://github.com/apache/apisix-dashboard/issues/2664


-- 
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] Baoyuantop commented on issue #2671: bug: APISIX Dashboard Route config setting error

2023-01-20 Thread GitBox


Baoyuantop commented on issue #2671:
URL: 
https://github.com/apache/apisix-dashboard/issues/2671#issuecomment-1398039150

   Hi @Laniakearr, can you try again with the dashboard latest 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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [apisix-dashboard] Baoyuantop commented on issue #2673: Source-based installation fails

2023-01-19 Thread GitBox


Baoyuantop commented on issue #2673:
URL: 
https://github.com/apache/apisix-dashboard/issues/2673#issuecomment-1398034281

   Hi @Cheol-Soon-Choi, if there are still problems, you can reopen


-- 
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] Baoyuantop closed issue #2673: Source-based installation fails

2023-01-19 Thread GitBox


Baoyuantop closed issue #2673: Source-based installation fails
URL: https://github.com/apache/apisix-dashboard/issues/2673


-- 
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] Baoyuantop commented on issue #2687: bug: dashboard should not support '!' in vars.

2023-01-19 Thread GitBox


Baoyuantop commented on issue #2687:
URL: 
https://github.com/apache/apisix-dashboard/issues/2687#issuecomment-1398030408

   Hi @840963657, can you help with the specific steps to create this route 
using the dashboard? Also, you can try the latest version of the dashboard.


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1398028051

   Perhaps you have other routes that conflict with this route? If you 
configure a /* route, it is possible that the route will override the match of 
the /food/* route. Please checkout


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1398026429

   I really don't know why this can't work for your local system.
   This is my step.
   First, create an upstream.
   ```json
   {
 "nodes": [
   {
 "host": "httpbin.org",
 "port": 80,
 "weight": 1
   }
 ],
 "timeout": {
   "connect": 6,
   "send": 6,
   "read": 6
 },
 "type": "roundrobin",
 "scheme": "http",
 "pass_host": "pass",
 "name": "httpbin",
 "keepalive_pool": {
   "idle_timeout": 60,
   "requests": 1000,
   "size": 320
 }
   }
   ```
   I'm using `httpbin.org` for the test.
   https://user-images.githubusercontent.com/11229089/213642473-53840dbd-b85f-414e-9004-54f236718e40.png;>
   Now I get an upstream id: 444043510262268609
   
   Secondly, open a `Raw data editor` in route config page.
   https://user-images.githubusercontent.com/11229089/213642851-a2ecf3bd-af5d-4bf3-ad5e-11d53bc9293f.png;>
   
   Then, paste all configs you are posting previously. Only change upstream id 
to my upstream id
   https://user-images.githubusercontent.com/11229089/213642943-87dd06a8-34b4-48e2-9e42-f1275cd5ccfe.png;>
   
   Then, create this route successfully.
   https://user-images.githubusercontent.com/11229089/213643215-80951ab0-2c5f-4edc-a2d8-011bf4479cce.png;>
   
   Finally, I access from curl
   https://user-images.githubusercontent.com/11229089/213643475-a98eddd4-50db-44e3-950d-7e3ebb8880a9.png;>
   
   https://user-images.githubusercontent.com/11229089/213643552-86877b07-c82c-4f59-90f4-f38d1bca2959.png;>
   
   You can see, I access using `/food/get`, and APISIX has rewritten this URL 
to `/get` success.
   So when I access `http://localhost:9080/food/get` APISIX will access 
`http://httpbin.org/get` and return what their return.
   This means everything works correctly.


-- 
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] Baoyuantop commented on issue #2691: dashboard can not start when a custom discovery plugins enabled

2023-01-19 Thread GitBox


Baoyuantop commented on issue #2691:
URL: 
https://github.com/apache/apisix-dashboard/issues/2691#issuecomment-1398026131

   Hi @jagerzhang, have you tried to synchronize schema information? You can 
refer here 
https://apisix.apache.org/docs/dashboard/FAQ/#4-after-modifying-the-plugin-schema-or-creating-a-custom-plugin-in-apache-apisix-why-cant-i-find-it-on-the-dashboard


-- 
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] codecov-commenter commented on pull request #2718: fix: plugin configuration lost in editor

2023-01-19 Thread GitBox


codecov-commenter commented on PR #2718:
URL: 
https://github.com/apache/apisix-dashboard/pull/2718#issuecomment-1398023204

   # 
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#2718](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (4e6c304) into 
[master](https://codecov.io/gh/apache/apisix-dashboard/commit/7b8d9c5b30a8eae2ca038b99cf2a3c36cdca2ddc?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (7b8d9c5) will **increase** coverage by `12.96%`.
   > The diff coverage is `100.00%`.
   
   ```diff
   @@ Coverage Diff @@
   ##   master#2718   +/-   ##
   ===
   + Coverage   55.01%   67.98%   +12.96% 
   ===
 Files 104   39   -65 
 Lines2983 1040 -1943 
 Branches  743  268  -475 
   ===
   - Hits 1641  707  -934 
   + Misses   1342  333 -1009 
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | frontend-e2e-test | `67.98% <100.00%> (+12.96%)` | :arrow_up: |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | [Impacted 
Files](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[web/src/components/Plugin/PluginDetail.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5EZXRhaWwudHN4)
 | `64.48% <100.00%> (-10.93%)` | :arrow_down: |
   | 
[web/src/components/PanelSection/index.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BhbmVsU2VjdGlvbi9pbmRleC50c3g=)
 | `75.00% <0.00%> (-25.00%)` | :arrow_down: |
   | 
[web/src/helpers.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9oZWxwZXJzLnRzeA==)
 | `19.67% <0.00%> (-19.68%)` | :arrow_down: |
   | 
[web/src/components/ActionBar/ActionBar.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL0FjdGlvbkJhci9BY3Rpb25CYXIudHN4)
 | `69.23% <0.00%> (-7.70%)` | :arrow_down: |
   | 
[web/src/components/Plugin/PluginPage.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5QYWdlLnRzeA==)
 | `82.00% <0.00%> (-5.00%)` | :arrow_down: |
   | 
[web/src/components/Plugin/UI/plugin.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9wbHVnaW4udHN4)
 | `87.50% <0.00%> (-4.17%)` | :arrow_down: |
   | 
[web/src/app.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9hcHAudHN4)
 | `87.87% <0.00%> (-3.04%)` | :arrow_down: |
   | 
[web/src/components/Plugin/UI/api-breaker.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9VSS9hcGktYnJlYWtlci50c3g=)
 | `69.23% <0.00%> (ø)` | |
   | 
[...mponents/Upstream/components/active-check/Port.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2718?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvYWN0aXZlLWNoZWNrL1BvcnQudHN4)
 | | |
   | ... and [77 

[GitHub] [apisix-dashboard] Baoyuantop commented on issue #2707: ip限制插件配置UI展示异常

2023-01-19 Thread GitBox


Baoyuantop commented on issue #2707:
URL: 
https://github.com/apache/apisix-dashboard/issues/2707#issuecomment-1398021204

   Looks similar to https://github.com/apache/apisix-dashboard/issues/2696


-- 
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] Baoyuantop opened a new pull request, #2718: fix: plugin configuration lost in editor

2023-01-19 Thread GitBox


Baoyuantop opened a new pull request, #2718:
URL: https://github.com/apache/apisix-dashboard/pull/2718

   Please answer these questions before submitting a pull request, **or your PR 
will get closed**.
   
   **Why submit this pull request?**
   
   - [x] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   - [ ] Backport patches
   
   **What changes will this PR take into?**
   
   Please update this section with detailed description.
   
   **Related issues**
   
   fix/resolve https://github.com/apache/apisix-dashboard/issues/2696
   
   **Checklist:**
   
   - [x] Did you explain what problem does this PR solve? Or what new features 
have been added?
   - [x] 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] xiaoxuanzi opened a new pull request, #8716: feat(clickhouse-logger): unify endpoints into one schema

2023-01-19 Thread GitBox


xiaoxuanzi opened a new pull request, #8716:
URL: https://github.com/apache/apisix/pull/8716

   ### Description
   
   clickhouse-logger: unify endpoints into one schema
   
   
   
   
   ### 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
   - [x] 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



[GitHub] [apisix-ingress-controller] tao12345666333 commented on a diff in pull request #1590: chore: update issue templates

2023-01-19 Thread GitBox


tao12345666333 commented on code in PR #1590:
URL: 
https://github.com/apache/apisix-ingress-controller/pull/1590#discussion_r1082165587


##
.github/ISSUE_TEMPLATE/improve_docs.yml:
##
@@ -0,0 +1,33 @@
+name: "Documentation Issue"
+description: Issues related to documentation.
+title: "docs: "
+labels: [doc]
+body:
+  - type: markdown
+attributes:
+  value: |
+_The more information you share, the faster we can help you._
+
+Prior to opening the issue, please make sure that you:
+
+- Use English to communicate.
+- Search the [open 
issues](https://github.com/apache/apisix-ingress-controller/issues) to avoid 
duplicating the issue.
+
+  - type: textarea
+id: current-state
+attributes:
+  label: Current State
+  description: Describe the current state of the documentation.
+  placeholder: |
+The documentation for the API in this page (url) is missing ...
+validations:
+  required: true
+  - type: textarea
+id: desired-state
+attributes:
+  label: Desired State
+  description: Describe the desired state the documentation should be in.
+  placeholder: |
+There should be line mentioning how the API behaves when ...

Review Comment:
   Ditto 



##
.github/ISSUE_TEMPLATE/improve_docs.yml:
##
@@ -0,0 +1,33 @@
+name: "Documentation Issue"
+description: Issues related to documentation.
+title: "docs: "
+labels: [doc]
+body:
+  - type: markdown
+attributes:
+  value: |
+_The more information you share, the faster we can help you._
+
+Prior to opening the issue, please make sure that you:
+
+- Use English to communicate.
+- Search the [open 
issues](https://github.com/apache/apisix-ingress-controller/issues) to avoid 
duplicating the issue.
+
+  - type: textarea
+id: current-state
+attributes:
+  label: Current State
+  description: Describe the current state of the documentation.
+  placeholder: |
+The documentation for the API in this page (url) is missing ...

Review Comment:
   For the function?



-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1397988920

   Hi @mscb402 ,
   
   **192.168.1.78:8090** is the address of the upstream and i have fixed it.
   now applying the below configuration
   
   ``` json
   {
 "uri": "/food/*",
 "name": "food-service-route",
 "methods": [
   "GET",
   "POST",
   "PUT",
   "DELETE",
   "PATCH",
   "HEAD",
   "OPTIONS",
   "CONNECT",
   "TRACE"
 ],
 "plugins": {
   "proxy-rewrite": {
 "regex_uri": [
   "^/food/(.*)",
   "/$1"
 ]
   }
 },
 "upstream_id": "443949312083231431",
 "status": 1
   }
   ```
   
   
![image](https://user-images.githubusercontent.com/21145779/213635779-ba5e1709-22a1-4f8e-906f-a79afc443e5d.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-ingress-controller] tao12345666333 commented on issue #1608: bug: Block request method with Apisix Ingress Controller annotation

2023-01-19 Thread GitBox


tao12345666333 commented on issue #1608:
URL: 
https://github.com/apache/apisix-ingress-controller/issues/1608#issuecomment-1397988082

   You mean the request should not be sent upstream, right?


-- 
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] Fabriceli commented on pull request #8651: fix: fix fetch all service info from consul

2023-01-19 Thread GitBox


Fabriceli commented on PR #8651:
URL: https://github.com/apache/apisix/pull/8651#issuecomment-1397953655

   @spacewander Please take a look at this PR, 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-website] SylviaBABY merged pull request #1478: docs: add apisix with consul blog

2023-01-19 Thread GitBox


SylviaBABY merged PR #1478:
URL: https://github.com/apache/apisix-website/pull/1478


-- 
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] Fabriceli commented on pull request #1478: docs: add apisix with consul blog

2023-01-19 Thread GitBox


Fabriceli commented on PR #1478:
URL: https://github.com/apache/apisix-website/pull/1478#issuecomment-1397944158

   @SylviaBABY @yzeng25 I have no permission to merge this PR, Could you merge 
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] navendu-pottekkat opened a new issue, #8715: docs: consolidate deployment modes guides

2023-01-19 Thread GitBox


navendu-pottekkat opened a new issue, #8715:
URL: https://github.com/apache/apisix/issues/8715

   ### Current State
   
   The [deployment modes documentation 
page](https://apisix.apache.org/docs/apisix/deployment-modes/) contains 
documentation about all the deployment modes. There is also another page for 
[stand-alone mode](https://apisix.apache.org/docs/apisix/stand-alone/) with 
just its documentation.
   
   ### Desired State
   
   Instead, these two pages should be merged to a single page and moved out of 
the "Others" section.


-- 
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-website] SkyeYoung closed issue #1481: [Site]: Blog "Accessing APISIX-Dashboard from Everywhere with Keycloak Authentication" not accessible

2023-01-19 Thread GitBox


SkyeYoung closed issue #1481: [Site]: Blog "Accessing APISIX-Dashboard from 
Everywhere with Keycloak Authentication" not accessible
URL: https://github.com/apache/apisix-website/issues/1481


-- 
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 merged pull request #1482: fix: front matter

2023-01-19 Thread GitBox


SkyeYoung merged PR #1482:
URL: https://github.com/apache/apisix-website/pull/1482


-- 
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 #1482: fix: front matter

2023-01-19 Thread GitBox


netlify[bot] commented on PR #1482:
URL: https://github.com/apache/apisix-website/pull/1482#issuecomment-1397880770

   ###  Deploy Preview for *apache-apisix* 
processing.
   
   
   |  Name | Link |
   |-||
   | Latest commit | 
e5502f779ec7b82d8df8a951c1df2ce42338268e |
   | Latest deploy log | 
https://app.netlify.com/sites/apache-apisix/deploys/63ca09d27c16330008635a25 |


-- 
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 opened a new pull request, #1482: fix: front matter

2023-01-19 Thread GitBox


SkyeYoung opened a new pull request, #1482:
URL: https://github.com/apache/apisix-website/pull/1482

   Fixes: #1481 
   Changes:
   
   * add missing author
   
   
   Screenshots of the change:
   
   
   


-- 
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] Fabriceli commented on a diff in pull request #1478: docs: add apisix with consul blog

2023-01-19 Thread GitBox


Fabriceli commented on code in PR #1478:
URL: https://github.com/apache/apisix-website/pull/1478#discussion_r1082069199


##
blog/en/blog/2023/01/18/consul-with-apisix.md:
##
@@ -0,0 +1,274 @@
+---
+title: "How to Integrate API Gateway and Consul? Not Consul K/V"
+authors:
+  - name: "Yihao LI"
+title: "Author"
+url: "https://github.com/Fabriceli;
+image_url: "https://github.com/Fabriceli.png;
+keywords:
+- API Gateway
+- Consul
+- Service Discovery
+- Service Register
+description: Apache APISIX supports the Consul service discovery registry. 
This article will walk you through the process of implementing service 
discovery and service registry in APISIX.
+tags: [Ecosystem]
+image: 
https://static.apiseven.com/2022/blog/0818/ecosystem/HashiCorp%20Consul.png
+---
+

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] SylviaBABY merged pull request #8714: docs: fix deployment doc links

2023-01-19 Thread GitBox


SylviaBABY merged PR #8714:
URL: https://github.com/apache/apisix/pull/8714


-- 
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] SkyeYoung closed issue #8698: [docs]: deployment-role.md is not exist

2023-01-19 Thread GitBox


SkyeYoung closed issue #8698: [docs]: deployment-role.md is not exist
URL: https://github.com/apache/apisix/issues/8698


-- 
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] SkyeYoung commented on issue #8698: [docs]: deployment-role.md is not exist

2023-01-19 Thread GitBox


SkyeYoung commented on issue #8698:
URL: https://github.com/apache/apisix/issues/8698#issuecomment-1397862577

   fixed in #8714 


-- 
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] SylviaBABY opened a new pull request, #8714: docs: fix deployment links

2023-01-19 Thread GitBox


SylviaBABY opened a new pull request, #8714:
URL: https://github.com/apache/apisix/pull/8714

   ### Description
   
   
   
   
   Fixes # https://github.com/apache/apisix/issues/8698, update ZH/EN docs link
   
   ### 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-website] yzeng25 commented on a diff in pull request #1478: docs: add apisix with consul blog

2023-01-19 Thread GitBox


yzeng25 commented on code in PR #1478:
URL: https://github.com/apache/apisix-website/pull/1478#discussion_r1082065305


##
blog/en/blog/2023/01/18/consul-with-apisix.md:
##
@@ -0,0 +1,274 @@
+---
+title: "How to Integrate API Gateway and Consul? Not Consul K/V"
+authors:
+  - name: "Yihao LI"
+title: "Author"
+url: "https://github.com/Fabriceli;
+image_url: "https://github.com/Fabriceli.png;
+keywords:
+- API Gateway
+- Consul
+- Service Discovery
+- Service Register
+description: Apache APISIX supports the Consul service discovery registry. 
This article will walk you through the process of implementing service 
discovery and service registry in APISIX.
+tags: [Ecosystem]
+image: 
https://static.apiseven.com/2022/blog/0818/ecosystem/HashiCorp%20Consul.png
+---
+

Review Comment:
   1. Need a `` here to cut actual contents off from the blog 
summary page
   2. Need a sentence or two to add a summary for this blog.
   ```suggestion
   > This article teaches you how to integrate Consul with Apache APISIX 
   
   
   ```
   Now it looks like this, and it is just not right.
   
   
![image](https://user-images.githubusercontent.com/36651058/213606541-05dafa7c-9119-4419-99be-df1a293d1f63.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-website] yzeng25 merged pull request #1479: docs: add weekly report 20230115

2023-01-19 Thread GitBox


yzeng25 merged PR #1479:
URL: https://github.com/apache/apisix-website/pull/1479


-- 
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] yzeng25 opened a new issue, #1481: [Site]: Blog "Accessing APISIX-Dashboard from Everywhere with Keycloak Authentication" not accessible

2023-01-19 Thread GitBox


yzeng25 opened a new issue, #1481:
URL: https://github.com/apache/apisix-website/issues/1481

   ### Current State
   
   **Description of the current state/issue**:
   
   Blog "Accessing APISIX-Dashboard from Everywhere with Keycloak 
Authentication" was added into the website via 
https://github.com/apache/apisix-website/pull/1468. But it is not accessible 
after merging.
   
   **Link to the page**:
   
   none
   
   **Steps to reproduce**
   
   **Actual behavior**
   
   Take the most recent English post as an example, the href should be: 
https://apisix.apache.org/blog/2022/12/30/release-apache-apisix-3.1.0/, in 
which:
   
   - `https://apisix.apache.org/blog/` is the prefix, fixed value fore each and 
every blog;
   - `2022/12/30/release-apache-apisix-3.1.0/` is the suffix, is generated from 
the file name.
   
   After merging https://github.com/apache/apisix-website/pull/1468, the blog 
is not showing up on the Blog summary page.
   
   Accessing the actual href 
`https://apisix.apache.org/blog/2023/01/02/accessing_apisix-dashboard_from_everywhere_with_keycloak_authentication.md`
 gives a `300` status with a 'Mutiple Choice' notification, see image below.
   
   
![image](https://user-images.githubusercontent.com/36651058/213604162-2ba0f856-f20e-4886-bc93-b310a3952f05.png)
   
   Then, all three options from the multiple choices return 404. As a result, 
this blog is not accessible.
   
   **Expected behavior**
   1. This blog shows up in the blog summary page.
   2. This blog is accessible, i.e. click the link and jump to blog content 
page.
   
   
   **Screenshots (if relevant)**:
   
   
![image](https://user-images.githubusercontent.com/36651058/213604664-527ac7ff-4ace-48b2-8ac0-35beba873459.png)
   
   
   
   ### Desired State
   
   **Description of the change you are proposing**:
   
   1. This blog shows up in the blog summary page.
   2. This blog is accessible, i.e. click the link and jump to blog content 
page.
   
   I think fixing the header part should fix this issue.
   
   ### Would you like to fix this issue?
   
   No. Please assign another contributor


-- 
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-website] yzeng25 commented on pull request #1468: Add blog post "Accessing Apisix-dashboard from everywhere with Keycloak authentication"

2023-01-19 Thread GitBox


yzeng25 commented on PR #1468:
URL: https://github.com/apache/apisix-website/pull/1468#issuecomment-1397839408

   Hi, this blog isn't showing up properly in the website. Probably need to fix 
the header part to fix this issue. The blog content part was great. You guys 
should at least check out the preview link before giving approvals and merging 
next time. @juzhiyuan @guoqqqi 


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1397820999

   Please make sure your upstream is working.


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1397819842

   This just your upstream time out. Please chekout


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1397819303

   Are you config the upstream address wrong? Dose 192.168.1.78:8090 is an 
APISIX instance?


-- 
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] MarkCupitt commented on issue #8663: bug: failed to find any SSL certificate by SNI:

2023-01-19 Thread GitBox


MarkCupitt commented on issue #8663:
URL: https://github.com/apache/apisix/issues/8663#issuecomment-1397802057

   Certs loaded by the dashboard V2 will NOT be seen in the routing, until the 
dashboard is made compatible, you MUST use the API's to load and manage certs
   
   Additionally, if you use a Wildcard Cert ["*.example.com"] the route MUST 
use `hosts: ["*.example.com"]`. if you need granular route matching, you will 
need to add a filter, or match on vars and/or priority
   
   if you use `host: "my.example.com"` it will not match, is it uses the 
literal SNI value `*.example.com`
   
   @tokers Might save a lot of grief for people if this could be added to the 
documentation in the following locations:
   
   https://apisix.apache.org/docs/apisix/admin-api/#request-body-parameters
   https://apisix.apache.org/docs/apisix/admin-api/#ssl-api
   
   Documenting in case anyone else has this 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] Baoyuantop merged pull request #2717: fix: adapt APISIX plugin disabled

2023-01-19 Thread GitBox


Baoyuantop merged PR #2717:
URL: https://github.com/apache/apisix-dashboard/pull/2717


-- 
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] MarkCupitt closed issue #8665: bug: uploading a cert via api - it is accepted with no error, shows in dashboard with correct SNI, but invalid expiry date. Cert can NOT be deleted

2023-01-19 Thread GitBox


MarkCupitt closed issue #8665: bug: uploading a cert via api - it is accepted 
with no error, shows in dashboard with correct SNI, but invalid expiry date. 
Cert can NOT be deleted
URL: https://github.com/apache/apisix/issues/8665


-- 
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] MarkCupitt commented on issue #8665: bug: uploading a cert via api - it is accepted with no error, shows in dashboard with correct SNI, but invalid expiry date. Cert can NOT be delet

2023-01-19 Thread GitBox


MarkCupitt commented on issue #8665:
URL: https://github.com/apache/apisix/issues/8665#issuecomment-1397796190

   So for those struggling with this, Dashboard V2, on ssl certs at least, is 
NOT compatible at all with apisix V3.x, you **MUST** use the apis.
   
   Documenting for anyone else who may come across this 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] MarkCupitt commented on issue #8665: bug: uploading a cert via api - it is accepted with no error, shows in dashboard with correct SNI, but invalid expiry date. Cert can NOT be delet

2023-01-19 Thread GitBox


MarkCupitt commented on issue #8665:
URL: https://github.com/apache/apisix/issues/8665#issuecomment-1397778959

   Additional effort still produces same result, 
   
   Bash Script used:
   
   ```
 local sslcrt=$(cat "./certs/$sslurl.crt")
 local sslkey=$(cat "./certs/$sslurl.key")
 
 local json=$(cat /dev/null
   ```
   Note on image:
   
   - Cert loaded via APi
   - Expiration time not decoded
   - SNI Decoded
   - Unable to delete. Noted its using /ssl and not ssls
   
   
![image](https://user-images.githubusercontent.com/5503184/213591433-2066d6c4-0e73-41f7-abae-5c2bb5196949.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-ingress-controller] hndoss opened a new issue, #1608: bug: Block request method with Apisix Ingress Controller annotation

2023-01-19 Thread GitBox


hndoss opened a new issue, #1608:
URL: https://github.com/apache/apisix-ingress-controller/issues/1608

   ### Issue description
   
   After implementing the new annotations, the affected Apisix Route has a 
plugin configuration as follows:
   ```
   "response-rewrite": {
 "body_base64": false,
 "status_code": 405,
 "vars": [
   [
 "request_method",
 "!",
 "in",
 [
   "GET",
   "POST",
   "PUT",
   "PATCH",
   "HEAD"
 ]
   ]
 ]
   }
   ```
   
   The plugin is `response-rewrite`. This plugin just overwrites the response 
from the Upstream. Meaning that, any DELETE will succeed, and Apisix will just 
overwrite the HTTP to a 405.
   
   Can 
[request-validation](https://apisix.apache.org/docs/apisix/plugins/request-validation/)
 plugin be used instead?
   
   ---
   Related 
[issue](https://github.com/apache/apisix-ingress-controller/issues/1463)
   
   ### Environment
   
   - your apisix-ingress-controller version (output of 
apisix-ingress-controller version --long):
   ```
   /ingress-apisix # /ingress-apisix/apisix-ingress-controller version --long
   Version: 1.6.0
   Git SHA: no-git-module
   Go Version: go1.19.4
   Building OS/Arch: linux/amd64
   Running OS/Arch: linux/amd64
   ```
   
   - your Kubernetes cluster version (output of kubectl version):
   ```
   Client Version: v1.25.4
   Kustomize Version: v4.5.7
   Server Version: v1.21.14-gke.3000
   ```
   
   - if you run apisix-ingress-controller in Bare-metal environment, also show 
your OS version (uname -a):
   
   
   ### Minimal test code / Steps to reproduce
   
   1. Create an ingress using Allow [http methods 
annotations](https://apisix.apache.org/docs/ingress-controller/concepts/annotations/#allow-http-methods)
   2. Attempt to use an HTTP method that is it white listed
   
   ### Actual result
   
   curl -I  -X DELETE https://my-domain.io/my-service/1
   HTTP/2 405 
   date: Thu, 19 Jan 2023 19:11:55 GMT
   content-type: application/vnd.schemaregistry.v1+json
   content-length: 4
   
   ### Error log
   
   curl -I  -X DELETE https://my-domain.io/my-service/1
   {"error_code":40404,"message":"Subject 1 was soft deleted.Set permanent=true 
to delete permanently"}
   
   ### Expected result
   
   The  https://my-domain.io/my-service/1 should be present since it is not 
possible to DELETE


-- 
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-docker] incubator4 opened a new pull request, #412: fix: apisix config etcd indent error

2023-01-19 Thread GitBox


incubator4 opened a new pull request, #412:
URL: https://github.com/apache/apisix-docker/pull/412

   Due to apisix `config.yaml` indent error, etcd configuration didn't 
passthrought to apisix,  it would cause problem that apisix use default config 
to connect etcd by `127.0.0.1:2379`


-- 
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] LiteSun commented on a diff in pull request #2704: fix plugins display in list of routes

2023-01-19 Thread GitBox


LiteSun commented on code in PR #2704:
URL: https://github.com/apache/apisix-dashboard/pull/2704#discussion_r1081381117


##
web/src/pages/Route/List.tsx:
##
@@ -550,6 +551,13 @@ const Page: React.FC = () => {
 {
   title: formatMessage({ id: 'menu.plugin' }),
   dataIndex: 'plugins',
+  width: 240,
+  render: (_, record) => {
+const plugins = record.plugins || {};
+return Object.keys(plugins).length > 0
+  ? Object.keys(plugins).map((key) =>  {key})

Review Comment:
   ```suggestion
 ? Object.keys(plugins).map((key) => {key})
   ```



-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1397020304

   Now I'm getting **504 Gateway timeout**
   
![image](https://user-images.githubusercontent.com/21145779/213460809-18921094-cf4e-4066-9d4a-d88cea34b789.png)
   
   and based on the log
   ```
   2023/01/19 13:53:24 [error] 51#51: *310593 upstream timed out (110: 
Operation timed out) while connecting to upstream, client: 172.18.0.1, server: 
_, request: "GET /food/v1/dishes HTTP/1.1", upstream: 
"http://192.168.1.78:8090/food/v1/dishes;, host: "localhost:9080"
   ```
   it is forwarding "http://192.168.1.78:8090/food/v1/dishes; and 
/food/v1/dishes dosent exists in port 8090
   
   
![image](https://user-images.githubusercontent.com/21145779/213460756-36d99f8b-004f-410d-a35c-c17547fa7448.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-ingress-controller] navendu-pottekkat opened a new pull request, #1607: docs: add Gateway API example to the "Getting started" guide

2023-01-19 Thread GitBox


navendu-pottekkat opened a new pull request, #1607:
URL: https://github.com/apache/apisix-ingress-controller/pull/1607

   Signed-off-by: Navendu Pottekkat 
   
   Add an example of the Gateway API to the "Getting started" guide.


-- 
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-docker] sbrouet opened a new issue, #411: Not same user (root VS apisix) used depending on Linux distribution

2023-01-19 Thread GitBox


sbrouet opened a new issue, #411:
URL: https://github.com/apache/apisix-docker/issues/411

   Hello Team,
   
   I noticed that for some Docker images, the `root`  user is used, such as 
`centos`.
   For others the `apisix` user is used, such as for `debian`.
   This could lead to different behavior since user rights are different, so 
something that would work in `centos` Docker image would fail in `debian` one.
   
   -> Suggestion: use applicative `apisix` in all Docker images and same user 
rights on directories
   
   Thank you,
   Sébastien


-- 
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-docker] sbrouet opened a new issue, #410: Possible fix in entrypoint.sh - hardcoded path

2023-01-19 Thread GitBox


sbrouet opened a new issue, #410:
URL: https://github.com/apache/apisix-docker/issues/410

   Hello Team,
   
   In entrypoint.sh for debian, at line 43 the path to configuration directory 
is hardcoded to `/usr/local/apisix`
   
   See 
https://github.com/apache/apisix-docker/blob/master/debian/docker-entrypoint.sh#L43
   ```
   exec /usr/local/openresty/bin/openresty -p /usr/local/apisix -g 'daemon off;'
   ```
   
   But configuration directory can be configured with variable PREFIX which is 
declared at beginning of script and used for some commands, for initializing 
configuration:
   ```
   PREFIX=${APISIX_PREFIX:=/usr/local/apisix}
   [...]
   cat > ${PREFIX}/conf/config.yaml << _EOC_
   [...]
   cat > ${PREFIX}/conf/apisix.yaml << _EOC_
   ```
   
   -> It seems to me that variable should be used also in exec command show 
above, which would look like:
   ```
   exec /usr/local/openresty/bin/openresty -p ${PREFIX} -g 'daemon off;'
   ```
   
   If change is needed, must be done in all `docker-entrypoint.sh` files in Git 
repository.
   
   Thank you,
   Sébastien
   


-- 
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-ingress-controller] tao12345666333 closed issue #1547: chore: improve make update-all

2023-01-19 Thread GitBox


tao12345666333 closed issue #1547: chore: improve make update-all 
URL: https://github.com/apache/apisix-ingress-controller/issues/1547


-- 
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] tao12345666333 commented on issue #454: doc: Fix standalone mode FAQ

2023-01-19 Thread GitBox


tao12345666333 commented on issue #454:
URL: 
https://github.com/apache/apisix-helm-chart/issues/454#issuecomment-1396939889

   Thanks! Would you like to open a PR for this?


-- 
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-docker] shivahnshankar commented on issue #409: Discuss: unable to pull arm64 etcd image correctly on M1 arm arch

2023-01-19 Thread GitBox


shivahnshankar commented on issue #409:
URL: https://github.com/apache/apisix-docker/issues/409#issuecomment-1396919772

   I even tried `pachyderm/etcd` and `docker/desktop-kubernetes-etcd`. The etcd 
service starts but the dashboard fails to connect to the service. 
   I am exploring whether this is the issue: 
[https://github.com/etcd-io/etcd/issues/10677#issuecomment-660935539](https://github.com/etcd-io/etcd/issues/10677#issuecomment-660935539)
 in etcd_data mount


-- 
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] sbrouet opened a new issue, #454: doc: Fix standalone mode FAQ

2023-01-19 Thread GitBox


sbrouet opened a new issue, #454:
URL: https://github.com/apache/apisix-helm-chart/issues/454

   Hello Team,
   
   I spent some time trying to have standalone mode working on Kubernetes.
   I used information from the official Helm Chart FAQ here: 
https://github.com/apache/apisix-helm-chart/blob/master/docs/en/latest/FAQ.md#how-to-install-apache-apisix-running-in-standalone-mode
   
   I found out several issues, and also would like to propose some remarks and 
changes.
   I'll explain all of them here, please feel free to comment.
   If adequate we can create separate issues and close this one.
   
   ### Issue 1: sample Kubernetes files out of date
    Issue 1.1 : to be removed: sample with alpine based Docker image
   Alpine docker image have been removed from apisix-docker repository (see 
https://github.com/apache/apisix-docker and 
https://github.com/apache/apisix-docker/pull/395)
   -> Recommendation: remove sample with alpine docker image
   
    Issue 1.2 : sample with centos based Docker image not working, but OK 
with debian
   When using exact sample file, the Apisix gateway pod does start, but the 
routes configuration file (apisix.yaml) is not present while it is supposely 
mounted by the deployment in `/apisix-config/` directory
   ```
   # From inside pod:
   ~ # find / -name apisix.yaml
   ~ # ls -l /apisix-config/
   ls: /apisix-config/: No such file or directory
   ```
   
   The `apisix.yaml` file located in `/usr/local/apisix/conf/` contains default 
file with empty routes list, instead of being replaced by symbolic link from 
/apisix-config/apisix.yaml.
   
   When running exact same file and just changing the docker image to a debian 
one, it worked perfectly:
   ```
   image: "apache/apisix:3.1.0-debian"
   ```
   Thank you @AlinsRan for you advice.
   -> Recommendation: change centos example to debian
   
   
   ### Issue 2:  possible fix in entrypoint.sh
   In entrypoint.sh for debian, at line 43 the path to configuration directory 
is hardcoded to `/usr/local/apisix`
   See 
https://github.com/apache/apisix-docker/blob/master/debian/docker-entrypoint.sh#L43
   ```
   exec /usr/local/openresty/bin/openresty -p /usr/local/apisix -g 'daemon off;'
   ```
   But configuration directory can be configured with variable  PREFIX which is 
declared at begining of script and used for some commands, for initializing 
configuration
   ```
   PREFIX=${APISIX_PREFIX:=/usr/local/apisix}
   [...]
   cat > ${PREFIX}/conf/config.yaml << _EOC_
   [...]
   cat > ${PREFIX}/conf/apisix.yaml << _EOC_
   ```
   
   -> It seems to me that variable should be used also in exec command show 
above, which would look like:
   ```
   exec /usr/local/openresty/bin/openresty -p ${PREFIX} -g 'daemon off;'
   ```
   If change is needed, must be done in all `docker-entrypoint.sh` files in Git 
repository.
   
   
   ### Issue 3: add recommendation for Linux distribution to be used
   Samples in FAQ for standalone mode propose different distributions.
   Official Helm Chart uses debian based docker image
   -> Suggestion: clarify in FAQ what should be recommended distribution. Seems 
logical to me it should be same one as Helm Charts. Also if any specifics are 
required they should be clear.
   
   ### Issue 4: remark about not same user (root VS apisix) used depending on 
Linux distribution
   I noticed that for some Docker images, the `root`  user is used, such as 
centos.
   For others the `apisix` user is used, such as for debian.
   This could lead to different behavior since user rights are different, so 
something that would work in centos Docker image would fail in debian one.
   -> Suggestion: use applicative `apisix` in all Docker images
   Probably this issue should be created in apache/apisix-docker repository, 
but I wanted to mention it here for people who have same issues as I had. 
   Let me know if I should move it.
   
   ### Issue 5: confusing volume name in sample Kubernetes sample files
   It seems to me the volume name `apisix-admin` is confusing. In fact it 
contains the gateway routes configuration.
   It could be mistaken with something related to the Apisix admin API.
   
   See samples at 
https://github.com/apache/apisix-helm-chart/blob/master/docs/en/latest/FAQ.md#how-to-install-apache-apisix-running-in-standalone-mode
   ```
   volumes:
   [...]
   - configMap:
   name: apisix.yaml
 name: **apisix-admin**
   ```
   
   -> Suggestion: rename it, avoid `admin`. Naming proposals: `apisix-routes` 
or `apisix-routes-config` or `apisix-routes-config-yaml`
   
   
   Hope this helps !
   Thank you all Team for your good work !
   
   Sébastien


-- 
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 

[GitHub] [apisix] uran0sH commented on issue #5453: request help: proxy-cache plugin, why the cache expiration time cannot be configured dynamically?

2023-01-19 Thread GitBox


uran0sH commented on issue #5453:
URL: https://github.com/apache/apisix/issues/5453#issuecomment-1396885215

   And I have another question: I modify the value of cache_ttl of proxy_cache 
to 1s in 'conf/default_config.yaml'. It doesn't 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] Cheol-Soon-Choi closed issue #8712: help request: traffic-split setting

2023-01-19 Thread GitBox


Cheol-Soon-Choi closed issue #8712: help request: traffic-split setting
URL: https://github.com/apache/apisix/issues/8712


-- 
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] Cheol-Soon-Choi commented on issue #8712: help request: traffic-split setting

2023-01-19 Thread GitBox


Cheol-Soon-Choi commented on issue #8712:
URL: https://github.com/apache/apisix/issues/8712#issuecomment-1396738130

   answer is 
   
   "vars": [
   ["http_test", "IN", ["1", "3"]]
 ]
   
   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] github-actions[bot] closed issue #3690: Support for AzureAD/OIDC integration with APISIX

2023-01-19 Thread GitBox


github-actions[bot] closed issue #3690: Support for AzureAD/OIDC integration 
with APISIX
URL: https://github.com/apache/apisix/issues/3690


-- 
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 #3690: Support for AzureAD/OIDC integration with APISIX

2023-01-19 Thread GitBox


github-actions[bot] commented on issue #3690:
URL: https://github.com/apache/apisix/issues/3690#issuecomment-1396718990

   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] kellyseeme opened a new issue, #8713: when the etcd use dns ,will be some error

2023-01-19 Thread GitBox


kellyseeme opened a new issue, #8713:
URL: https://github.com/apache/apisix/issues/8713

   ### Description
   
   when the etcd connection is use domain,then in two vm cluster there one have 
some error
   `2023/01/19 11:09:53 [warn] 10007#10007: *10702332 [lua] v3.lua:716: 
request_chunk(): http://lb-etcd-03.test.com:2379: failed to parse domain: 
failed to parse domain. Retrying, context: ngx.timer
   `
   interval have 1min,10min
   
   how this happen ?the dns resolve is good ,becase another vm resolve  is ok.
   
   ### Environment
   
   - APISIX version (run `apisix version`):
   - Operating system (run `uname -a`):
   - 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] kellyseeme commented on issue #8702: if etcd is down,the apisix not affect service?

2023-01-19 Thread GitBox


kellyseeme commented on issue #8702:
URL: https://github.com/apache/apisix/issues/8702#issuecomment-1396692532

   if etcd clsuter is down,the apisix is cache the configuretion ?if this is 
,then apisix will have the cache time,so how much time will be affect if etcd 
cluster down?


-- 
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] Cheol-Soon-Choi opened a new issue, #8712: help request: traffic-split setting

2023-01-19 Thread GitBox


Cheol-Soon-Choi opened a new issue, #8712:
URL: https://github.com/apache/apisix/issues/8712

   ### Description
   
   Referring to "https://github.com/api7/lua-resty-expr#operator-list;, 
traffic-split was set up.
   I'd appreciate it if you could check out how to use "in" in vars Attributes.
   
   ex1) response ok
 "vars": [
   ["http_test", "==", "1"]
 ]
   ex2) response ok
 "vars": [
   ["http_test", "==", "2"]
 ]
   ex3) 503 error
 "vars": [
   ["http_test", "in", "{\"1\", \"2\"}"]
 ]
   ex4) 503 error
 "vars": [
   ["http_test", "in", "{1, 2}"]
 ]
   
   Please check if there is a problem with usage.
   
   ### Environment
   
   - APISIX version (run `apisix version`):
   - Operating system (run `uname -a`):
   - 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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396640312

   I don't know what happens too. Maybe you can try to change `^/food/(.*)$` to 
`^/food/(.*)`.


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396628735

   Look like APISIX still using `/food/v1/dishes` to access your upstream. Look 
like this plugin does not config correctly.


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396621439

   here is the access.log 
   ```
   172.20.0.1 - - [18/Jan/2023:08:59:51 +] 127.0.0.1:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.142 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.144 "http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:00:28 +] 127.0.0.1:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.017 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.016 "http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:00:32 +] 127.0.0.1:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.006 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.008 "http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:01:26 +] 127.0.0.1:9080 "GET /food 
HTTP/1.1" 404 47 0.000 "-" "curl/7.81.0" - - - "http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:01:31 +] 127.0.0.1:9080 "GET /food/ 
HTTP/1.1" 404 105 0.006 "-" "curl/7.81.0" 192.168.1.78:8090 404 0.008 
"http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:01:34 +] 127.0.0.1:9080 "GET /food/v1/ 
HTTP/1.1" 404 108 0.015 "-" "curl/7.81.0" 192.168.1.78:8090 404 0.016 
"http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:01:37 +] 127.0.0.1:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.018 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.016 "http://127.0.0.1:9080;
   172.20.0.1 - - [18/Jan/2023:09:01:56 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.014 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.012 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:03:55 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 502 154 0.000 "-" "curl/7.81.0"  - 0.000 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:04:12 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 502 154 0.000 "-" "curl/7.81.0"  - 0.000 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:04:23 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.012 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.012 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:05:18 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.019 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.016 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:06:19 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.009 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.008 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:06:24 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.016 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.016 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:10:12 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.006 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.008 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:11:16 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.015 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.012 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:25:07 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 404 47 0.000 "-" "curl/7.81.0" - - - "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:25:11 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.011 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.012 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:25:59 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.004 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.004 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:26:36 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.007 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.008 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:26:51 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.004 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.004 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:27:38 +] localhost:9080 "GET 
/food/v1/dishes HTTP/1.1" 404 114 0.009 "-" "curl/7.81.0" 192.168.1.78:8090 404 
0.008 "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:09:29:46 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.015 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.016 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:13:02:43 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.428 "-" "curl/7.81.0" 192.168.1.78:8090 200 0.428 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:13:02:55 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 200 493 0.009 "-" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:108.0) 
Gecko/20100101 Firefox/108.0" 192.168.1.78:8090 200 0.008 
"http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:13:14:21 +] localhost:9080 "GET /v1/dishes 
HTTP/1.1" 404 47 0.000 "-" "Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:108.0) 
Gecko/20100101 Firefox/108.0" - - - "http://localhost:9080;
   172.20.0.1 - - [18/Jan/2023:13:14:21 +] localhost:9080 "GET /favicon.ico 
HTTP/1.1" 404 47 0.000 "http://localhost:9080/v1/dishes; "Mozilla/5.0 (X11; 
Ubuntu; Linux x86_64; rv:108.0) Gecko/20100101 Firefox/108.0" - - - 

[GitHub] [apisix] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396620177

   Okay, Look like this is a DNS error to query httpbin.org. But that's not 
relevant to your previous question. I just want to know what path the upstream 
server receives. Httpbin will help me to look at the request message. Do you 
have any way to print the access log from your upstream server?


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396603017

   yeah here for the last update you wanted me to do
   
   ```
   2023/01/19 08:14:37 [error] 51#51: *458188 [lua] resolver.lua:47: 
parse_domain(): failed to parse domain: httpbin.org, error: failed to query the 
DNS server: dns server error: 5 refused, client: 172.19.0.1, server: _, 
request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   2023/01/19 08:14:37 [error] 51#51: *458188 [lua] init.lua:181: 
parse_domain_for_nodes(): dns resolver domain: httpbin.org error: failed to 
query the DNS server: dns server error: 5 refused, client: 172.19.0.1, server: 
_, request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   2023/01/19 08:14:37 [error] 51#51: *458188 [lua] init.lua:512: 
http_access_phase(): failed to set upstream: no valid upstream node, client: 
172.19.0.1, server: _, request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   2023/01/19 08:17:55 [error] 50#50: *480654 [lua] resolver.lua:47: 
parse_domain(): failed to parse domain: httpbin.org, error: failed to query the 
DNS server: dns server error: 5 refused, client: 172.19.0.1, server: _, 
request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   2023/01/19 08:17:55 [error] 50#50: *480654 [lua] init.lua:181: 
parse_domain_for_nodes(): dns resolver domain: httpbin.org error: failed to 
query the DNS server: dns server error: 5 refused, client: 172.19.0.1, server: 
_, request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   2023/01/19 08:17:55 [error] 50#50: *480654 [lua] init.lua:512: 
http_access_phase(): failed to set upstream: no valid upstream node, client: 
172.19.0.1, server: _, request: "GET /food/get HTTP/1.1", host: "localhost:9080"
   ```


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396600507

   I think maybe this is not deployed APISIX right? This should happen. Do you 
see any error log in APISiX


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396595345

   ``` json
   {
 "uri": "/food/*",
 "name": "food-route",
 "methods": [
   "GET",
   "POST",
   "PUT",
   "DELETE",
   "PATCH",
   "HEAD",
   "OPTIONS",
   "CONNECT",
   "TRACE"
 ],
 "plugins": {
   "proxy-rewrite": {
 "regex_uri": [
   "^/food/(.*)$",
   "/$1"
 ]
   }
 },
 "upstream_id": "443902377922134721",
 "status": 1
   }
   ```
   
   ``` json
   {
 "nodes": {
   "httpbin.org": 1
 },
 "timeout": {
   "connect": 6,
   "send": 6,
   "read": 6
 },
 "type": "roundrobin",
 "scheme": "http",
 "pass_host": "pass",
 "name": "test",
 "keepalive_pool": {
   "idle_timeout": 60,
   "requests": 1000,
   "size": 320
 }
   }
   ```
   
   **result**
   
![image](https://user-images.githubusercontent.com/21145779/213389783-4669f6eb-7d4a-4d54-9574-8752c2ee87ea.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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396587217

   This is so weird. Can you can upstream the server to `httpbin.org` and 
access `/food/get` the share what returns for me?


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-19 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396576223

   @mscb402 tried that
   
![image](https://user-images.githubusercontent.com/21145779/213386420-057f3135-8200-4be2-b47a-ec2417be7c7a.png)
   
   `
   {
 "uri": "/food/*",
 "name": "food-route",
 "methods": [
   "GET",
   "POST",
   "PUT",
   "DELETE",
   "PATCH",
   "HEAD",
   "OPTIONS",
   "CONNECT",
   "TRACE"
 ],
 "plugins": {
   "proxy-rewrite": {
 "regex_uri": [
   "^/food/(.*)$",
   "/$1"
 ]
   }
 },
 "upstream_id": "443897094827147969",
 "status": 1
   }`
   
   still getting 404
   
![image](https://user-images.githubusercontent.com/21145779/213386659-d41a36d5-a848-4e6f-b5ac-823e4388bfa7.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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396573733

   You should change Path `/*` to `/food/*` 


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396572945

   
![image](https://user-images.githubusercontent.com/21145779/213386026-568e7b52-b2fa-495b-aac6-4e249ecb1052.png)
   
![image](https://user-images.githubusercontent.com/21145779/213386058-49cc5224-3911-434c-bb4a-5e54f2bd6fa3.png)
   
![image](https://user-images.githubusercontent.com/21145779/213386082-c0b36ad0-3522-4aca-b348-4cfa9e8c0560.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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396571630

   Sorry, I can't view the image in GitHub. Can you post this image on GitHub 
issue page?


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396568620

   Jonathan,
   
   Tried that
   with
   [image: image.png]
   and
   [image: image.png]
   but 404
   [image: image.png]
   
   
   On Thu, Jan 19, 2023 at 3:46 PM Joanthan Chen ***@***.***>
   wrote:
   
   > I think you can try inputting ^/food/(.*)$ to regexp. And input /$1 to
   > the template.
   >
   > —
   > Reply to this email directly, view it on GitHub
   > ,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you authored the thread.Message ID:
   > ***@***.***>
   >
   


-- 
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] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396561442

   I think you can try inputting `^/food/(.*)$` to regexp. And input `/$1` to 
the template.


-- 
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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396544406

   Hi Jonathan,
   
   Thank you, I understand the regex value is ["^/food/(.*)$", "/$1"] but how
   about the template?
   
   On Thu, Jan 19, 2023 at 2:42 PM Joanthan Chen ***@***.***>
   wrote:
   
   > I see you are using the dashboard. You can set config in here
   > [image: image]
   > 

   >
   > —
   > Reply to this email directly, view it on GitHub
   > ,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you authored the thread.Message ID:
   > ***@***.***>
   >
   


-- 
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] Gallardot commented on pull request #1604: docs: update prowered-by.md

2023-01-18 Thread GitBox


Gallardot commented on PR #1604:
URL: 
https://github.com/apache/apisix-ingress-controller/pull/1604#issuecomment-1396513239

   Fake CI action. Need retry.


-- 
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] Gallardot commented on issue #1547: chore: improve make update-all

2023-01-18 Thread GitBox


Gallardot commented on issue #1547:
URL: 
https://github.com/apache/apisix-ingress-controller/issues/1547#issuecomment-1396511035

   Hi @tao12345666333 
   
   I think this issue has been fixed in this PR.  
[#1500](https://github.com/apache/apisix-ingress-controller/pull/1500/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R242)
   
   But I found another issue, that is, `make update-all` will execute 
`update-gofmt` to fix the imports issue, but the current github CI doesn't 
check this. So I create a PR to fix it. #1606 


-- 
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] codecov-commenter commented on pull request #2717: fix: adapt APISIX plugin disabled

2023-01-18 Thread GitBox


codecov-commenter commented on PR #2717:
URL: 
https://github.com/apache/apisix-dashboard/pull/2717#issuecomment-1396509313

   # 
[Codecov](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#2717](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (30a7a04) into 
[master](https://codecov.io/gh/apache/apisix-dashboard/commit/5ccb55be081303e54584f76333fb372eb7c33964?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (5ccb55b) will **decrease** coverage by `19.30%`.
   > The diff coverage is `100.00%`.
   
   ```diff
   @@ Coverage Diff @@
   ##   master#2717   +/-   ##
   ===
   - Coverage   74.27%   54.97%   -19.30% 
   ===
 Files 201  104   -97 
 Lines7833 2983 -4850 
 Branches  874  743  -131 
   ===
   - Hits 5818 1640 -4178 
   + Misses   1712 1343  -369 
   + Partials  3030  -303 
   ```
   
   | Flag | Coverage Δ | |
   |---|---|---|
   | backend-e2e-test-ginkgo | `?` | |
   | backend-unit-test | `?` | |
   | frontend-e2e-test | `54.97% <100.00%> (-21.43%)` | :arrow_down: |
   
   Flags with carried forward coverage won't be shown. [Click 
here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#carryforward-flags-in-the-pull-request-comment)
 to find out more.
   
   | [Impacted 
Files](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[web/src/components/Plugin/PluginPage.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1BsdWdpbi9QbHVnaW5QYWdlLnRzeA==)
 | `87.00% <100.00%> (-12.00%)` | :arrow_down: |
   | 
[...omponents/Upstream/components/ServiceDiscovery.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvU2VydmljZURpc2NvdmVyeS50c3g=)
 | `16.66% <0.00%> (-83.34%)` | :arrow_down: |
   | 
[web/src/components/LabelsfDrawer/LabelsDrawer.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL0xhYmVsc2ZEcmF3ZXIvTGFiZWxzRHJhd2VyLnRzeA==)
 | `3.92% <0.00%> (-80.40%)` | :arrow_down: |
   | 
[...nents/Upstream/components/ServiceDiscoveryArgs.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvU2VydmljZURpc2NvdmVyeUFyZ3MudHN4)
 | `20.00% <0.00%> (-80.00%)` | :arrow_down: |
   | 
[...b/src/pages/Route/components/DataLoader/Import.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9wYWdlcy9Sb3V0ZS9jb21wb25lbnRzL0RhdGFMb2FkZXIvSW1wb3J0LnRzeA==)
 | `4.91% <0.00%> (-78.69%)` | :arrow_down: |
   | 
[...mponents/Upstream/components/active-check/Host.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvYWN0aXZlLWNoZWNrL0hvc3QudHN4)
 | `33.33% <0.00%> (-66.67%)` | :arrow_down: |
   | 
[...mponents/Upstream/components/active-check/Port.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvYWN0aXZlLWNoZWNrL1BvcnQudHN4)
 | `33.33% <0.00%> (-66.67%)` | :arrow_down: |
   | 
[...mponents/Upstream/components/active-check/Type.tsx](https://codecov.io/gh/apache/apisix-dashboard/pull/2717?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-d2ViL3NyYy9jb21wb25lbnRzL1Vwc3RyZWFtL2NvbXBvbmVudHMvYWN0aXZlLWNoZWNrL1R5cGUudHN4)
 | `33.33% <0.00%> 

[GitHub] [apisix] mscb402 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mscb402 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396507031

   I see you are using the dashboard. You can set config in here
   
![image](https://user-images.githubusercontent.com/11229089/213373384-0531903f-0431-4cf2-98ff-7e7b6c9e9080.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-ingress-controller] codecov-commenter commented on pull request #1606: ci: add goimports-reviser

2023-01-18 Thread GitBox


codecov-commenter commented on PR #1606:
URL: 
https://github.com/apache/apisix-ingress-controller/pull/1606#issuecomment-1396504418

   # 
[Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1606?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#1606](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1606?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (0012ad5) into 
[master](https://codecov.io/gh/apache/apisix-ingress-controller/commit/31714eb4ae13c25ea0de1b2b7af0469f01938738?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (31714eb) will **not change** coverage.
   > The diff coverage is `n/a`.
   
   > :exclamation: Current head 0012ad5 differs from pull request most recent 
head 239a219. Consider uploading reports for the commit 239a219 to get more 
accurate results
   
   ```diff
   @@   Coverage Diff   @@
   ##   master#1606   +/-   ##
   ===
 Coverage   41.30%   41.30%   
   ===
 Files  87   87   
 Lines7460 7460   
   ===
 Hits 3081 3081   
 Misses   4026 4026   
 Partials  353  353   
   ```
   
   
   
   :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-ingress-controller] Gallardot opened a new pull request, #1606: ci: add goimports-reviser

2023-01-18 Thread GitBox


Gallardot opened a new pull request, #1606:
URL: https://github.com/apache/apisix-ingress-controller/pull/1606

   Signed-off-by: Gallardot 
   
   
   
   ### Type of change:
   
   
   
   - [ ] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   - [ ] Backport patches
   
   ### What this PR does / why we need it:
   
   
   
   ### Pre-submission 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](https://github.com/apache/apisix-ingress-controller#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-dashboard] Baoyuantop opened a new pull request, #2717: fix: adapt APISIX plugin disabled

2023-01-18 Thread GitBox


Baoyuantop opened a new pull request, #2717:
URL: https://github.com/apache/apisix-dashboard/pull/2717

   Please answer these questions before submitting a pull request, **or your PR 
will get closed**.
   
   **Why submit this pull request?**
   
   - [x] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   - [ ] Backport patches
   
   **What changes will this PR take into?**
   Reference https://github.com/apache/apisix/issues/7482. In the new version 
of APISIX, the plugin disabled move to _meta, and the dashboard needs to be 
changed synchronously.
   
   **Related issues**
   
   **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] mgonzaga1990 commented on issue #8709: help request: Setting up routing path

2023-01-18 Thread GitBox


mgonzaga1990 commented on issue #8709:
URL: https://github.com/apache/apisix/issues/8709#issuecomment-1396485978

   Hi Chen,
   
   I don't see the proxy-rewrite plugin, is there a way for me to include that
   in the list of plugins?
   
   On Thu, Jan 19, 2023 at 11:55 AM Joanthan Chen ***@***.***>
   wrote:
   
   > This is a example of how to config regex_uri in proxy_rewrite regex_uri
   > :["^/food/(.*)$", "/$1"]
   >
   > —
   > Reply to this email directly, view it on GitHub
   > ,
   > or unsubscribe
   > 

   > .
   > You are receiving this because you authored the thread.Message ID:
   > ***@***.***>
   >
   


-- 
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



  1   2   3   4   5   6   7   8   9   10   >