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

handle none ISO-8859-1 username #505

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

Conversation

PetersonZhao
Copy link

No description provided.

@PetersonZhao PetersonZhao deleted the BE-15445 branch September 2, 2024 09:17
@PetersonZhao PetersonZhao restored the BE-15445 branch September 2, 2024 09:18
@PetersonZhao PetersonZhao changed the title [BE-15445] - handle none ascii username handle none ISO-8859-1 username Sep 2, 2024
@PetersonZhao PetersonZhao reopened this Sep 2, 2024
@PetersonZhao
Copy link
Author

根据HTTP的规范,header中的Value必须是ISO-8859-1的字符,但是,username有可能会超出这个范围,所以,需要单独处理,用base64加密可以解决大部分问题,考虑到兼容以前的版本,所以多加了一个flag

@blackholll
Copy link
Owner

为什么会有这么特殊的username? 给个例子?

@PetersonZhao
Copy link
Author

为什么会有这么特殊的username? 给个例子?

https://en.wikipedia.org/wiki/ISO/IEC_8859-1

Languages with incomplete coverage,在实际使用过程中,会碰到一些欧洲人姓名

@blackholll
Copy link
Owner

这个更改的话 前端登录页也需要做相应修改, 你这边是前端自己写的,没用自带的前端么

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