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 -在开启认证的情况下增加token解析认证处理方式 #10297

Closed
wants to merge 4,563 commits into from

Conversation

MentosL
Copy link
Contributor

@MentosL MentosL commented Apr 10, 2023

ref : #10169 #9109 #8107 #10176

Please do not create a Pull Request without creating an issue first.

What is the purpose of the change

XXXXX

Brief changelog

XX

Verifying this change

XXXX

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
  • Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
  • Run mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test to make sure integration-test pass.

chenhao26-nineteen and others added 30 commits October 24, 2022 10:01
* support Batch DeRegister

* Adjust the de registration method

* use batchRegisterService
…alibaba#9112)

* Commit ConfigInfoMapper.

* Update ConfigInfoMapper and create ConfigInfoAggrMapper.
* [ISSUE alibaba#8312] Create ConfigInfoTagMapper, HisConfigInfoMapper and TenantInfoMapper.

* [ISSUE alibaba#8312] Add more mapper.

* [ISSUE alibaba#8312]Let mappers extend Mapper.
* [ISSUE alibaba#8312]Update the mappers.

* [ISSUE alibaba#8312]Update derby mappers.
* [ISSUE alibaba#8312]Finish all mappers.

* [ISSUE alibaba#8312]Fix some bugs about mapper.
* [ISSUE alibaba#8312]Finish the datasource plugin.

* [ISSUE alibaba#8312]Update derby PersistService impl.
…aba#9038)

* Enhance MetricsMonitor with several new metrics

* Add config read and write rt

* Add ServiceEvent publisher queue size

* Add fuzzy search count

* Add naming subscriber and publisher count with v1 or v2 version tag

* Add config subscriber count with v1 or v2 version tag

* Add pending push task count and empty push count metrics

* collect pending push task count by scheduled task

* add unit test for increment metrics in NacosMonitorPushResultHook

* Add topn counter container and topn config change

* Create private MeterRegistry for config change

* Add scheduled clear for config change count, add removeAll for TopnCounterMetricsContainer

* Add topn service change

* Reconstruct metrics monitor with NacosMeterRegistryCenter.

* divide globalRegistry to different modules' CompositeMeterRegistry

* Add dynamic meters refresh service for config and naming module

* fix unit test

* Add try catch when init NacosMeterRegistryCenter
…aba#9388)

* fix sql

* fix history sql

* fix clone sql

* fix beta sql
# Conflicts:
#	config/src/main/java/com/alibaba/nacos/config/server/controller/ConfigController.java
YunWZ and others added 5 commits April 6, 2023 15:07
…r/00-00---000-VIPSRV_FAILOVER_SWITCH-000---00-00`. (alibaba#10272)
* expose way to add config filter

* add @SInCE

* fix since time
* 取消默认openssl

* 连接上下文标签中添加tls.enable

* test fix

* checkstyle
@CLAassistant
Copy link

CLAassistant commented Apr 10, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
23 out of 24 committers have signed the CLA.

✅ YunWZ
✅ karsonto
✅ shiyiyue1102
✅ dspo
✅ MajorHe1
✅ varyuan
✅ wilsonwu
✅ 985492783
✅ KANLON
✅ Daydreamer-ia
✅ KomachiSion
✅ LLLWD
✅ yuyijq
✅ ZhangShenao
✅ wangshengji
✅ yuhaowin
✅ abuexclusive
✅ xuxiaowei-com-cn
✅ panxiaojun233
✅ xuechaos
✅ aloneboat
✅ MentosL
✅ vividfish
❌ lushun


lushun seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@MentosL MentosL closed this Apr 10, 2023
KomachiSion and others added 20 commits April 11, 2023 17:34
* Fix dynamic change token to illegal value will use old token.

* Add description in pom for release.

* Upgrade to 2.2.2

* Upgrade to 2.3.0-SNAPSHOT
* code optimization for `ConfigChangeHandler`

* [ISSUE alibaba#9930] fix word spelling error in com.alibaba.nacos.common.http.AbstractHttpClientFactory#ASYNC_IO_REACTOR_NAME

* [ISSUE alibaba#10020] Use Generic Type When Declaring `authFilterRegistration` in com.alibaba.nacos.core.auth.AuthConfig

* fix code format

* fix code format

* Fix Code Format

* Fix Code Format

* [ISSUE alibaba#10308] Use Generic Type
* add some unit test for api module.

* fix code style error
…va when push fail (alibaba#10312)

* [ISSUE alibaba#10310]Update log output in RpcConfigChangeNotifier.java when push fail

Update log output in RpcConfigChangeNotifier.java when push fail

* update indentation level

update indentation level
* fix import bug

* fix import bug
* remove ambiguous comments

* remove ambiguous comments
* Add UT for api module remote package

* Add UT for api module remote.response package

* For checkstyle.
[ISSUE#9821] Use new datasource plugin mapper api to support mysql and derby.
* Add unit test for api module remote.request package.

* For checkstyle.
* Supple unit tests for api module config.remote package.

* Supple unit tests for api module config package.
…libaba#10158)

* Replace sync forward request with async request in DistroFilter. issue alibaba#10148

* extract method for config default headers

* add env switch for async distro forward.

* Fixed code review problems:
1. Move async forward switch from sys module to naming module.
2. use nacos code style to format code.

* Fixed nacos code checkstyle:
1. one import per Class
2. add javadoc

* In order to avoid additional overhead, move switch from GlobalConfig to ClientConfig and cache the env switch.

* Move switch from ClientConfig to DistroConfig.

* Removed unused import.

* Add test for async forward for DistroFilter

* Add license

* rename test method name

* Should enable async forward

* fixed test

* set async forward request switch to true in test

* fixed test: create and set property with MockEnvironment

* fixed check style

* move MockEnvironment init to BeforeClass

* add setter for asyncForwardRequest switch
@MentosL MentosL reopened this May 3, 2023
@MentosL MentosL closed this May 3, 2023
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.