Skip to content

fix: openresty directory being overwritten#7014

Merged
zhengkunwang223 merged 1 commit into1Panel-dev:devfrom
luoyue712:patch-1
Mar 17, 2026
Merged

fix: openresty directory being overwritten#7014
zhengkunwang223 merged 1 commit into1Panel-dev:devfrom
luoyue712:patch-1

Conversation

@luoyue712
Copy link
Copy Markdown
Contributor

@luoyue712 luoyue712 commented Mar 16, 2026

你好,我需要编译openresty的ja4模块,它会对nginx进行补丁,因此我需要提前解压openresty源码并进行补丁,但是我补丁后发现后续会继续解压一次,导致了我之前的补丁无法生效。

参数:--add-module=/tmp/ja4-nginx-module/src
软件包:git,patch,curl
脚本:git clone https://github.com/FoxIO-LLC/ja4-nginx-module.git  /tmp/ja4-nginx-module && cd /tmp && tar xzf openresty-${RESTY_VERSION}.tar.gz && cd /tmp/openresty-${RESTY_VERSION}/bundle/nginx-1.27.1/ && patch -p1 < /tmp/ja4-nginx-module/patches/nginx.patch

代码经过本地验证过,已成功编译。

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot bot commented Mar 16, 2026

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@f2c-ci-robot
Copy link
Copy Markdown

f2c-ci-robot bot commented Mar 16, 2026

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@zhengkunwang223
Copy link
Copy Markdown
Member

👌

@zhengkunwang223 zhengkunwang223 merged commit 028761d into 1Panel-dev:dev Mar 17, 2026
1 check was pending
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants