Differential D11258

[scripts] Hide password
ClosedPublic
Actions

Authored by inka on Mar 6 2024, 6:26 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Dec 18, 9:57 PM

	Unknown Object (File)
	Mon, Dec 16, 2:10 PM

	Unknown Object (File)
	Mon, Dec 16, 2:10 PM

	Unknown Object (File)
	Mon, Dec 16, 2:10 PM

	Unknown Object (File)
	Mon, Dec 16, 2:10 PM

	Unknown Object (File)
	Wed, Dec 11, 7:56 PM

	Unknown Object (File)
	Nov 13 2024, 4:35 AM

	Unknown Object (File)
	Nov 13 2024, 4:35 AM

View All 52 Files

Subscribers

Details

Reviewers

tomek
michal

Commits

rCOMM9f1938b9e6ea: [scripts] Hide password

Summary

issue: ENG-6874
We want to hide the password in console

Test Plan

ran $PRJ_ROOT/scripts/set-up-authoritative-keyserver.sh from dev-shell.nix
checked that the password is not visible

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Mar 6 2024, 6:26 AM

Herald added a subscriber: ashoat. · View Herald TranscriptMar 6 2024, 6:26 AM

inka edited the test plan for this revision. (Show Details)Mar 6 2024, 6:27 AM

inka added a parent revision: D11246: [scripts] Add scrips for setting up a new authoritative keyserver.

Harbormaster completed remote builds in B27363: Diff 37894.Mar 6 2024, 6:57 AM

inka requested review of this revision.Mar 6 2024, 6:57 AM

inka added a child revision: D11267: [keyserver] Print warnings if usingIdentityCredentials not set.Mar 7 2024, 3:22 AM

This looks really hacky, but we probably don't have a clear way of doing this.

This revision is now accepted and ready to land.Mar 7 2024, 3:24 AM

Closed by commit rCOMM9f1938b9e6ea: [scripts] Hide password (authored by inka). · Explain WhyMar 13 2024, 3:57 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMM9f1938b9e6ea: [scripts] Hide password.

Revision Contents
Changeset List

Path

Size

scripts/

set-user-credentials.js

22 lines

Diff 38035

scripts/set-user-credentials.js

Loading...