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: 关闭控制台登录时请求的Authorization为null #12442

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

LHyphen
Copy link
Contributor

@LHyphen LHyphen commented Jul 31, 2024

What is the purpose of the change

关闭nacos控制台的登录,使用nginx的auth_basic限制控制台访问的时候,部分接口的请求Authorization会被置为null,导致auth_basic无法正常使用

Brief changelog

token不存在时Authorization置为undefined

Verifying this change

能正常用nginx的auth_basic了

@CLAassistant
Copy link

CLAassistant commented Jul 31, 2024

CLA assistant check
All committers have signed the CLA.

@KomachiSion KomachiSion merged commit cfa3312 into alibaba:develop Jul 31, 2024
7 checks passed
@KomachiSion KomachiSion added this to the 2.4.1 milestone Jul 31, 2024
@KomachiSion KomachiSion added the kind/enhancement Category issues or prs related to enhancement. label Jul 31, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
kind/enhancement Category issues or prs related to enhancement. plugin
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants