Skip to content
New issue

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

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

Already on GitHub? # 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

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

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants