Skip to content

Correct rndPass usage #7047

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Correct rndPass usage #7047

wants to merge 1 commit into from

Conversation

pearj
Copy link

@pearj pearj commented Feb 10, 2023

While testing this on Windows Server 2019, I discovered that +rndPass shouldn't be combined with /pass, it is supposed to be used on it's own. Otherwise +rndPass becomes the actual password, which is unexpected and might not meet password complexity requirements.

Running a ktpass /? shows the that options +rndPass and /pass are separate from each other.

While testing this on Windows Server 2019, I discovered that `+rndPass` shouldn't be combined with `/pass`, it is supposed to be used on it's own. Otherwise `+rndPass` becomes the actual password, which is unexpected and might not meet password complexity requirements.

Running a `ktpass /?` shows the that options `+rndPass` and `/pass` are separate from each other.
@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

1 similar comment
@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

4 similar comments
@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@pearj
Copy link
Author

pearj commented Feb 10, 2023

In addition to this issue, I have noticed that command line option that starts with a / is also allowed to start with a - according to ktpass /?. Also the capitalisation of some of the commands has been lost too, eg: rndPass, minPass, maxPass, DesOnly, etc.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit c56314f:

✅ Validation status: passed

File Status Preview URL Details
WindowsServerDocs/administration/windows-commands/ktpass.md ✅Succeeded

For more details, please refer to the build report.

Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.

For any questions, please:

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Court72
Copy link
Contributor

Court72 commented Feb 10, 2023

@robinharwood, @Xelu86

Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Feb 10, 2023
@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

1 similar comment
@prmerger-automator
Copy link
Contributor

@pearj : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants