Add reset/passwd capability for local users #3287
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
alex-kovoy
requested review from
klizhentas,
benarent,
russjones and
fspmarshall
alex-kovoy
force-pushed
the
alexey/webapps-usertokens
branch
from
15dfbb9
to
ef9f4ae
Compare
|
Once this PR reviewed, I will send another with |
alex-kovoy
commented
Jan 16, 2020
| @@ -0,0 +1,84 @@ | |||
| /* | |||
| Copyright 2015 Gravitational, Inc. | |||
There was a problem hiding this comment.
FYI, new_web_user.go has been renamed to user.go. No code changes in regards to user functions.
fspmarshall
reviewed
Jan 17, 2020
Only |
alex-kovoy
force-pushed
the
alexey/webapps-usertokens
branch
from
1a20de1
to
d422161
Compare
|
RE:
This falls into the world of #3278 and I'm a little anxious of breaking compatibility. I chatted with @russjones and if we do, we need to plan accordingly. I'm going to write up risks and benefits of this approach into a doc and setup a meeting to review with the team. |
|
On a second thought, most of the old web ui will work fine with the new |
russjones
reviewed
Jan 20, 2020
alex-kovoy
force-pushed
the
alexey/webapps-usertokens
branch
from
d8a7bda
to
fb1e0d9
Compare
russjones
reviewed
Jan 28, 2020
alex-kovoy
force-pushed
the
alexey/webapps-usertokens
branch
2 times, most recently
from
9ae0c15
to
ebe42c2
Compare
alex-kovoy
force-pushed
the
alexey/webapps-usertokens
branch
from
ebe42c2
to
e596d1b
Compare
russjones
approved these changes
Jan 29, 2020
russjones
reviewed
Jan 31, 2020
alex-kovoy
added a commit
that referenced
this pull request
Mar 12, 2020
* Add UserTokens to allow password resets * Pass context down through ChangePasswordWithToken * Rename UserToken to ResetPasswordToken * Add auto formatting for proto files * Add common Marshaller interfaces to reset password token
This was referenced Mar 30, 2020
Closed
alex-kovoy
added a commit
that referenced
this pull request
Apr 14, 2020
* Add UserTokens to allow password resets * Pass context down through ChangePasswordWithToken * Rename UserToken to ResetPasswordToken * Add auto formatting for proto files * Add common Marshaller interfaces to reset password token
alex-kovoy
added a commit
that referenced
this pull request
Apr 15, 2020
* Add UserTokens to allow password resets * Pass context down through ChangePasswordWithToken * Rename UserToken to ResetPasswordToken * Add auto formatting for proto files * Add common Marshaller interfaces to reset password token
alex-kovoy
added a commit
that referenced
this pull request
Apr 15, 2020
* Add monorepo * Add reset/passwd capability for local users (#3287) * Add UserTokens to allow password resets * Pass context down through ChangePasswordWithToken * Rename UserToken to ResetPasswordToken * Add auto formatting for proto files * Add common Marshaller interfaces to reset password token * Allow enterprise "tctl" reuse OSS user methods (#3344) * Pass localAuthEnabled flag to UI (#3412) * Added LocalAuthEnabled prop to WebConfigAuthSetting struct in webconfig.go * Added LocalAuthEnabled state as part of webCfg in apiserver.go * update e-refs * Fix a regression bug after merge * Update tctl CLI output msgs (#3442) * Use local user client when resolving user roles * Update webapps ref * Add and retrieve fields from Cluster struct (#3476) * Set Teleport versions for node, auth, proxy init heartbeat * Add and retrieve fields NodeCount, PublicURL, AuthVersion from Clusters * Remove debug logging to avoid log pollution when getting public_addr of proxy * Create helper func GuessProxyHost to get the public_addr of a proxy host * Refactor newResetPasswordToken to use GuessProxyHost and remove publicUrl func * Remove webapps submodule * Add webassets submodule * Replace webapps sub-module reference with webassets * Update webassets path in Makefile * Update webassets 1b11b26 Simplify and clean up Makefile (#62) gravitational/webapps@1b11b26 * Retrieve cluster details for user context (#3515) * Let GuessProxyHost also return proxy's version * Unit test GuessProxyHostAndVersion & GetClusterDetails * Update webassets 4dfef4e Fix build pipeline (#66) gravitational/webapps@4dfef4e * Update e-ref * Update webassets 0647568 Fix OSS redirects gravitational/webapps@0647568 * update e-ref * Update webassets e0f4189 Address security audit warnings Updates "minimist" package which is used by 7y old "optimist". gravitational/webapps@e0f4189 * Add new attr to Session struct (#3574) * Add fields ServerHostname and ServerAddr * Set these fields on newSession * Ensure webassets submodule during build * Update e-ref * Ensure webassets before running unit-tests * Update E-ref Co-authored-by: Lisa Kim <lisa@gravitational.com> Co-authored-by: Pierre Beaucamp <pierre@gravitational.com> Co-authored-by: Jenkins <jenkins@gravitational.io>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR contains the following changes:
UserTokenandUserTokenSecrets.tctl users reset <username>command.Note, this PR is against
alexey/webappsbranch.Fixes #3265