Skip to content

add nginx #38

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.

Sign up for GitHub

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

Jump to bottom
Merged
iBug merged 17 commits into from
Oct 18, 2025
Merged

add nginx #38

iBug merged 17 commits into from
Oct 18, 2025

Conversation

@Cherrling
Copy link
Contributor

@Cherrling Cherrling commented Dec 8, 2024

No description provided.

@taoky
Copy link
Member

taoky commented Dec 8, 2024

可以将 nginx 部分移动到「网络服务实践」下面吗？

@Cherrling
Copy link
Contributor Author

Cherrling commented Dec 8, 2024 •
edited
Loading

可以将 nginx 部分移动到「网络服务实践」下面吗？

LGTM

@Cherrling
Copy link
Contributor Author

Cherrling commented Dec 9, 2024 •
edited
Loading

可以将 nginx 部分移动到「网络服务实践」下面吗？

改到 网络服务实践 下面了， 但是这个 CI fail 我不太懂（？）

@iBug
Copy link
Member

iBug commented Dec 9, 2024

移动到「网络服务实践」应该指的是 docs/ops/network-service/ 下，再改一下 mkdocs.yml 里的位置。

CI fail 可以看 log，具体是 markdownlint 没过，可以考虑改成两个 bullet point，比如

- Date: ...
- Author: ...

以及我个人觉得日期也没必要写了，写一下 author 就行了。

@Cherrling
Copy link
Contributor Author

Cherrling commented Dec 9, 2024

移动到「网络服务实践」应该指的是 docs/ops/network-service/ 下，再改一下 mkdocs.yml 里的位置。

CI fail 可以看 log，具体是 markdownlint 没过，可以考虑改成两个 bullet point，比如

- Date: ...
- Author: ...

以及我个人觉得日期也没必要写了，写一下 author 就行了。

破案了，早上弄完因为网络原因没push上来

@taoky
Copy link
Member

taoky commented Dec 10, 2024

关于贡献者，有一个单独的页面标注（https://201.ustclug.org/spec/contributors/）。

Cherrling
Cherrling commented Dec 24, 2024
View reviewed changes
taoky
taoky previously requested changes Jan 1, 2025
View reviewed changes
docs/ops/network-service/nginx.md

作为 WebServer，必不可少的功能就是支持 HTTPS。你可以在 [https://cherr.cc/ssl.html](https://cherr.cc/ssl.html) 找到 SSL/TLS 的原理解释。

首先，你需要为你的域名申请一个 SSL 证书。你可以使用免费的 Let's Encrypt 证书，也可以购买商业证书。
Copy link
Member

@taoky taoky Jan 1, 2025

Choose a reason for hiding this comment

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

（要不把 acme.sh 也介绍一下

docs/ops/network-service/nginx.md
server_name alist.cherr.cc; # 指定的域名

location / {
proxy_pass http://localhost:5244; # 反向代理的地址
Copy link
Member

@taoky taoky Jan 1, 2025

Choose a reason for hiding this comment

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

可以顺便介绍一下用 include 避免重复的配置编写。

tiankaima
tiankaima reviewed Jan 2, 2025
View reviewed changes
tiankaima
tiankaima reviewed Jan 2, 2025
View reviewed changes
tiankaima
tiankaima reviewed Jan 2, 2025
View reviewed changes
@taoky taoky force-pushed the master branch 2 times, most recently from bee8306 to 92226b0 Compare April 6, 2025 07:01
@iBug iBug dismissed taoky’s stale review October 18, 2025 07:16

Expedite merging

@iBug iBug merged commit 215d8fc into ustclug:master Oct 18, 2025
3 checks passed
github-actions bot pushed a commit that referenced this pull request Oct 18, 2025
@web-flow
Auto deploy from GitHub Actions build 664
aeddd79 
白日梦主义: [215d8fc] add nginx (#38)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tiankaima tiankaima tiankaima left review comments

@taoky taoky taoky left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Cherrling @taoky @iBug @tiankaima