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

fix: panic encoding alphabet includes duplicate symbols in go1.22 #470

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

poneding
Copy link

中文说明: 提交你的代码

解决使用 go1.22 版本编译 openrasp 程序运行报错问题。

#469

@CaledoniaProject
Copy link
Collaborator

CaledoniaProject commented Apr 12, 2024

base64 这个改的好像不太对,你把重复参数去了就行了。现在的改法,安全性有所降低

@poneding
Copy link
Author

poneding commented Apr 15, 2024

base64 这个改的好像不太对,你把重复参数去了就行了。现在的改法,安全性有所降低

不能直接去掉重复参数,因为参数必须要求是长度为 64 ,StdEncoding 就是不重复的参数,不知道你说的安全性降低是什么意思。
image

@CaledoniaProject
Copy link
Collaborator

嗯,原先代码是替换了编码用的字符串的,看起来你只需要去掉重复的字符串就行了

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