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

feat: add more parameters to nacosConfig, then you can pass usr/pwd p… #1242

Merged
merged 1 commit into from
Sep 27, 2022

Conversation

colin-lee
Copy link
Contributor

when nacos-registry enable auth,it will fail when register service.

Motivation:

bypass username/password parameters to nacosConfig

please specify it in application.properties

com.alipay.sofa.rpc.registry.address=nacos://172.31.101.100:8848,172.31.101.101:8848,172.31.101.102:8848/fxiaoke?username=xxxx&password=yyyy

@sofastack-bot
Copy link

sofastack-bot bot commented Sep 12, 2022

Hi @colin-lee, welcome to SOFAStack community, Please sign Contributor License Agreement!

After you signed CLA, we will automatically sync the status of this pull request in 3 minutes.

@sofastack-bot sofastack-bot bot added cla:no Need sign CLA First-time contributor First-time contributor question Further information is requested size/XS labels Sep 12, 2022
@EvenLjj EvenLjj added this to the 5.9.0 milestone Sep 13, 2022
@codecov
Copy link

codecov bot commented Sep 25, 2022

Codecov Report

Merging #1242 (4a6cf93) into master (169016c) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1242      +/-   ##
============================================
- Coverage     72.04%   72.02%   -0.02%     
  Complexity      779      779              
============================================
  Files           410      410              
  Lines         17376    17378       +2     
  Branches       2703     2703              
============================================
- Hits          12519    12517       -2     
- Misses         3475     3479       +4     
  Partials       1382     1382              
Impacted Files Coverage Δ
.../java/com/alipay/sofa/rpc/common/RpcConstants.java 100.00% <ø> (ø)
.../main/java/com/alipay/sofa/rpc/common/Version.java 100.00% <ø> (ø)
...ipay/sofa/rpc/codec/bolt/SofaRpcSerialization.java 76.19% <100.00%> (+0.17%) ⬆️
...ofa/rpc/registry/consul/HealthServiceInformer.java 84.78% <0.00%> (-4.35%) ⬇️
...n/java/com/alipay/sofa/rpc/log/TimeWaitLogger.java 80.00% <0.00%> (-4.00%) ⬇️
...om/alipay/sofa/rpc/server/triple/TripleServer.java 74.86% <0.00%> (-1.07%) ⬇️
...ay/sofa/rpc/client/AllConnectConnectionHolder.java 60.76% <0.00%> (+0.25%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Collaborator

@EvenLjj EvenLjj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@OrezzerO OrezzerO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@EvenLjj EvenLjj merged commit 2bc39bf into sofastack:master Sep 27, 2022
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
cla:no Need sign CLA First-time contributor First-time contributor question Further information is requested size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants