Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add weekly report 1031 #691

Merged
merged 14 commits into from
Nov 1, 2021

Conversation

lijing-21
Copy link
Contributor

add weekly report 1031

@lijing-21 lijing-21 marked this pull request as ready for review November 1, 2021 04:41

**问题描述**: 路由之间的限制计数器:需要在 lrucache 中指定路由对应的 limit-count 的 key,这样同一个limit对象就可以在多个路由中共享。lrucache 的key(以下称为 group,以区别于limit key)目前是自动生成的,确保每个路由的 group 是独立的。对于这一变化,我们需要能够在 limit-count 中指定组:

```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
```
```yaml


**问题描述**: 在模式中添加一个 request_body 开关,每个主体可以被 expr 用来决定是否记录。如果没有这个开关,主体就不会被记录。

```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
```
```json


测试 APISIX 中的 proxy-mirror 插件时,发现这个插件的特性与 nginx 中的 ngx_http_mirror_module 不同。 nginx 的镜像模块可以在 “proxy_pass” 指令中添加主机后面的 uri,例如:

```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add language type after "```"


Currently I has test the proxy-mirror plugin in apisix, but I find that the feature of this plugin is different from the ngx_http_mirror_module in nginx. The mirror moudle of nginx can add the uri behind the host in "proxy_pass" directive, for example:

```
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add language type after "```"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码格式化,开头加些空格

website/blog/2021/11/02/weekly-report-1031.md Outdated Show resolved Hide resolved
website/blog/2021/11/02/weekly-report-1031.md Outdated Show resolved Hide resolved
website/blog/2021/11/02/weekly-report-1031.md Outdated Show resolved Hide resolved

**链接**: https://github.com/apache/apisix-website/issues/686

**问题描述**:为了加快 Apache APISIX 官网图片的加载速度,需要将所有图片迁移到CDN。
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

迁移到 CDN

Comment on lines 43 to 45
**问题描述**:

测试 APISIX 中的 proxy-mirror 插件时,发现这个插件的特性与 nginx 中的 ngx_http_mirror_module 不同。 nginx 的镜像模块可以在 “proxy_pass” 指令中添加主机后面的 uri,例如:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

问题描述内容跟标题写到同一行吧,我看其他几条都是一起的,只有这条单独起了一行

lijing-21 and others added 5 commits November 1, 2021 13:46
Co-authored-by: yilinzeng <36651058+yzeng25@users.noreply.github.com>
Co-authored-by: yilinzeng <36651058+yzeng25@users.noreply.github.com>
Co-authored-by: yilinzeng <36651058+yzeng25@users.noreply.github.com>
- APISIX
- API Gateway
- Apache
description: If you want to go fast, go alone.If you want to go far, go together. The Apache APISIX Community Weekly Report hopes to help community members better understand the weekly progress of the Apache APISIX community and and facilitate everyone to participate in the Apache APISIX community.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you want to go fast, go alone.If you want to go far, go together. 这句话要不要从英文摘要里删除?直接写 weekly report 的介绍

}
```

N注意,同一 group 的配置需要相同,目前需要调用者保证,否则 group 获得的限制对象将与配置不同。
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多了一个字母 N

lijing-21 and others added 3 commits November 1, 2021 13:58
@juzhiyuan juzhiyuan merged commit c22151d into apache:master Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants