[GitHub] [apisix-ingress-controller] codecov-commenter commented on pull request #1203: ci: Add timer verify k8s version
codecov-commenter commented on PR #1203: URL: https://github.com/apache/apisix-ingress-controller/pull/1203#issuecomment-1202094090 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1203?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report > Merging [#1203](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1203?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (68bcd9f) into [master](https://codecov.io/gh/apache/apisix-ingress-controller/commit/cccad72a1e0ef60525c69371b4b27c4598c587c1?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (cccad72) will **not change** coverage. > The diff coverage is `n/a`. > :exclamation: Current head 68bcd9f differs from pull request most recent head 06b579b. Consider uploading reports for the commit 06b579b to get more accurate results ```diff @@ Coverage Diff @@ ## master#1203 +/- ## === Coverage 29.55% 29.55% === Files 81 81 Lines 1016610166 === Hits 3005 3005 Misses 6834 6834 Partials 327 327 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] SylviaBABY commented on a diff in pull request #1256: docs:(4) update blog's description, keywords and tags
SylviaBABY commented on code in PR #1256: URL: https://github.com/apache/apisix-website/pull/1256#discussion_r935179538 ## blog/en/blog/2021/08/18/Auth-with-Casbin-in-Apache-APISIX.md: ## @@ -8,8 +8,8 @@ keywords: - Apache APISIX - Casbin - RBAC -description: When we are using Apache APISIX, we may want to add complex authorization logic to our application. In this article, we will use the built-in Casbin plugin (authz-casbin) of Apache APISIX to implement the role-based access control (RBAC) model. -tags: [Practical Case] +description: When we are using API Gateway Apache APISIX, we may need to add complex authorization logic. we can implement RBAC using APISIX's built-in Casbin plugin (authz-casbin). +tags: [Ecosystem, Plugins ] Review Comment: ```suggestion tags: [Ecosystem, Plugins] ``` -- 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] Sh1-5 commented on issue #2434: Feature request: support regex_uri attribute in redirect plugin
Sh1-5 commented on issue #2434: URL: https://github.com/apache/apisix-dashboard/issues/2434#issuecomment-1202088033 I'm completing it, should I change the Redirect from the select to a radio, or add a new option on the original basis? -- 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] AlinsRan opened a new pull request, #1203: ci: Add timer verify k8s version
AlinsRan opened a new pull request, #1203: URL: https://github.com/apache/apisix-ingress-controller/pull/1203 ### 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-docker] Chever-John commented on pull request #340: fix(grafana): fix issues #329
Chever-John commented on PR #340: URL: https://github.com/apache/apisix-docker/pull/340#issuecomment-1202084659 > I hope you can add a comment in the description of the PR, or in the configuration file, explaining the specific meaning of each line modification. Thanks Yes, I've updated the description. -- 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] Chever-John commented on a diff in pull request #340: fix(grafana): fix issues #329
Chever-John commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935170670 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: The source of this problem is from the APISIX Dashboard project. This is because it has updated the CSP (content security policy) feature. This caused the site to need to configure some security policy in order to display properly. What I have done here is to add this rule. In fact, only the last rule is really effective and solves this issues. At the same time, the Dashboard project itself has this problem and I have submitted a PR. You can refer to this [link](https://github.com/apache/apisix-dashboard/pull/2548). One more note, the Dashboard project's default configuration file has this optional configuration added, but it was problematic, so I submitted a [PR](https://github.com/apache/apisix-dashboard/pull/2548). You can see its default configuration options via this [link](https://github.com/apache/apisix-dashboard/blob/master/api/conf/conf.yaml#:~:text=%23-,security%3A,-%23%20%20%20access_control_allow_origin%3A) to see its default configuration options. -- 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] Chever-John commented on a diff in pull request #340: fix(grafana): fix issues #329
Chever-John commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935170670 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: The source of this problem is from the APISIX Dashboard project. This is because it has updated the CSP (content security policy) feature. This caused the site to need to configure some security policy in order to display properly. What I have done here is to add this rule. In fact, only the last rule is really effective and solves this issues. At the same time, the Dashboard project itself has this problem and I have submitted a PR. You can refer to this [link](https://github.com/apache/apisix-dashboard/pull/2548). One more note, the Dashboard project's default configuration file has this optional configuration added, but it was problematic, so I submitted a PR. You can see its default configuration options via this [link](https://github.com/apache/apisix-dashboard/blob/master/api/conf/ conf.yaml#:~:text=%23%20security%3A,%27unsafe%2Dinline%27%22%22) to see its default configuration options. -- 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] Chever-John commented on a diff in pull request #340: fix(grafana): fix issues #329
Chever-John commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935170670 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: The source of this problem is from the APISIX Dashboard project. This is because it has updated the CSP (content security policy) feature. This caused the site to need to configure some security policy in order to display properly. What I have done here is to add this rule. In fact, only the last rule is really effective and solves this issues. At the same time, the Dashboard project itself has this problem and I have submitted a PR. You can refer to this [link](https://github.com/apache/apisix-dashboard/pull/2548). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] liuning89757 opened a new issue, #7591: help request:
liuning89757 opened a new issue, #7591: URL: https://github.com/apache/apisix/issues/7591 ### Description Apisix support UDP proxy but seems cannot support UDP health check, according to https://apisix.apache.org/docs/apisix/health-check/ . I test it with DNS service, when stop DNS service on one backend apisix can not recognize the change. The [lua-resty-healthcheck] document also has nothing about UDP. Suggestions are greatly appreciated. ### Environment - APISIX version (run `apisix version`): 2.13 - Operating system (run `uname -a`): centos7.6 - OpenResty / Nginx version (run `openresty -V` or `nginx -V`): 1.19 -- 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] kingluo opened a new pull request, #7590: feat(ldap-auth): use lua-resty-ldap instead of lualdap
kingluo opened a new pull request, #7590: URL: https://github.com/apache/apisix/pull/7590 ### Description Use lua-resty-ldap to do nonblocking ldap auth. ### 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-website] netlify[bot] commented on pull request #1260: feat: shuffle picked posts
netlify[bot] commented on PR #1260: URL: https://github.com/apache/apisix-website/pull/1260#issuecomment-1202070240 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | ff20232fcb2afb3561fb4fc42907731c7387bddb | |🔍 Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62e8c39cbe6efc0008e9988a | -- 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, #1260: feat: shuffle picked posts
SkyeYoung opened a new pull request, #1260: URL: https://github.com/apache/apisix-website/pull/1260 Fixes: #[Add issue number here] Changes: Randomly insert selected posts starting from the second page of the blog listing page. 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-docker] tao12345666333 commented on a diff in pull request #340: fix(grafana): fix issues #329
tao12345666333 commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935160436 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: Since you only mentioned fixing #329 in the title and description, this still confuses 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-website] hf400159 opened a new pull request, #1259: docs: update blog
hf400159 opened a new pull request, #1259: URL: https://github.com/apache/apisix-website/pull/1259 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-website] netlify[bot] commented on pull request #1259: docs: update blog
netlify[bot] commented on PR #1259: URL: https://github.com/apache/apisix-website/pull/1259#issuecomment-1202062672 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | 75a4a73daccd5c052ac24484ccbc1355b1d5e90a | |🔍 Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62e8c0fdfa4a430009cd5073 | -- 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] Chever-John commented on a diff in pull request #340: fix(grafana): fix issues #329
Chever-John commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935152892 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: In fact, these are all optional configuration options, so why don't we comment them out? -- 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-java-plugin-runner] dargoner commented on issue #177: request help: 插件间的调用顺序或者优先级
dargoner commented on issue #177: URL: https://github.com/apache/apisix-java-plugin-runner/issues/177#issuecomment-1202059847 不好意思,我的代码错误 -- 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-java-plugin-runner] dargoner closed issue #177: request help: 插件间的调用顺序或者优先级
dargoner closed issue #177: request help: 插件间的调用顺序或者优先级 URL: https://github.com/apache/apisix-java-plugin-runner/issues/177 -- 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] tao12345666333 commented on a diff in pull request #340: fix(grafana): fix issues #329
tao12345666333 commented on code in PR #340: URL: https://github.com/apache/apisix-docker/pull/340#discussion_r935149016 ## example/apisix_conf/config.yaml: ## @@ -35,6 +35,13 @@ apisix: control: ip: "0.0.0.0" port: 9092 + security: +access_control_allow_origin: "http://httpbin.org"; Review Comment: why set it to `"http://httpbin.org"`? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander opened a new pull request, #7589: ci: make sure the correct version of click is installed
spacewander opened a new pull request, #7589: URL: https://github.com/apache/apisix/pull/7589 Address broken fuzzing CI like https://github.com/apache/apisix/runs/7625185369?check_suite_focus=true https://github.com/apache/apisix/runs/7625351307?check_suite_focus=true Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [ ] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: fix: search bar, posts order, style (#1258)
This is an automated email from the ASF dual-hosted git repository. guoqi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new 1c6b8aec976 fix: search bar, posts order, style (#1258) 1c6b8aec976 is described below commit 1c6b8aec9769679613540219d7621b984cfbf494 Author: Young AuthorDate: Tue Aug 2 13:53:17 2022 +0800 fix: search bar, posts order, style (#1258) --- blog/src/theme/BlogPosts/index.tsx| 19 ++- blog/src/theme/BlogPosts/style.module.scss| 2 +- blog/src/theme/SearchBar/index.js | 202 ++ blog/src/theme/SearchBar/styles.css | 22 +++ blog/src/theme/SearchBar/styles.module.css| 21 +++ doc/src/theme/SearchBar/index.js | 202 ++ doc/src/theme/SearchBar/styles.css| 22 +++ doc/src/theme/SearchBar/styles.module.css | 21 +++ website/src/theme/SearchBar/index.js | 202 ++ website/src/theme/SearchBar/styles.css| 22 +++ website/src/theme/SearchBar/styles.module.css | 21 +++ yarn.lock | 2 +- 12 files changed, 746 insertions(+), 12 deletions(-) diff --git a/blog/src/theme/BlogPosts/index.tsx b/blog/src/theme/BlogPosts/index.tsx index 91bd6abcf87..5316246c0e8 100644 --- a/blog/src/theme/BlogPosts/index.tsx +++ b/blog/src/theme/BlogPosts/index.tsx @@ -70,7 +70,12 @@ const BlogPostItem: FC = (props) => { const image = assets?.image ?? frontMatter.image ?? defaultImg; return ( -http://schema.org/BlogPosting";> +http://schema.org/BlogPosting"; +> = ({ useIntersectionObserver, ...props }) => { - let posts = items.map(({ content: BlogPostContent }) => ( + const posts = items.map(({ content: BlogPostContent }) => ( = ({ )); - const max = (pickedPosts.length > 10 ? pickedPosts.length - 10 : pickedPosts.length); - const endIdx = isFirstPage -? 2 * Math.floor(max / 2) -: 3 * Math.floor(max / 3); + const max = pickedPosts.length > 10 ? pickedPosts.length - 10 : pickedPosts.length; + const endIdx = isFirstPage ? 2 * Math.floor(max / 2) : 3; const { pathname } = useLocation(); if (!pathname.includes('/tags/')) { @@ -212,10 +215,6 @@ const BlogPosts: FC = ({ )), ); - -if (!isFirstPage) { - posts = shuffle(posts); -} } return ( diff --git a/blog/src/theme/BlogPosts/style.module.scss b/blog/src/theme/BlogPosts/style.module.scss index bf6563e3e2b..f2f0cca0122 100644 --- a/blog/src/theme/BlogPosts/style.module.scss +++ b/blog/src/theme/BlogPosts/style.module.scss @@ -64,7 +64,7 @@ font-family: apple-system, system-ui, sans-serif; article { -margin: 0 1.25rem 2.5rem; +margin: 0 1.25rem 3rem; border-radius: 1rem; overflow: hidden; border: 2px solid transparent; diff --git a/blog/src/theme/SearchBar/index.js b/blog/src/theme/SearchBar/index.js new file mode 100644 index 000..1c65093dee5 --- /dev/null +++ b/blog/src/theme/SearchBar/index.js @@ -0,0 +1,202 @@ +/* eslint-disable */ +/** + * Copyright (c) Facebook, Inc. and its affiliates. + * + * This source code is licensed under the MIT license found in the + * LICENSE file in the root directory of this source tree. + */ + +import React, { useState, useRef, useCallback, useMemo } from 'react'; +import { createPortal } from 'react-dom'; +import useDocusaurusContext from '@docusaurus/useDocusaurusContext'; +import { useHistory } from '@docusaurus/router'; +import { useBaseUrlUtils } from '@docusaurus/useBaseUrl'; +import Link from '@docusaurus/Link'; +import Head from '@docusaurus/Head'; +import useSearchQuery from '@theme/hooks/useSearchQuery'; +import { DocSearchButton, useDocSearchKeyboardEvents } from '@docsearch/react'; +import useAlgoliaContextualFacetFilters from '@theme/hooks/useAlgoliaContextualFacetFilters'; +import { translate } from '@docusaurus/Translate'; +import styles from './styles.module.css'; + +let DocSearchModal = null; + +function Hit({ hit, children }) { + return {children}; +} + +function ResultsFooter({ state, onClose }) { + const { generateSearchPageLink } = useSearchQuery(); + + return ( + + See all {state.context.nbHits} results + + ); +} + +function DocSearch({ contextualSearch, ...props }) { + const { siteMetadata } = useDocusaurusContext(); + + const contextualSearchFacetFilters = useAlgoliaContextualFacetFilters(); + + const configFacetFilters = props.searchParameters?.facetFilters ?? []; + + const facetFilters = contextualSearch +? // Merge contextual search filters with config filters + [...contextualSearchFacetFilters, ...configFacetFilters] +: // ... or use config facetFilters + configFacetFilters; + + // we let user override default searchParameters if he wants to + const searchParameters =
[GitHub] [apisix-website] guoqqqi merged pull request #1258: fix: search bar, posts order, style
guoqqqi merged PR #1258: URL: https://github.com/apache/apisix-website/pull/1258 -- 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] guoqqqi closed issue #1253: [Site]: search result throws 404 by default
guoqqqi closed issue #1253: [Site]: search result throws 404 by default URL: https://github.com/apache/apisix-website/issues/1253 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Error while running github feature from .asf.yaml in apisix!
An error occurred while running github feature in .asf.yaml!: 422 {"message": "Validation Failed", "errors": ["A repository cannot have more than 20 topics."], "documentation_url": "https://docs.github.com/rest/reference/repos#replace-all-repository-topics"}
[apisix] branch master updated: fix(redirect): compatible with old version data (#7584)
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git The following commit(s) were added to refs/heads/master by this push: new 639f9e0ec fix(redirect): compatible with old version data (#7584) 639f9e0ec is described below commit 639f9e0ec60d053e7977613366fef668e9945d0c Author: tzssangglass AuthorDate: Tue Aug 2 13:51:53 2022 +0800 fix(redirect): compatible with old version data (#7584) --- apisix/plugins/redirect.lua | 11 +++ t/plugin/redirect.t | 2 +- t/plugin/redirect2.t| 29 + 3 files changed, 37 insertions(+), 5 deletions(-) diff --git a/apisix/plugins/redirect.lua b/apisix/plugins/redirect.lua index 5c45b7946..d858b9c86 100644 --- a/apisix/plugins/redirect.lua +++ b/apisix/plugins/redirect.lua @@ -55,14 +55,12 @@ local schema = { }, http_to_https = {type = "boolean"}, encode_uri = {type = "boolean", default = false}, -append_query_string = {type = "boolean"}, +append_query_string = {type = "boolean", default = false}, }, oneOf = { {required = {"uri"}}, {required = {"regex_uri"}}, -{required = {"http_to_https"}, ["not"] = { - required = {"append_query_string"} -}} +{required = {"http_to_https"}} } } @@ -103,6 +101,7 @@ end function _M.check_schema(conf) local ok, err = core.schema.check(schema, conf) + if not ok then return false, err end @@ -117,6 +116,10 @@ function _M.check_schema(conf) end end +if conf.http_to_https and conf.append_query_string then +return false, "only one of `http_to_https` and `append_query_string` can be configured." +end + return true end diff --git a/t/plugin/redirect.t b/t/plugin/redirect.t index 246c73ce7..ec753f00b 100644 --- a/t/plugin/redirect.t +++ b/t/plugin/redirect.t @@ -1146,6 +1146,6 @@ Location: https://foo.com:9443/hello GET /t --- error_code: 400 --- response_body eval -qr/error_msg":"failed to check the configuration of plugin redirect err: value should match only one schema, but matches none/ +qr/error_msg":"failed to check the configuration of plugin redirect err: only one of `http_to_https` and `append_query_string` can be configured."/ --- no_error_log [error] diff --git a/t/plugin/redirect2.t b/t/plugin/redirect2.t index ac840e6d4..24f6f8ebf 100644 --- a/t/plugin/redirect2.t +++ b/t/plugin/redirect2.t @@ -79,3 +79,32 @@ GET /test/hello?o=apache --- response_headers Location: http://test.com/hello?q=apisix&o=apache --- error_code: 302 + + + +=== TEST 3: compatible with old version configuration +--- config +location /t { +content_by_lua_block { +local t = require("lib.test_admin").test +local code, body = t('/apisix/admin/routes/1', +ngx.HTTP_PUT, +[[{ +"uri": "/hello", +"host": "foo.com", +"plugins": { +"redirect": { +"http_to_https": true, +"append_query_string": false +} +} +}]] +) +if code >= 300 then +ngx.status = code +end +ngx.say(body) +} +} +--- response_body +passed
[GitHub] [apisix] spacewander closed issue #7583: bug: Incompatible redirect plugin parameters after upgrade to 2.15
spacewander closed issue #7583: bug: Incompatible redirect plugin parameters after upgrade to 2.15 URL: https://github.com/apache/apisix/issues/7583 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander merged pull request #7584: fix(redirect): compatible with old version data
spacewander merged PR #7584: URL: https://github.com/apache/apisix/pull/7584 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on pull request #7585: feat: V3 features from the next branch
spacewander commented on PR #7585: URL: https://github.com/apache/apisix/pull/7585#issuecomment-1202039592 > should we remove the branch check of `next` in ci yaml? > > I guess if the `next` branch has lost its mission, it can be removed. Yes, let's do 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-helm-chart] lzxuan commented on pull request #278: feat: add ServiceAccount to apisix
lzxuan commented on PR #278: URL: https://github.com/apache/apisix-helm-chart/pull/278#issuecomment-1202037776 @tokers I'll update the necessary RBAC resources to support Kubernetes service discovery. 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
Error while running github feature from .asf.yaml in apisix!
An error occurred while running github feature in .asf.yaml!: 422 {"message": "Validation Failed", "errors": ["A repository cannot have more than 20 topics."], "documentation_url": "https://docs.github.com/rest/reference/repos#replace-all-repository-topics"}
[apisix] branch master updated: chore: upgrade lua-resty-etcd to 1.8.3 (#7565)
This is an automated email from the ASF dual-hosted git repository. spacewander pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix.git The following commit(s) were added to refs/heads/master by this push: new 9749a9c13 chore: upgrade lua-resty-etcd to 1.8.3 (#7565) 9749a9c13 is described below commit 9749a9c1314cc19a194384ee2409b705dc7b5481 Author: tzssangglass AuthorDate: Tue Aug 2 13:34:26 2022 +0800 chore: upgrade lua-resty-etcd to 1.8.3 (#7565) --- rockspec/apisix-master-0.rockspec | 2 +- t/deployment/conf_server.t| 4 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/rockspec/apisix-master-0.rockspec b/rockspec/apisix-master-0.rockspec index bb0b39e29..e012abd72 100644 --- a/rockspec/apisix-master-0.rockspec +++ b/rockspec/apisix-master-0.rockspec @@ -34,7 +34,7 @@ dependencies = { "lua-resty-ctxdump = 0.1-0", "lua-resty-dns-client = 6.0.2", "lua-resty-template = 2.0", -"lua-resty-etcd = 1.8.2", +"lua-resty-etcd = 1.8.3", "api7-lua-resty-http = 0.2.0", "lua-resty-balancer = 0.04", "lua-resty-ngxvar = 0.5.2", diff --git a/t/deployment/conf_server.t b/t/deployment/conf_server.t index 7b06cf09d..4943c78e4 100644 --- a/t/deployment/conf_server.t +++ b/t/deployment/conf_server.t @@ -124,8 +124,6 @@ deployment: - http://[::1]:2379 --- error_log dns resolve localhost, result: no_error_log -[error] --- response_body foo @@ -236,8 +234,6 @@ OK --- error_log x.com is resolved to: 127.0.0.3 x.com is resolved to: 127.0.0.2 no_error_log -[error]
[GitHub] [apisix] spacewander merged pull request #7565: chore: upgrade lua-resty-etcd to 1.8.3
spacewander merged PR #7565: URL: https://github.com/apache/apisix/pull/7565 -- 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] lzxuan commented on pull request #278: feat: add ServiceAccount to apisix
lzxuan commented on PR #278: URL: https://github.com/apache/apisix-helm-chart/pull/278#issuecomment-1202034177 @tao12345666333 For the case of Kubernetes service discovery, yeah, ClusterRole and ClusterRoleBinding are also needed. If we are going into this use case specifically, I'll need to update the title of this PR together with the RBAC. Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-ingress-controller] AlinsRan closed pull request #1202: ci: add timer verify k8s version
AlinsRan closed pull request #1202: ci: add timer verify k8s version URL: https://github.com/apache/apisix-ingress-controller/pull/1202 -- 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 #1202: ci: add timer verify k8s version
codecov-commenter commented on PR #1202: URL: https://github.com/apache/apisix-ingress-controller/pull/1202#issuecomment-1202016987 # [Codecov](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1202?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report > Merging [#1202](https://codecov.io/gh/apache/apisix-ingress-controller/pull/1202?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (1311239) into [master](https://codecov.io/gh/apache/apisix-ingress-controller/commit/cccad72a1e0ef60525c69371b4b27c4598c587c1?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (cccad72) will **not change** coverage. > The diff coverage is `n/a`. > :exclamation: Current head 1311239 differs from pull request most recent head a953bcb. Consider uploading reports for the commit a953bcb to get more accurate results ```diff @@ Coverage Diff @@ ## master#1202 +/- ## === Coverage 29.55% 29.55% === Files 81 81 Lines 1016610166 === Hits 3005 3005 Misses 6834 6834 Partials 327 327 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_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] AlinsRan opened a new pull request, #1202: ci: add timer verify k8s version
AlinsRan opened a new pull request, #1202: URL: https://github.com/apache/apisix-ingress-controller/pull/1202 ### 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-website] netlify[bot] commented on pull request #1258: fix: search bar, posts order, style
netlify[bot] commented on PR #1258: URL: https://github.com/apache/apisix-website/pull/1258#issuecomment-1202013254 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | 852279301b7a949051b8bc6de193ad76d0efe5e9 | |🔍 Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62e8ac8f24fcff00092d782e | -- 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, #1258: fix: search bar, posts order, style
SkyeYoung opened a new pull request, #1258: URL: https://github.com/apache/apisix-website/pull/1258 Fixes: #1253 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-java-plugin-runner] dargoner opened a new issue, #178: request help: 插件间如何传递变量
dargoner opened a new issue, #178: URL: https://github.com/apache/apisix-java-plugin-runner/issues/178 ### Issue description 比如Java plugin第一个插件filter中设置新 Header,但是顺序执行的第二插件,filter取不到第一个插件设置的Header,请问如何在插件间如何传递变量 ### Environment * your apisix-java-plugin-runner version 0.2 -- 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-java-plugin-runner] dargoner opened a new issue, #177: request help: 插件间的调用顺序或者优先级
dargoner opened a new issue, #177: URL: https://github.com/apache/apisix-java-plugin-runner/issues/177 ### Issue description 在全局插件中设置,ext-plugin-pre-req,里面设置了3个自定义插件,但是实际调用的时候,发现插件的调用不是按照conf中插件的顺序来调用的,请问如何设置插件调用的优先级或者顺序 ### Environment * your apisix-java-plugin-runner version 0.2.0 -- 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] MalWalters commented on issue #7587: help request: limit-count plugin
MalWalters commented on issue #7587: URL: https://github.com/apache/apisix/issues/7587#issuecomment-1201988371 Thanks @tzssangglass , much appreciated. Do you know of any specific plugins that support the functionality required? -- 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] bzp2010 merged pull request #1249: docs: update blog description, keywords and tags
bzp2010 merged PR #1249: URL: https://github.com/apache/apisix-website/pull/1249 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated (6a372244bff -> 505b04866b1)
This is an automated email from the ASF dual-hosted git repository. bzp2010 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git from 6a372244bff docs: update blog description, keywords and tags (#1249) add 505b04866b1 docs: (3) update blog description, keywords and tags (#1252) No new revisions were added by this update. Summary of changes: blog/en/blog/2021/07/06/celebrate-200-contributors.md | 6 +++--- ...stomization-of-Sina-Weibo-API-gateway-based-on-Apache-APISIX.md | 4 ++-- blog/en/blog/2021/07/21/Apache-APISIX-Kubernetes.md| 7 --- blog/en/blog/2021/07/25/apachecon-asia.md | 4 ++-- .../2021/07/27/use-of-plugin-orchestration-in-Apache-APISIX.md | 2 +- blog/en/blog/2021/07/28/release-apache-apisix-2.8.md | 5 +++-- ...stomization-of-Sina-Weibo-API-gateway-based-on-Apache-APISIX.md | 4 ++-- blog/zh/blog/2021/07/21/Apache-APISIX-Kubernetes.md| 4 ++-- blog/zh/blog/2021/07/25/apachecon-asia.md | 7 --- .../2021/07/27/use-of-plugin-orchestration-in-Apache-APISIX.md | 6 ++ blog/zh/blog/2021/07/28/release-apache-apisix-2.8.md | 5 +++-- 11 files changed, 28 insertions(+), 26 deletions(-)
[GitHub] [apisix-website] bzp2010 merged pull request #1252: docs: (3) update blog description, keywords and tags
bzp2010 merged PR #1252: URL: https://github.com/apache/apisix-website/pull/1252 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated (982d85912ad -> 6a372244bff)
This is an automated email from the ASF dual-hosted git repository. bzp2010 pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git from 982d85912ad docs: add `why-we-need-apache-apisix` blog (#1250) add 6a372244bff docs: update blog description, keywords and tags (#1249) No new revisions were added by this update. Summary of changes: ...Dashboard-Access-Control-Bypass-Vulnerability-Announcement.md | 2 +- ...-version-v1.0-of-Apache-APISIX-Ingress-Controller-released.md | 4 ++-- .../blog/2021/06/21/use-Java-to-write-Apache-APISIX-plugins.md | 4 ++-- ...ploy-great-open-source-gateway-and-ingress-controller-fast.md | 5 ++--- blog/en/blog/2021/06/29/release-apache-apisix-2.7.md | 8 +++- blog/en/blog/2021/06/30/etcd3-support-HTTP-access-perfectly.md | 4 ++-- ...Dashboard-Access-Control-Bypass-Vulnerability-Announcement.md | 2 +- ...-version-v1.0-of-Apache-APISIX-Ingress-Controller-released.md | 4 ++-- .../blog/2021/06/21/use-Java-to-write-Apache-APISIX-plugins.md | 8 ...ploy-great-open-source-gateway-and-ingress-controller-fast.md | 4 ++-- blog/zh/blog/2021/06/29/release-apache-apisix-2.7.md | 9 +++-- blog/zh/blog/2021/06/30/etcd3-support-HTTP-access-perfectly.md | 4 ++-- 12 files changed, 26 insertions(+), 32 deletions(-)
[GitHub] [apisix-website] SkyeYoung commented on a diff in pull request #1244: docs: update Blog description, tags and keywords
SkyeYoung commented on code in PR #1244: URL: https://github.com/apache/apisix-website/pull/1244#discussion_r935087281 ## blog/en/blog/2020/12/11/beike-how-to-build-gateway-based-on-apache-apisix.md: ## @@ -7,8 +7,8 @@ keywords: - Apache APISIX - Kong - Nginx -description: Beike uses Apache APISIX as the API gateway of the production system, processing over 100 million production traffic every day, with excellent performance and stability. It happens that Apache APISIX has just joined the Apache incubator. In addition to congratulations, I would like to share why I chose Apache APISIX in the first place and some experiences in the process of using it. -tags: [User Case] +description: Beike chose to use the API gateway Apache APISIX as the API gateway, which handles hundreds of millions of traffic every day with excellent and stable performance. Review Comment: ```suggestion description: Beike chose to use the Apache APISIX as the API gateway, which handles hundreds of millions of traffic every day with excellent and stable performance. ``` Maybe better? -- 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] Chever-John opened a new pull request, #340: fix(grafana): fix issues #329
Chever-John opened a new pull request, #340: URL: https://github.com/apache/apisix-docker/pull/340 As title says. -- 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 commented on a diff in pull request #1254: docs: Update 2022-01 blogs tags/keywords/decs
SylviaBABY commented on code in PR #1254: URL: https://github.com/apache/apisix-website/pull/1254#discussion_r935082357 ## blog/en/blog/2022/01/13/how-to-proxy-dubbo-in-apache-apisix.md: ## @@ -13,12 +13,12 @@ keywords: - Apache APISIX - Apache Dubbo - Proxy -- Plugin -description: In this article, we introduced how to use Apache APISIX to implement a proxy for Dubbo Service. By introducing the dubbo-proxy plugin, you can build a simpler and more efficient traffic link for the back-end system of Dubbo framework. -tags: [Technology] +- API Gateway +description: This article describes how to use APISIX to implement a proxy for Dubbo. You can use dubbo-proxy plugin to build backend system of Dubbo framework. +tags: [Ecosystem,Plugin] Review Comment: ```suggestion tags: [Ecosystem,Plugins] ``` ## blog/en/blog/2022/01/26/apisix-integrate-forward-auth-plugin.md: ## @@ -11,11 +11,11 @@ authors: image_url: "https://avatars.githubusercontent.com/u/39793568?v=4"; keywords: - Apache APISIX +- API Gateway - Authentication -- Plugin - Ecosystem -description: This article describes the use of `forward-auth`, a new plugin in Apache APISIX, and provides detailed instructions on how to use this cleanly designed authentication model. -tags: [Technology,Ecosystem,Authentication] +description: This article describes `forward-auth`, a new plugin in APISIX, and show the details on how to use this cleanly designed authentication model. +tags: [Authentication,Plugin] Review Comment: ```suggestion tags: [Authentication,Plugins] ``` ## blog/zh/blog/2022/01/13/how-to-proxy-dubbo-in-apache-apisix.md: ## @@ -12,13 +12,13 @@ authors: keywords: - Apache APISIX - Apache Dubbo -- 代理操作 -- 插件 -description: 本文为大家介绍了如何借助 Apache APISIX 实现 Dubbo Service 的代理,通过引入 dubbo-proxy 插件便可为 Dubbo 框架的后端系统构建更简单更高效的流量链路。 -tags: [Technology] +- 代理 +- API Gateway +description: 本文为介绍了如何借助 Apache APISIX 实现 Dubbo Service 的代理,通过引入 dubbo-proxy 插件便可为 Dubbo 框架的后端系统构建更简单更高效的流量链路。 +tags: [Ecosystem,Plugin] Review Comment: ```suggestion tags: [Ecosystem,Plugins] ``` ## blog/zh/blog/2022/01/26/apisix-integrate-forward-auth-plugin.md: ## @@ -11,11 +11,11 @@ authors: image_url: "https://avatars.githubusercontent.com/u/39793568?v=4"; keywords: - Apache APISIX +- API Gateway - 认证 -- 插件 - 生态 description: 本文将介绍 Apache APISIX 中新增插件 `forward-auth` 的使用方法,为大家详细说明如何使用这款设计简洁的认证模型。 -tags: [Technology,Ecosystem,Authentication] +tags: [Authentication,Plugin] Review Comment: ```suggestion tags: [Authentication,Plugins] ``` ## blog/en/blog/2022/01/17/apisix-kafka-integration.md: ## @@ -15,11 +15,11 @@ keywords: - Kafka - logging - monitoring -description: Apache APISIX has been providing support for Apache Kafka since version 1.2 with the kafka-logger plugin release. kafka-logger has been enhanced several times since then to provide very mature and complete functionality. It supports pushing API request logs, request bodies, and response bodies, to a Kafka cluster in JSON format. -tags: [Technology] +description: This article describes how to use the kafka-logger plugin with APISIX. Wiht enhancements, the plugin now has mature and complete functions. +tags: [Ecosystem,Plugin] Review Comment: ```suggestion tags: [Ecosystem,Plugins] ``` ## blog/en/blog/2022/01/25/apisix-grpc-web-integration.md: ## @@ -12,10 +12,10 @@ authors: keywords: - Apache APISIX - gRPC -- gRPC-Web +- API Gateway - Ecology -description: Apache APISIX already supports gRPC protocol proxies, as well as HTTP(s) to gRPC Server proxies via the gRPC Transcode plugin. Through active community discussion and contributions, Apache APISIX has broadened the scope of support for the gRPC ecosystem, support for the gRPC Web Protocol Request Proxy. -tags: [Technology,Ecosystem] +description: APISIX has broadened the scope of support for the gRPC ecosystem. This article will introduce details of using the gRPC web protocol request proxy. +tags: [Ecosystem,Plugin] Review Comment: ```suggestion tags: [Ecosystem,Plugins] ``` ## blog/zh/blog/2022/01/25/apisix-grpc-web-integration.md: ## @@ -12,10 +12,10 @@ authors: keywords: - Apache APISIX - gRPC -- gRPC-Web -- Ecology -description: Apache APISIX 已经支持了 gRPC 协议代理,以及通过 gRPC Transcode 插件支持了 HTTP(s) 到 gRPC Server 的代理。通过社区的积极讨论和贡献,Apache APISIX 又拓宽了 gRPC 生态的支持范围:支持 gRPC Web 协议请求代理。 -tags: [Technology,Ecosystem] +- API Gateway +- Ecosystem +description: 通过社区的积极讨论和贡献,Apache APISIX 又拓宽了 gRPC 生态的支持范围。本文将介绍 gRPC Web 协议请求代理的具体使用步骤及细节。 +tags: [Ecosystem,Plugin] Review Comment: ```suggestion tags: [Ecosystem,Plugins] ``` ## blog/zh/blog/2022/01/17/apisix-kafka-integration.md: ## @@ -15,8 +15,8 @@ keywords: - Kafka - 日志 - 监控 -description: Apache APISIX 早在 1.2 版本开始就已经提供了 kafka-logger 插件的支持,其后又经过多次功能强化,目前已具备非常成熟且完善的
[GitHub] [apisix-docker] Chever-John commented on issue #329: apisix配置grafana地址
Chever-John commented on issue #329: URL: https://github.com/apache/apisix-docker/issues/329#issuecomment-1201971928 Hi, I will solve 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-website] netlify[bot] commented on pull request #1257: docs: Update 2022-456-month tag/keyword/decs
netlify[bot] commented on PR #1257: URL: https://github.com/apache/apisix-website/pull/1257#issuecomment-1201942395 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | fa44c3ffbb9383b837a2df7a9f94ae02a39fef9a | |🔍 Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62e88b30df03bd0008465252 | -- 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 opened a new pull request, #1257: docs: Update 2022-456-month tag/keyword/decs
SylviaBABY opened a new pull request, #1257: URL: https://github.com/apache/apisix-website/pull/1257 Changes: Update 2022 4/5/6 blogs' tag/keyword/decs -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
tzssangglass commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1201934734 > but the rest of the split access-log will also occupy the cache, unless which logs are manually deleted we can use `max_kept` in `log-rotate` plugins to delete old logs files. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on pull request #7565: chore: upgrade lua-resty-etcd to 1.8.3
tzssangglass commented on PR #7565: URL: https://github.com/apache/apisix/pull/7565#issuecomment-1201924906 This PR will need to be placed in the 2.15.1 LTS release in the future. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on a diff in pull request #7584: fix(redirect): compatible with old version data
tzssangglass commented on code in PR #7584: URL: https://github.com/apache/apisix/pull/7584#discussion_r935045048 ## t/plugin/redirect2.t: ## @@ -79,3 +79,32 @@ GET /test/hello?o=apache --- response_headers Location: http://test.com/hello?q=apisix&o=apache --- error_code: 302 + + + +=== TEST 3: compatible with old version configuration Review Comment: Already have, ref:https://github.com/apache/apisix/pull/7584/files#diff-97c7ac2f91e3c7a47c9832ebc576be52e9d69af74dde165200128e0f056531b6R1120-R1149 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] wolgod commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
wolgod commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1201923924 > > Now it seems that it should be caused by writing the accesslog locally. Delete the accesslog log and the cache will be reduced. > > You can see the size of the `access.log`. Logrotate splits the log, one per hour, but the access-log is only about 2g, but the rest of the split access-log will also occupy the cache, unless which logs are manually deleted -- 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] tokers commented on a diff in pull request #7584: fix(redirect): compatible with old version data
tokers commented on code in PR #7584: URL: https://github.com/apache/apisix/pull/7584#discussion_r935043035 ## t/plugin/redirect2.t: ## @@ -79,3 +79,32 @@ GET /test/hello?o=apache --- response_headers Location: http://test.com/hello?q=apisix&o=apache --- error_code: 302 + + + +=== TEST 3: compatible with old version configuration Review Comment: Also test the case that `http_to_https` is `true` and `append_query_string` is also `true`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-website] yzeng25 merged pull request #1250: docs: add `why-we-need-apache-apisix` blog
yzeng25 merged PR #1250: URL: https://github.com/apache/apisix-website/pull/1250 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: docs: add `why-we-need-apache-apisix` blog (#1250)
This is an automated email from the ASF dual-hosted git repository. yilinzeng pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new 982d85912ad docs: add `why-we-need-apache-apisix` blog (#1250) 982d85912ad is described below commit 982d85912ad8c74ac3d3eaf2173c17d222d7097d Author: Fei Han <97138894+hf400...@users.noreply.github.com> AuthorDate: Tue Aug 2 09:39:05 2022 +0800 docs: add `why-we-need-apache-apisix` blog (#1250) --- .../blog/2021/06/28/why-we-need-Apache-APISIX.md | 2 +- .../blog/2021/06/28/why-we-need-Apache-APISIX.md | 4 +- .../blog/2022/07/29/why-we-need-apache-apisix.md | 109 + 3 files changed, 112 insertions(+), 3 deletions(-) diff --git a/blog/en/blog/2021/06/28/why-we-need-Apache-APISIX.md b/blog/en/blog/2021/06/28/why-we-need-Apache-APISIX.md index 44593cdbbea..8f761afe7e6 100644 --- a/blog/en/blog/2021/06/28/why-we-need-Apache-APISIX.md +++ b/blog/en/blog/2021/06/28/why-we-need-Apache-APISIX.md @@ -12,7 +12,7 @@ keywords: - Open Source - API Management description: This article describes the history of the open source API Gateway Apache APISIX architecture's evolution and compares the advantages of the two frameworks, Apache APISIX and Nginx. -tags: [Technology] +tags: [Ecosystem] --- > This article describes the history of the open source API Gateway Apache > APISIX architecture's evolution and compares the advantages of the two > frameworks, Apache APISIX and Nginx. diff --git a/blog/zh/blog/2021/06/28/why-we-need-Apache-APISIX.md b/blog/zh/blog/2021/06/28/why-we-need-Apache-APISIX.md index 3e50a6d39fd..a4638b1a199 100644 --- a/blog/zh/blog/2021/06/28/why-we-need-Apache-APISIX.md +++ b/blog/zh/blog/2021/06/28/why-we-need-Apache-APISIX.md @@ -10,8 +10,8 @@ keywords: - Nginx - API 网关 - 开源 -description: 本文介绍了 Apache APISIX 架构演进历史,对比 Nginx 和 Kong 这两个框架 Apache APISIX 的优势是什么。 -tags: [Technology] +description: 本文介绍了云原生 API 网关 Apache APISIX 架构演进历史,对比 Nginx 和 Kong 这两个框架 Apache APISIX 的优势是什么,并且介绍 APISIX 未来的趋势。 +tags: [Ecosystem] --- > 本文介绍了 Apache APISIX 架构演进历史,对比 Nginx 和 Kong 这两个框架 Apache APISIX 的优势是什么。 diff --git a/blog/zh/blog/2022/07/29/why-we-need-apache-apisix.md b/blog/zh/blog/2022/07/29/why-we-need-apache-apisix.md new file mode 100644 index 000..44fd491cbc9 --- /dev/null +++ b/blog/zh/blog/2022/07/29/why-we-need-apache-apisix.md @@ -0,0 +1,109 @@ +--- +title: "有了 NGINX 和 Kong,为什么还需要 Apache APISIX?" +author: "韩飞" +authorURL: "https://github.com/hf400159"; +authorImageURL: "https://github.com/hf400159.png"; +keywords: +- Apache APISIX +- API 网关 +- NGINX +- Kong +- 开源网关 +description: 本文介绍了云原生 API 网关 Apache APISIX 的特点,以及 Kong 和 NGINX 的存在的痛点,并介绍了 Apache APISIX 如何解决 NGINX 和 Kong 所带来的业务痛点。 +tags: [Case Studies] +--- + +> 本文介绍了 Kong 和 NGINX 的痛点,以及 Apache APISIX 如何解决它们的痛点,并且为你介绍了 Apache APISIX 的应用场景。 + + + +云原生时代,动态和可观测性成为衡量 API 网关的标准之一。Apache APISIX 自诞生之初就一直跟随着云原生的脚步前行。然而 Apache APISIX 作为一个诞生刚刚三年的新一代网关,为什么可以从诞生 20 多年的 NGINX 与开源 8 年的 Kong 中突出重围,成为云原生时代最流行以及最活跃的网关?我认为其中最重要的原因是解决了开发者和企业在使用 NGINX 和 Kong 中的痛点。 + +## NGINX 与 Kong 的痛点 + +在单体服务时代,使用 NGINX 可以应对大多数的场景,而到了云原生时代,NGINX 因为其自身架构的原因则会出现两个问题: + +- 首先是 NGINX 不支持集群管理。几乎每家互联网厂商都有自己的 NGINX 配置管理系统,系统虽然大同小异但是一直没有统一的方案。 +- 其次是 NGINX 不支持配置的热加载。很多公司一旦修改了配置,重新加载 NGINX 的时间可能需要半个小时以上。并且在 Kubernetes 体系下,上游会经常发生变化,如果使用 NGINX 来处理就需要频繁重启服务,这对于企业是不可接受的。 + +而 Kong 的出现则解决了 NGINX 的痛点,但是又带来了新的问题: + +- Kong 需要依赖于 PostgreSQL 或 Cassandra 数据库,这使 Kong 的整个架构非常臃肿,并且会给企业带来高可用的问题。如果数据库故障了,那么整个 API 网关都会出现故障。 +- Kong 的路由使用的是遍历查找,当网关内有超过上千个路由时,它的性能就会出现比较急剧的下降。 + +而 APISIX 的出现则解决了上述所有问题,成为了云原生时代最完美的 API 网关。那么 Apache APISIX 的优势到底是什么?为什么可以在短短三年的时间里成为全世界最活跃的网关? + +## APISIX 的优势 + +### 优异的架构 + +首先 Apache APISIX 拥有优异的架构,现在很多应用都在向微服务、容器化迁移,形成新的云原生时代。云原生作为当下的技术潮流,将重写传统企业的技术架构。而 APISIX 自诞生之初就跟随技术潮流,并将其设计为云原生架构: + +![APISIX](https://static.apiseven.com/2022/blog/0729/1.png) + +如上图所示,左右分别是 APISIX 的数据面(Data Plane)和控制面(Control Plane): + +- 数据面:以 NGINX 的网络库为基础(未使用 NGINX 的路由匹配、静态配置和 C 模块),使用Lua 和 NGINX 动态控制请求流量; +- 控制面:使用 etcd 来存储和同步网关的配置数据,管理员通过 Admin API 或者 Dashboard 可以在毫秒级别内通知到所有数据面节点。 + +在更新数据上,Kong 采用轮询数据库的方式,但是可能需要 5-10 秒才能获取到最新的配置;而 APISIX 则采用监听 etcd 的配置变更的方式,可以将时间控制在毫秒级,达到实时生效的效果。 +而且由于 APISIX 和 etcd 均支持多点部署,因此在 APISIX 当前架构中,任何一个服务出现异常宕机等事故,都不会影响 APISIX 正常对外提供服务的能力。 + +### 完善的生态 + +下图为 APISIX 的生态图,从该图可以准确看到 APISIX 已经支持的 7 层协议有 HTTP(S)、HTTP2、Dubbo、QUIC 和物联网协议 MQTT 等等,4 层协议有 TCP/UDP。 +右侧部分则是一些开源或者 SaaS 服务,比如 SkyWalking、Prometheus 、Vault 等等。而最下面则是比较常见的操作系统环境、云厂商和硬件环境。而作为一个开源软件,APISIX 也支持在 ARM64 的服务器上运行。 + +![APISIX's Ecosystem](https://static.apiseven.com/2022/blog/0729/2.PNG) + +APISIX 不仅支持众多协议与操作系统,而且也支持多语言编程插件。APISIX 诞生之初仅支持使用 Lua 语言编写插件,这种情况下就需要开发者掌握 Lua 和 NGINX 相关的技术栈。然而 Lua 和 NGINX 属于相对小众的技术,开发者很少。因此我们在 APISIX 上支持了多语言开发插件,目前已经正式支持了 Java、Golang、Node.js、Python 等语言。 + +![programming language](https://static.apiseven.com/2022/blog/0729/3.png) + +### 活跃的社区 + +下图是贡献者增长
[GitHub] [apisix-ingress-controller] github-actions[bot] commented on issue #945: request help: How to rewrite http to https in the apisix ingress controller
github-actions[bot] commented on issue #945: URL: https://github.com/apache/apisix-ingress-controller/issues/945#issuecomment-1201913120 This issue has been marked as stale due to 90 days of inactivity. It will be closed in 30 days if no further activity occurs. If this issue is still 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] tzssangglass commented on issue #7587: help request: limit-count plugin
tzssangglass commented on issue #7587: URL: https://github.com/apache/apisix/issues/7587#issuecomment-1201909168 > Does it make this distinction per customer? We can set the `key` to be `$consumer_name` to distinguish the customer, e.g.: ``` "plugins": { "limit-count": { "count": 2, "time_window": 60, "rejected_code": 503, "key": "$consumer_name" } }, ``` for more details, ref: https://github.com/apache/apisix/blob/master/docs/en/latest/plugins/limit-count.md#attributes `X-API-Key` is used for the admin API of APISIX. It is not related to customs and should not be used to count requests generated by customs. > Is there a way that we can expose the usage count for the client via a GET call so they can assess and budget their usage? None. But you can add this functionality based on open source plugins. -- 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 commented on issue #1245: [Docs]: Need thumbnail pictures for the blogs
SkyeYoung commented on issue #1245: URL: https://github.com/apache/apisix-website/issues/1245#issuecomment-1201902371 Before we can do that, we need to do a few more things. update the project documentation 2. test and adjust how the thumbnail pictures should appear in the blog -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander opened a new pull request, #7588: test: check if the connection reuse works well
spacewander opened a new pull request, #7588: URL: https://github.com/apache/apisix/pull/7588 It can detect the bug fixed in 7b6b30f4aa84e7d91e2a62280c27b5fab3fce46a Signed-off-by: spacewander ### Description Fixes # (issue) ### Checklist - [x] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new features added to this PR - [x] 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 issue #1245: [Docs]: Need thumbnail pictures for the blogs
yzeng25 commented on issue #1245: URL: https://github.com/apache/apisix-website/issues/1245#issuecomment-1201897816 > Please assign me this issue. What kind of picture would you like to add to other blogs? My first thought is: - Each blog post has at least one picture in its content. - Grab the most suitable picture from the content and make it the cover. - Or generate a cover picture on your own Due to the recent refactorization, I am not sure how it works. You would need some consultations from @SkyeYoung . -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[apisix-website] branch master updated: feat: featured blog posts (#1255)
This is an automated email from the ASF dual-hosted git repository. guoqi pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new 7b92b094dfc feat: featured blog posts (#1255) 7b92b094dfc is described below commit 7b92b094dfc0e1e41e61c178acf0140e79cedab5 Author: Young AuthorDate: Tue Aug 2 08:57:15 2022 +0800 feat: featured blog posts (#1255) --- .gitignore | 2 + blog/en/config/picked-posts.json | 5 + blog/i18n/zh/code.json | 4 + blog/package.json | 1 + blog/src/theme/BlogPosts/index.tsx | 99 ++-- blog/src/theme/BlogPosts/style.module.scss | 156 +-- blog/zh/config/picked-posts.json | 5 + package.json | 3 +- scripts/generate-picked-posts-info.mjs | 142 ++ scripts/package.json | 10 +- yarn.lock | 695 - 11 files changed, 1043 insertions(+), 79 deletions(-) diff --git a/.gitignore b/.gitignore index 74567297845..15c62d28128 100644 --- a/.gitignore +++ b/.gitignore @@ -17,6 +17,8 @@ lib/core/MetadataBlog.js /blog/zh/i18n /blog/en/src /blog/en/static +/blog/en/config/picked-posts-info.js +/blog/zh/config/picked-posts-info.js # misc diff --git a/blog/en/config/picked-posts.json b/blog/en/config/picked-posts.json new file mode 100644 index 000..a54ab83cce2 --- /dev/null +++ b/blog/en/config/picked-posts.json @@ -0,0 +1,5 @@ +[ +"blog/en/blog/2022/07/29/release-apache-apisix-2.15.md", +"blog/en/blog/2022/07/22/how-is-google-cloud-tau-t2a-performing.md", +"blog/en/blog/2022/07/06/use-keycloak-with-api-gateway-to-secure-apis.md" +] diff --git a/blog/i18n/zh/code.json b/blog/i18n/zh/code.json index fd16a8e39a8..359a916cb9f 100644 --- a/blog/i18n/zh/code.json +++ b/blog/i18n/zh/code.json @@ -336,5 +336,9 @@ }, "theme.blog.post.readingTime.plurals": { "message": "阅读需约 {readingTime} 分钟" + }, + "blog.picked.posts.component.title": { +"message": "精选", +"description": "Picked posts" } } diff --git a/blog/package.json b/blog/package.json index 5e301e3d109..0f5a29e8089 100644 --- a/blog/package.json +++ b/blog/package.json @@ -40,6 +40,7 @@ "fitty": "^2.3.6", "gsap": "^3.7.1", "lethargy": "^1.0.9", +"lodash.shuffle": "^4.2.0", "raw-loader": "^4.0.2", "rc-image": "^5.6.2", "react": "^17.0.2", diff --git a/blog/src/theme/BlogPosts/index.tsx b/blog/src/theme/BlogPosts/index.tsx index c95e947a0b1..91bd6abcf87 100644 --- a/blog/src/theme/BlogPosts/index.tsx +++ b/blog/src/theme/BlogPosts/index.tsx @@ -10,6 +10,14 @@ import clsx from 'clsx'; import type { FC, HTMLAttributes, DetailedHTMLProps } from 'react'; import React from 'react'; import useWindowType from '@theme/hooks/useWindowSize'; +import shuffle from 'lodash.shuffle'; +import { useLocation } from '@docusaurus/router'; +import { translate } from '@docusaurus/Translate'; + +// pickedPosts will be auto generated +// eslint-disable-next-line import/no-unresolved +import pickedPosts from '../../../config/picked-posts-info'; + import 'react-lazy-load-image-component/src/effects/blur.css'; import style from './style.module.scss'; @@ -45,6 +53,7 @@ const BlogPostItem: FC = (props) => { delayMethod, delayTime, useIntersectionObserver, +className, } = props; const delayProps = { scrollPosition, @@ -58,10 +67,10 @@ const BlogPostItem: FC = (props) => { const windowType = useWindowType(); const effect = windowType === 'mobile' ? 'opacity' : 'blur'; - const image = assets.image ?? frontMatter.image ?? defaultImg; + const image = assets?.image ?? frontMatter.image ?? defaultImg; return ( -http://schema.org/BlogPosting";> +http://schema.org/BlogPosting";> = ({ delayTime, useIntersectionObserver, ...props -}) => ( - -{items.map(({ content: BlogPostContent }) => ( - - - -))} - -); +}) => { + let posts = items.map(({ content: BlogPostContent }) => ( + + + + )); + + const max = (pickedPosts.length > 10 ? pickedPosts.length - 10 : pickedPosts.length); + const endIdx = isFirstPage +? 2 * Math.floor(max / 2) +: 3 * Math.floor(max / 3); + const { pathname } = useLocation(); + + if (!pathname.includes('/tags/')) { +posts.splice( + 1, + 0, + (isFirstPage ? pickedPosts : shuffle(pickedPosts)).slice(0, endIdx).map((info) => ( + + +{translate({ + id: 'blog.picked.posts.component.title', + message: 'Featured', +})} + + {info.summary} + + )), +); + +if (!isFirstPage) { + posts = shuffle(posts); +} + } + + return ( + + {posts} + + ); +}
[GitHub] [apisix-website] guoqqqi merged pull request #1255: feat: featured blog posts
guoqqqi merged PR #1255: URL: https://github.com/apache/apisix-website/pull/1255 -- 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] MalWalters opened a new issue, #7587: help request: limit-count plugin
MalWalters opened a new issue, #7587: URL: https://github.com/apache/apisix/issues/7587 ### Description Hi, we have a use case where we would like to use APISix to provide a prepaid way to access our API. e.g. a client would subscribe for 12 months or 1,000,000 API calls and it looks like the `limit-count` plugin supports this. I have two questions though. Does it make this distinction per customer? We are anticipate using the X-API-Key for authentication and would be ideal if we could group by this. Is there a way that we can expose the usage count for the client via a GET call so they can assess and budget their usage? Thanks in advance ### 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] FangSen9000 commented on issue #2562: test: adding route batch deleted function test cases
FangSen9000 commented on issue #2562: URL: https://github.com/apache/apisix-dashboard/issues/2562#issuecomment-1201319415 My god, is this finished? amazing! @zfkube0x1 -- 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] adarrssh commented on issue #1245: [Docs]: Need thumbnail pictures for the blogs
adarrssh commented on issue #1245: URL: https://github.com/apache/apisix-website/issues/1245#issuecomment-1201314337 Please assign me this issue. What kind of picture would you like to add to other blogs? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] FangSen9000 commented on issue #2562: test: adding route batch deleted function test cases
FangSen9000 commented on issue #2562: URL: https://github.com/apache/apisix-dashboard/issues/2562#issuecomment-1201303392 > Hi @FangSen9000 [FangSen9000#3](https://github.com/FangSen9000/apisix-dashboard/pull/3) I think you have opened a pr. do you want me to submit it on that or do you want to write 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] hf400159 opened a new pull request, #7586: docs: refactor skywalking-logger plugin docs
hf400159 opened a new pull request, #7586: URL: https://github.com/apache/apisix/pull/7586 ### Description Fixes # (issue) ### Checklist - [ ] I have explained the need for this PR and the problem it solves - [ ] I have explained the changes or the new features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [ ] I have verified that this change is backward compatible (If not, please discuss on the [APISIX mailing list](https://github.com/apache/apisix/tree/master#community) first) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix-dashboard] FangSen9000 commented on issue #2562: test: adding route batch deleted function test cases
FangSen9000 commented on issue #2562: URL: https://github.com/apache/apisix-dashboard/issues/2562#issuecomment-1201298266 > Hi @FangSen9000 [FangSen9000#3](https://github.com/FangSen9000/apisix-dashboard/pull/3) Hi, what's going on -- 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] zfkube0x1 commented on issue #2562: test: adding route batch deleted function test cases
zfkube0x1 commented on issue #2562: URL: https://github.com/apache/apisix-dashboard/issues/2562#issuecomment-1201294695 Hi @FangSen9000 https://github.com/FangSen9000/apisix-dashboard/pull/3 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #7571: help request: Apisix 404 route not found And Apisix背后的etcd 报查询时间长
tzssangglass commented on issue #7571: URL: https://github.com/apache/apisix/issues/7571#issuecomment-1201224115 Excluding the ETCD problem (3.4.18 is OK), I think it may be the `server-info` plugin, upgrade APISIX to the 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] tzssangglass commented on issue #7571: help request: Apisix 404 route not found And Apisix背后的etcd 报查询时间长
tzssangglass commented on issue #7571: URL: https://github.com/apache/apisix/issues/7571#issuecomment-1201221274 Can you find this route in etcd (not via 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] tzssangglass commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
tzssangglass commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1201219469 > Now it seems that it should be caused by writing the accesslog locally. Delete the accesslog log and the cache will be reduced. You can see the size of the `access.log`. -- 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] shuqingzai commented on issue #2554: import openapi.yaml 无法识别 x-apisix-service_id
shuqingzai commented on issue #2554: URL: https://github.com/apache/apisix-dashboard/issues/2554#issuecomment-1201182092 > 您好,请查看新版本的Open API 导入。 什么意思?新版本?最新版的不是 `v2.13` 吗? -- 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] ccxhwmy commented on a diff in pull request #7529: feat(grafana json): update grafana json for nginx shared dict in prometheus
ccxhwmy commented on code in PR #7529: URL: https://github.com/apache/apisix/pull/7529#discussion_r934511071 ## docs/assets/other/json/apisix-grafana-dashboard.json: ## @@ -1622,6 +1622,111 @@ "timeShift": null, "title": "Nginx metric errors", "type": "stat" +}, +{ + "aliasColors": {}, + "bars": false, + "dashLength": 10, + "dashes": false, + "datasource": "${DS_PROMETHEUS}", + "description": "The free space percent of each nginx shared DICT since APISIX start", + "fieldConfig": { +"defaults": { + "custom": {}, + "links": [] +}, +"overrides": [] + }, + "fill": 1, + "fillGradient": 0, + "gridPos": { +"h": 8, +"w": 24, +"x": 0, +"y": 57 + }, + "hiddenSeries": false, + "id": 35, + "legend": { +"alignAsTable": false, +"avg": false, +"current": false, +"max": false, +"min": false, +"rightSide": false, +"show": true, +"total": false, +"values": false + }, + "lines": true, + "linewidth": 1, + "nullPointMode": "null", + "options": { +"alertThreshold": true + }, + "percentage": false, + "pluginVersion": "7.3.7", + "pointradius": 2, + "points": false, + "renderer": "flot", + "seriesOverrides": [], + "spaceLength": 10, + "stack": false, + "steppedLine": false, + "targets": [ +{ + "expr": "(apisix_shared_dict_free_space_bytes * 100) / on (name) apisix_shared_dict_capacity_bytes", Review Comment: > If you are worried about showing instances will make the graph too complex, maybe we can add a comment that people can modify this query to show per-instance graph? Does you mean add a comment like `# we can modify the expr to show per-instance graph` in json file? -- 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] zfkube0x1 commented on issue #2554: import openapi.yaml 无法识别 x-apisix-service_id
zfkube0x1 commented on issue #2554: URL: https://github.com/apache/apisix-dashboard/issues/2554#issuecomment-1201166365 您好,请查看新版本的Open API 导入。 -- 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] yangboyd opened a new pull request, #2564: upstream discovery allow server name pass from request host
yangboyd opened a new pull request, #2564: URL: https://github.com/apache/apisix-dashboard/pull/2564 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?** upstream discovery allow server name pass from request host add service_name_type field with value: pass, rewrite. The original service_name field is used as rewrite service name. Please update this section with detailed description. **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] tzssangglass commented on pull request #7585: feat: V3 features from the next branch
tzssangglass commented on PR #7585: URL: https://github.com/apache/apisix/pull/7585#issuecomment-1201087207 should we remove the branch check of `next` in ci yaml? I guess if the `next` branch has lost its mission, it can be removed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass opened a new pull request, #7585: feat: V3 features from the next branch
tzssangglass opened a new pull request, #7585: URL: https://github.com/apache/apisix/pull/7585 ### Description Fixes #7320 ### 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] wolgod commented on issue #7578: help request: high cache usage when using Kafka-logger, Prometheus plugins
wolgod commented on issue #7578: URL: https://github.com/apache/apisix/issues/7578#issuecomment-1201072510 Now it seems that it should be caused by writing the accesslog locally. Delete the accesslog log and the cache will be reduced. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass opened a new pull request, #7584: fix(redirect): compatible with old version data
tzssangglass opened a new pull request, #7584: URL: https://github.com/apache/apisix/pull/7584 ### Description Fixes #7583 ### Checklist - [x] I have explained the need for this PR and the problem it solves - [x] I have explained the changes or the new features added to this PR - [ ] I have added tests corresponding to this change - [ ] I have updated the documentation to reflect this change - [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] Chever-John commented on issue #1201: bug: Discuss how to face the docker compose update
Chever-John commented on issue #1201: URL: https://github.com/apache/apisix-ingress-controller/issues/1201#issuecomment-1200994585 @AlinsRan Maybe I didn't express myself clearly. What I meant was that the latest version of docker compose executes the command `docker compose` instead of `docker-compose`. So do we have to improve the makefile to accommodate this change. Or does anyone have a better suggestion? -- 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 #4829: request help: kafka-logger batch-processor error
github-actions[bot] commented on issue #4829: URL: https://github.com/apache/apisix/issues/4829#issuecomment-1200989811 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] github-actions[bot] commented on issue #4796: request help: Whether the active health check is not perfect?
github-actions[bot] commented on issue #4796: URL: https://github.com/apache/apisix/issues/4796#issuecomment-1200989780 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] github-actions[bot] commented on issue #4794: bug: APISIX v2.7 doesn't support S3(Object Storage API) signature_v4
github-actions[bot] commented on issue #4794: URL: https://github.com/apache/apisix/issues/4794#issuecomment-1200989736 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] hf400159 commented on a diff in pull request #1250: docs: add `why-we-need-apache-apisix` blog
hf400159 commented on code in PR #1250: URL: https://github.com/apache/apisix-website/pull/1250#discussion_r934353325 ## blog/zh/blog/2022/07/29/why-we-need-apache-apisix.md: ## @@ -0,0 +1,109 @@ +--- +title: "有了 NGINX 和 Kong,为什么还需要 Apache APISIX?" +author: "韩飞" +authorURL: "https://github.com/hf400159"; +authorImageURL: "https://github.com/hf400159.png"; Review Comment: This is a new article. -- 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] tokers commented on issue #7573: help request: whether apisix supports changing `:authority` in the grpc protocol
tokers commented on issue #7573: URL: https://github.com/apache/apisix/issues/7573#issuecomment-1200980181 > > Or just using the `upstream_host` field in Upstream object. > > This should only change the `host`, not the `:authority` ? `: authority` should be identical with `host`, just try `host`. -- 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 #1256: docs:(4) update blog's description, keywords and tags
netlify[bot] commented on PR #1256: URL: https://github.com/apache/apisix-website/pull/1256#issuecomment-1200974095 ### 👷 Deploy Preview for *apache-apisix* processing. | Name | Link | |-|| |🔨 Latest commit | 9197f2905cc7182c2f37482d17ae36f0a178ff1d | |🔍 Latest deploy log | https://app.netlify.com/sites/apache-apisix/deploys/62e7a1b1b6504900098e0079 | -- 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] hf400159 opened a new pull request, #1256: docs:(4) update blog's description, keywords and tags
hf400159 opened a new pull request, #1256: URL: https://github.com/apache/apisix-website/pull/1256 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] hf400159 commented on a diff in pull request #7581: docs: add websocket rate limit example
hf400159 commented on code in PR #7581: URL: https://github.com/apache/apisix/pull/7581#discussion_r934227000 ## docs/en/latest/plugins/limit-conn.md: ## @@ -141,3 +141,79 @@ curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f13 } }' ``` + +## Limit the rate of WebSocket connections + +Apache APISIX supports WebSocket proxy, we can use `limit-conn` plugin to limit the number of concurrent WebSocket connections. + +1. Start WebSocket Server + +shell +docker run -p 1980:8080 --name websocket-demo casperklein/websocket-demo + + +2. Register the route, enable the WebSocket proxy on the route and enable the limit-conn plugin + +shell +curl http://127.0.0.1:9080/apisix/admin/routes/1 -H 'X-API-KEY: edd1c9f034335f136f87ad84b625c8f1' -X PUT -d ' +{ +"uri": "/ws", +"enable_websocket":true, +"plugins": { +"limit-conn": { +"conn": 1, +"burst": 0, +"default_conn_delay": 0.1, +"rejected_code": 503, +"key_type": "var", +"key": "remote_addr" +} +}, +"upstream": { +"type": "roundrobin", +"nodes": { +"127.0.0.1:1980": 1 +} +} +}' + + +The above route enables the WebSocket proxy on `/ws`, and limits the number of concurrent WebSocket connections to 1. More than 1 concurrent WebSocket connection will return 503 to reject the request. + +3. Initiate a WebSocket request, and the link is established successfully + +shell +curl --include \ + --no-buffer \ + --header "Connection: Upgrade" \ + --header "Upgrade: websocket" \ + --header "Sec-WebSocket-Key: x3JJHMbDL1EzLkh9GBhXDw==" \ + --header "Sec-WebSocket-Version: 13" \ + --http1.1 \ + http://127.0.0.1:9080/ws + +HTTP/1.1 101 Switching Protocols +Connection: upgrade +Upgrade: websocket +Sec-WebSocket-Accept: HSmrc0sMlYUkAGmm5OPpG2HaGWk= +Server: APISIX/2.15.0 + + +4. Initiate the WebSocket request again in another terminal, the request will be rejected + +shell +HTTP/1.1 503 Service Temporarily Unavailable +Date: Mon, 01 Aug 2022 03:49:17 GMT +Content-Type: text/html; charset=utf-8 +Content-Length: 194 +Connection: keep-alive +Server: APISIX/2.15.0 + + +503 Service Temporarily Unavailable + +503 Service Temporarily Unavailable +openresty + + Review Comment: ditto -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] spacewander commented on issue #7583: bug: Incompatible redirect plugin parameters after upgrade to 2.15
spacewander commented on issue #7583: URL: https://github.com/apache/apisix/issues/7583#issuecomment-1200966849 I think the fix is incorrect. ``` "redirect": { "http_to_https": true, "append_query_string": true } ``` should be rejected. But ``` "redirect":{ "http_to_https":true, "append_query_string":false } ``` should be OK. -- 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] zuiyangqingzhou commented on issue #7573: help request: whether apisix supports changing `:authority` in the grpc protocol
zuiyangqingzhou commented on issue #7573: URL: https://github.com/apache/apisix/issues/7573#issuecomment-1200951545 > Or just using the `upstream_host` field in Upstream object. This should only change the `host`, not the `:authority` ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@apisix.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[GitHub] [apisix] tzssangglass commented on issue #7583: bug: Incompatible redirect plugin parameters after upgrade to 2.15
tzssangglass commented on issue #7583: URL: https://github.com/apache/apisix/issues/7583#issuecomment-1200950360 @crazyMonkey1995, thank you for your feedback. in 2.14.1, if we config `redirect` plugin like: ``` "plugins": { "redirect": { "http_to_https": true } } ``` it will be stored in etcd as: ``` "plugins":{ "redirect":{ "http_to_https":true, "encode_uri":false, "append_query_string":false, "ret_code":302 } }, ``` if we upgrade APISIX to 2.15.0, would get error like: ``` config_etcd.lua:243: load_full_data(): failed to check item data of [/apisix/routes] err:failed to check the configuration of plugin redirect err: value should match only one schema, but matches none ,val: {\"host\":\"foo.com\",\"plugins\":{\"redirect\":{\"http_to_https\":true,\"encode_uri\":false,\"append_query_string\":false,\"ret_code\":302}},\"status\":1,\"uri\":\"\/hello\",\"id\":\"1\",\"priority\":0,\"create_time\":1659345713,\"update_time\":1659345713}, context: init_worker_by_lua*" (req 0) # 2022/08/01 17:22:29 [info] 6059\#6059: *2 [lua] plugin.lua:746: check_single_plugin_schema(): check plugin schema, name: redirect, configurations: {"append_query_string":false,"ret_code":302,"http_to_https":true,"encode_uri":false}, context: init_worker_by_lua* ``` -- 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] zuiyangqingzhou commented on issue #7573: help request: whether apisix supports changing `:authority` in the grpc protocol
zuiyangqingzhou commented on issue #7573: URL: https://github.com/apache/apisix/issues/7573#issuecomment-1200945786 > Change the pseudo `:authority` should be same to change the host header. Try to use `proxy-rewrite` plugin. proxy-rewrite plugin does not work,the following is the OpenResty documentation https://user-images.githubusercontent.com/17425270/182117023-2779a020-8b33-4dfc-b00b-98964d5e8baa.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
[apisix-website] branch master updated: feat: Blog tags header (#1251)
This is an automated email from the ASF dual-hosted git repository. juzhiyuan pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/apisix-website.git The following commit(s) were added to refs/heads/master by this push: new 7f2e6d957b1 feat: Blog tags header (#1251) 7f2e6d957b1 is described below commit 7f2e6d957b171b16ea4336fdcfee9f0d13851eab Author: Young AuthorDate: Mon Aug 1 17:10:47 2022 +0800 feat: Blog tags header (#1251) --- blog/package.json | 1 + blog/src/css/customTheme.css| 5 +++ blog/src/theme/BlogLayout/index.tsx | 46 ++-- blog/src/theme/BlogLayout/style.module.scss | 44 +++ yarn.lock | 47 ++--- 5 files changed, 136 insertions(+), 7 deletions(-) diff --git a/blog/package.json b/blog/package.json index 20f3c4c4ffe..5e301e3d109 100644 --- a/blog/package.json +++ b/blog/package.json @@ -47,6 +47,7 @@ "react-lazy-load-image-component": "^1.5.4", "react-share": "^4.4.0", "react-spring": "^9.4.5", +"react-stickynode": "^4.1.0", "react-transition-group": "^4.4.1", "react-use": "^17.4.0", "reading-time": "^1.5.0", diff --git a/blog/src/css/customTheme.css b/blog/src/css/customTheme.css index c24abdaaa80..7d9335d8dc4 100644 --- a/blog/src/css/customTheme.css +++ b/blog/src/css/customTheme.css @@ -165,6 +165,11 @@ header h2 { color: var(--color-primary); } +.navbar { + box-shadow: none; + border-bottom: 1px solid #e8e8ed; +} + .navbar__title { font-size: 18px; height: 30px; diff --git a/blog/src/theme/BlogLayout/index.tsx b/blog/src/theme/BlogLayout/index.tsx index 1020d6433d2..bb02c042147 100644 --- a/blog/src/theme/BlogLayout/index.tsx +++ b/blog/src/theme/BlogLayout/index.tsx @@ -7,8 +7,8 @@ * LICENSE file in the root directory of this source tree. */ +import type { FC } from 'react'; import React from 'react'; -import clsx from 'clsx'; import Layout from '@theme/Layout'; import BlogSidebar from '@theme/BlogSidebar'; import TOC from '@theme/TOC'; @@ -22,6 +22,9 @@ import { } from 'react-share'; import useWindowType from '@theme/hooks/useWindowSize'; import type { Props } from '@theme/BlogLayout'; +import Link from '@docusaurus/Link'; +import Sticky from 'react-stickynode'; +import clsx from 'clsx'; import style from './style.module.scss'; const Share = ({ metadata }) => { @@ -45,7 +48,44 @@ const Share = ({ metadata }) => { ); }; -const BlogLayout = (props: Props): JSX.Element => { +const tags = [ + { +label: 'Community', +url: '/blog/tags/community/', + }, + { +label: 'Case Studies', +url: '/blog/tags/user-case/', + }, + { +label: 'Ecosystem', +url: '/blog/tags/ecosystem/', + }, + { +label: 'Authentication', +url: '/blog/tags/authentication/', + }, + { +label: 'Security', +url: '/blog/tags/security/', + }, +]; + +const TagsHeader: FC = () => ( + +{(s) => ( + +{tags.map((tag) => ( + +{tag.label} + +))} + +)} + +); + +const BlogLayout: FC = (props) => { const { sidebar, toc, children, metadata, ...layoutProps } = props; @@ -54,6 +94,8 @@ const BlogLayout = (props: Props): JSX.Element => { return ( + + {hasSidebar && ( diff --git a/blog/src/theme/BlogLayout/style.module.scss b/blog/src/theme/BlogLayout/style.module.scss index 71ca2f3c582..cdaea46f53c 100644 --- a/blog/src/theme/BlogLayout/style.module.scss +++ b/blog/src/theme/BlogLayout/style.module.scss @@ -14,3 +14,47 @@ height: fit-content; margin-left: 1rem; } + +.tagsHeader { + display: flex; + justify-content: flex-end; + padding: 0.25rem 1rem; + transition: all 0.5s ease-in-out; + border-bottom: 1px solid #e8e8ed; + font-family: apple-system, system-ui, sans-serif; + + &::before { +content: ""; +width: 100%; +height: 100%; +z-index: -1; +position: absolute; +top: 0; +backdrop-filter: blur(12px); + } + + > a { +display: inline-block; +padding: 0 0.5rem; +font-weight: 400; +font-size: 14px; +transition: inherit; + +&:default { + color: #1d1d1f; +} + } + + &.expand { +padding: 1rem; +box-shadow: none; + +&::before { + background-color: hsl(100deg 100% 100% / 85%); +} + +> a { + padding: 0 1rem; +} + } +} diff --git a/yarn.lock b/yarn.lock index 5d7b9c67aad..332e5602940 100644 --- a/yarn.lock +++ b/yarn.lock @@ -3972,7 +3972,7 @@ class-utils@^0.3.5: isobject "^3.0.0" static-extend "^0.1.1" -classnames@^2.2.1, classnames@^2.2.5, classnames@^2.2.6: +classnames@^2.0.0, classnames@^2.2.1, classnames@^2.2.5, classnames@^2.2.6: version "2.3.1" resolved "https://registry.yarnpkg.com/classnames/-/classnames-2.3.1.tgz#dfcfa3891e306ec1dad105d0e88f4417b8535e8e"; integrit
[GitHub] [apisix-website] juzhiyuan merged pull request #1251: feat: Blog tags header
juzhiyuan merged PR #1251: URL: https://github.com/apache/apisix-website/pull/1251 -- 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] AlinsRan commented on issue #1201: bug: Discuss how to face the docker compose update
AlinsRan commented on issue #1201: URL: https://github.com/apache/apisix-ingress-controller/issues/1201#issuecomment-1200923977 You have not installed docker-compose. -- 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 commented on a diff in pull request #1249: docs: update blog description, keywords and tags
SylviaBABY commented on code in PR #1249: URL: https://github.com/apache/apisix-website/pull/1249#discussion_r934294431 ## blog/zh/blog/2021/06/29/release-apache-apisix-2.7.md: ## @@ -6,16 +6,13 @@ authorImageURL: "https://avatars.githubusercontent.com/u/4161644?v=4"; keywords: - Apache APISIX - Release Notes -Description: Apache APISIX 2.7.0-Release 正式发布! -tags: [Release] +Description: 云原生 API 网关 Apache APISIX 2.7.0-Release 正式发布!该版本支持了多语言插件、增强了四层 TCP 代理以及 NGINX 功能。 Review Comment: ```suggestion Description: 云原生 API 网关 Apache APISIX 2.7.0 版本正式发布!该版本支持了多语言插件、增强了四层 TCP 代理以及 NGINX 功能。 ``` ## blog/zh/blog/2021/06/29/release-apache-apisix-2.7.md: ## @@ -6,16 +6,13 @@ authorImageURL: "https://avatars.githubusercontent.com/u/4161644?v=4"; keywords: - Apache APISIX - Release Notes -Description: Apache APISIX 2.7.0-Release 正式发布! -tags: [Release] +Description: 云原生 API 网关 Apache APISIX 2.7.0-Release 正式发布!该版本支持了多语言插件、增强了四层 TCP 代理以及 NGINX 功能。 +tags: [Community] --- -> Apache APISIX 2.7.0-Release 正式发布! - +> Apache APISIX 2.7.0-Release 正式发布!欢迎大家下载使用。 Review Comment: ```suggestion > Apache APISIX 2.7.0 版本正式发布!欢迎大家下载使用。 ``` ## blog/en/blog/2021/06/21/use-Java-to-write-Apache-APISIX-plugins.md: ## @@ -9,8 +9,8 @@ keywords: - Apache APISIX - Java - plugin -description: Apache APISIX now supports writing plugins in Java! You can now write plugins in a programming language you are familiar with. -tags: [Practical Case] +description: The cloud native API Gateway Apache APISIX already supports Java to write plug-ins. Users can not only write plug-ins in Java language, but also integrate into the Spring Cloud ecosystem. Review Comment: ```suggestion description: The cloud native API Gateway Apache APISIX already supports Java to write plugins. Users can not only write plugins in Java language, but also integrate into the Spring Cloud ecosystem. ``` ## blog/en/blog/2021/06/29/release-apache-apisix-2.7.md: ## @@ -6,16 +6,14 @@ authorImageURL: "https://avatars.githubusercontent.com/u/23514812?v=4"; keywords: - Apache APISIX - Release Notes -Description: Apache APISIX 2.7.0 has been released! -tags: [Release] +Description: API Gateway Apache APISIX 2.7.0-Release is officially released! This release supports multilingual plug-ins, enhanced four-layer TCP proxy, and NGINX functionality. Review Comment: ```suggestion Description: API Gateway Apache APISIX 2.7.0 is officially released! This release supports multilingual plugins, enhanced four-layer TCP proxy, and NGINX functionality. ``` ## blog/en/blog/2021/06/23/deploy-great-open-source-gateway-and-ingress-controller-fast.md: ## @@ -8,9 +8,8 @@ keywords: - Rancher - Helm Chart - API Gateway -description: This article shows the procedure of installing and deploying Apache APISIX and Apache APISIX Ingress Controller in the Rancher App Store (Catalog), and how to proxy Kubernetes services through them. - -tags: [Practical Case] +description: Through the cloud-native API gateway Apache APISIX Helm repository, you can directly deploy Apache APISIX and APISIX Ingress controller in Rancher. +tags: [Tutorials,Ecosystem ] Review Comment: ```suggestion tags: [Tutorials,Ecosystem] ``` -- 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