-
Notifications
You must be signed in to change notification settings - Fork 858
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
Feat: add default user in dex #3753
Conversation
ece1224
to
b79f4ff
Compare
Codecov Report
@@ Coverage Diff @@
## master #3753 +/- ##
===========================================
+ Coverage 27.50% 63.84% +36.34%
===========================================
Files 244 310 +66
Lines 19665 29343 +9678
===========================================
+ Hits 5408 18735 +13327
+ Misses 13290 8184 -5106
- Partials 967 2424 +1457
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
@@ -105,29 +93,6 @@ func (u *userUsecaseImpl) Init(ctx context.Context) error { | |||
}); err != nil { | |||
return err | |||
} | |||
// print default password of admin user in log | |||
log.Logger.Infof("initialized admin username and password: admin / %s\n", pwd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
keep print this log.
@@ -105,29 +93,6 @@ func (u *userUsecaseImpl) Init(ctx context.Context) error { | |||
}); err != nil { | |||
return err | |||
} | |||
// print default password of admin user in log | |||
log.Logger.Infof("initialized admin username and password: admin / %s\n", pwd) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, please keep it as users will read the old doc for the password.
Signed-off-by: FogDong <dongtianxin.tx@alibaba-inc.com>
Signed-off-by: FogDong <dongtianxin.tx@alibaba-inc.com>
Signed-off-by: FogDong <dongtianxin.tx@alibaba-inc.com>
8854b28
to
c68c5d0
Compare
Signed-off-by: FogDong dongtianxin.tx@alibaba-inc.com
Description of your changes
add default user in dex
I have:
make reviewable
to ensure this PR is ready for review.backport release-x.y
labels to auto-backport this PR if necessary.How has this code been tested
Special notes for your reviewer
/cc @barnettZQG