Skip to content

Conversation

@beyondiary
Copy link

issue:
https://gitee.com/samwaf/SamWaf/issues/ICVLXQ
#438

The first issue on gitee was raised by me.

@welcome
Copy link

welcome bot commented Sep 1, 2025

Thanks for opening this pull request! Please check out our contributing guidelines.

@beyondiary
Copy link
Author

feat: fix ssl port forward error bug.
issue:
https://gitee.com/samwaf/SamWaf/issues/ICVLXQ
#438

The first issue on gitee was raised by me.

@samwafgo
Copy link
Owner

samwafgo commented Sep 2, 2025

非常感谢beyondirary 您得问题反馈和问题修复。

改成路径加载得话我们得证书续期时候加载可能不及时。

	// 使用新的重定向服务器
				redirectServer := &wafhttpserver.RedirectingHTTPSServer{
					Server: &http.Server{
						Addr:    ":" + strconv.Itoa(innruntime.Port),
						Handler: waf,
						TLSConfig: &tls.Config{
							GetCertificate: waf.GetCertificateFunc,
							MinVersion:     utils.ParseTLSVersion(global.GCONFIG_RECORD_SSLMinVerson),
							MaxVersion:     utils.ParseTLSVersion(global.GCONFIG_RECORD_SSLMaxVerson),
						},
					},
				}

							GetCertificate: waf.GetCertificateFunc,

您看下是不是这里 在某些情况下有问题呢?

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.

2 participants