Aplikacja konsoli do maskowania hasła

201

Próbowałem następującego kodu ...

string pass = "";
Console.Write("Enter your password: ");
ConsoleKeyInfo key;

do
{
    key = Console.ReadKey(true);

    // Backspace Should Not Work
    if (key.Key != ConsoleKey.Backspace)
    {
        pass += key.KeyChar;
        Console.Write("*");
    }
    else
    {
        Console.Write("\b");
    }
}
// Stops Receving Keys Once Enter is Pressed
while (key.Key != ConsoleKey.Enter);

Console.WriteLine();
Console.WriteLine("The Password You entered is : " + pass);

Ale w ten sposób funkcja backspace nie działa podczas wpisywania hasła. Jakieś sugestie?

Mohammad Nadeem
źródło
11
Sugeruję, żebyś nie powtarzał niczego z powrotem do konsoli, ponieważ ujawni to długość hasła.
Ray Cheng
8
@RayCheng - wystarczy, ale bardzo niewiele interfejsów użytkownika (innych niż w niektórych systemach uniksowych) w ogóle nic nie echa. Aby zachować spójność z innymi aplikacjami i stronami internetowymi, prawdopodobnie najlepiej jest pokazywać znaki *.
Stephen Holt
4
@StephenHolt Jestem dość pewien, że każde hasło, które kiedykolwiek napotkałem na terminalu, wybrało nic do echa terminala. Biorąc pod uwagę zalety bezpieczeństwa i fakt, że jest to dobrze znana konwencja w świecie uniksowym, osobiście uważam, że echo niczego nie jest właściwym wyborem, chyba że uważasz, że twoja baza użytkowników prawdopodobnie nie zna korzystania z terminali (w takim przypadku i tak prawdopodobnie najlepiej jest użyć GUI).
Ajedi32

Odpowiedzi:

225

Console.Write("\b \b");usunie znak gwiazdki z ekranu, ale nie masz kodu w elsebloku, który usuwa poprzednio wprowadzony znak ze passzmiennej ciągu.

Oto odpowiedni działający kod, który powinien zrobić to, czego potrzebujesz:

string pass = "";
do
{
    ConsoleKeyInfo key = Console.ReadKey(true);
    // Backspace Should Not Work
    if (key.Key != ConsoleKey.Backspace && key.Key != ConsoleKey.Enter)
    {
        pass += key.KeyChar;
        Console.Write("*");
    }
    else
    {
        if (key.Key == ConsoleKey.Backspace && pass.Length > 0)
        {
            pass = pass.Substring(0, (pass.Length - 1));
            Console.Write("\b \b");
        }
        else if(key.Key == ConsoleKey.Enter)
        {
            break;
        }
    }
} while (true);
CraigTP
źródło
2
Och, myślałem, że \ b \ b zabierze mnie dwa miejsca z powrotem. Niemniej jednak wydaje się, że działa to idealnie.
Mohammad Nadeem,
8
@Nadeem: Zwróć uwagę na spację ( ' ') między znakami backspace ( '\b'). "\b \b"zabiera cię o jedno miejsce wstecz, a następnie drukuje spację (która zabiera cię o jedno miejsce do przodu), a następnie zabiera cię z powrotem, dzięki czemu kończysz tam, gdzie była usunięta '*'postać.
dtb
14
@Nadeem - Pierwsze \bprzesuwa kursor do tyłu o jedną pozycję (obecnie pod ostatnim *char The. [space]Character „drukuje ponad” gwiazdką, ale także przesuwa kursor o jeden znak znowu do przodu, więc w ostatnich \bruchów tył kursora na którym ostatni *używany do be! (Uff - Mam nadzieję, że to ma sens!)
CraigTP
3
if (pass.Length > 0)powinno być if (key.Key == ConsoleKey.Backspace && pass.Length > 0)inaczej, nie dostaniesz ostatniego znaku hasła.
MemphiZ
9
Jeśli nie chcesz, aby użytkownik mógł znaków sterujących zapisem (jak F5 lub ucieczki), można zastąpić if (key.Key != ConsoleKey.Backspace && key.Key != ConsoleKey.Enter)z if (!char.IsControl(key.KeyChar)).
Safron
90

W tym celu należy użyć System.Security.SecureString

public SecureString GetPassword()
{
    var pwd = new SecureString();
    while (true)
    {
        ConsoleKeyInfo i = Console.ReadKey(true);
        if (i.Key == ConsoleKey.Enter)
        {
            break;
        }
        else if (i.Key == ConsoleKey.Backspace)
        {
            if (pwd.Length > 0)
            {
                pwd.RemoveAt(pwd.Length - 1);
                Console.Write("\b \b");
            }
        }
        else if (i.KeyChar != '\u0000' ) // KeyChar == '\u0000' if the key pressed does not correspond to a printable character, e.g. F1, Pause-Break, etc
        {
            pwd.AppendChar(i.KeyChar);
            Console.Write("*");
        }
    }
    return pwd;
}
Damian Leszczyński - Vash
źródło
Zajmie mi to tylko dwa miejsca z powrotem. Ale potrzebuję, aby po naciśnięciu klawisza Backspace ostatni znak powinien zostać usunięty. Podobnie jak oryginalna funkcjonalność backspace.
Mohammad Nadeem,
1
Musiałem zagnieździć if( pwd.Length > 0)się w pierwszej instrukcji, aby ludzie nie
usuwali
1
Podobnie jak komentarz Safrona do obecnie akceptowanej odpowiedzi, elseklauzula końcowa skorzystałaby z testu if (!char.IsControl(i.KeyChar))(a przynajmniej if (i.KeyChar != '\u0000')).
Peter Taylor
1
Jak mogę zmienić to hasło w ciąg znaków?
Joseph Kreifels II
47

Kompletne rozwiązanie, waniliowy C # .net 3.5+

Wytnij i wklej :)

    using System;
    using System.Collections.Generic;
    using System.Linq;
    using System.Text;

    namespace ConsoleReadPasswords
    {
        class Program
        {
            static void Main(string[] args)
            {
                Console.Write("Password:");

                string password = Orb.App.Console.ReadPassword();

                Console.WriteLine("Sorry - I just can't keep a secret!");
                Console.WriteLine("Your password was:\n<Password>{0}</Password>", password);

                Console.ReadLine();
            }
        }
    }

    namespace Orb.App
    {
        /// <summary>
        /// Adds some nice help to the console. Static extension methods don't exist (probably for a good reason) so the next best thing is congruent naming.
        /// </summary>
        static public class Console
        {
            /// <summary>
            /// Like System.Console.ReadLine(), only with a mask.
            /// </summary>
            /// <param name="mask">a <c>char</c> representing your choice of console mask</param>
            /// <returns>the string the user typed in </returns>
            public static string ReadPassword(char mask)
            {
                const int ENTER = 13, BACKSP = 8, CTRLBACKSP = 127;
                int[] FILTERED = { 0, 27, 9, 10 /*, 32 space, if you care */ }; // const

                var pass = new Stack<char>();
                char chr = (char)0;

                while ((chr = System.Console.ReadKey(true).KeyChar) != ENTER)
                {
                    if (chr == BACKSP)
                    {
                        if (pass.Count > 0)
                        {
                            System.Console.Write("\b \b");
                            pass.Pop();
                        }
                    }
                    else if (chr == CTRLBACKSP)
                    {
                        while (pass.Count > 0)
                        {
                            System.Console.Write("\b \b");
                            pass.Pop();
                        }
                    }
                    else if (FILTERED.Count(x => chr == x) > 0) { }
                    else
                    {
                        pass.Push((char)chr);
                        System.Console.Write(mask);
                    }
                }

                System.Console.WriteLine();

                return new string(pass.Reverse().ToArray());
            }

            /// <summary>
            /// Like System.Console.ReadLine(), only with a mask.
            /// </summary>
            /// <returns>the string the user typed in </returns>
            public static string ReadPassword()
            {
                return Orb.App.Console.ReadPassword('*');
            }
        }
    }
Shermy
źródło
3
Zawsze może być trudniej :) Nie działa na Macu / Linuksie, ponieważ nowa linia nie jest rozpoznawana. Environment.NewLine ma ciąg znaków nowej linii. Więc zmodyfikowałem to w: while (! Environment.NewLine.Contains (chr = System.Console.ReadKey (true) .KeyChar))
Hugo Logmans
18

Biorąc najwyższą odpowiedź, a także sugestie z jej komentarzy i modyfikując ją, aby używała SecureString zamiast String, przetestuj wszystkie klawisze sterujące, a nie popełniaj błędów lub napisz dodatkowe „*” na ekranie, gdy hasło ma wartość 0, moje rozwiązanie to:

public static SecureString getPasswordFromConsole(String displayMessage) {
    SecureString pass = new SecureString();
    Console.Write(displayMessage);
    ConsoleKeyInfo key;

    do {
        key = Console.ReadKey(true);

        // Backspace Should Not Work
        if (!char.IsControl(key.KeyChar)) {
            pass.AppendChar(key.KeyChar);
            Console.Write("*");
        } else {
            if (key.Key == ConsoleKey.Backspace && pass.Length > 0) {
                pass.RemoveAt(pass.Length - 1);
                Console.Write("\b \b");
            }
        }
    }
    // Stops Receving Keys Once Enter is Pressed
    while (key.Key != ConsoleKey.Enter);
    return pass;
}
MDMoore313
źródło
15

Mój ignoruje znaki kontrolne i obsługuje zawijanie wierszy:

public static string ReadLineMasked(char mask = '*')
{
    var sb = new StringBuilder();
    ConsoleKeyInfo keyInfo;
    while ((keyInfo = Console.ReadKey(true)).Key != ConsoleKey.Enter)
    {
        if (!char.IsControl(keyInfo.KeyChar))
        {
            sb.Append(keyInfo.KeyChar);
            Console.Write(mask);
        }
        else if (keyInfo.Key == ConsoleKey.Backspace && sb.Length > 0)
        {
            sb.Remove(sb.Length - 1, 1);

            if (Console.CursorLeft == 0)
            {
                Console.SetCursorPosition(Console.BufferWidth - 1, Console.CursorTop - 1);
                Console.Write(' ');
                Console.SetCursorPosition(Console.BufferWidth - 1, Console.CursorTop - 1);
            }
            else Console.Write("\b \b");
        }
    }
    Console.WriteLine();
    return sb.ToString();
}
Ronnie Overby
źródło
Działa świetnie. Może być jednak konieczne dodanie kodu, aby DELETE znak kasował cały wprowadzony tekst. Jego sekwencja kluczy to, CTRL + BACKSPACEa kod char to 0x7f.
Alex Essilfie
9

Czytanie danych wejściowych konsoli jest trudne, musisz obsługiwać specjalne klawisze, takie jak Ctrl, Alt, a także klawisze kursora i Backspace / Delete. Na niektórych układach klawiatury, takich jak szwedzki Ctrl, jest nawet potrzebny do wprowadzania klawiszy, które istnieją bezpośrednio na klawiaturze amerykańskiej. Uważam, że próbuję sobie z tym poradzić przy użyciu „niskiego poziomu”Console.ReadKey(true) jest po prostu bardzo trudna, więc najłatwiejszym i najsolidniejszym sposobem jest po prostu wyłączenie „echa wejścia konsoli” podczas wprowadzania hasła przy użyciu odrobiny WINAPI.

Poniższy przykład oparty jest na odpowiedzi na pytanie Przeczytaj hasło z pytania std :: cin .

    private enum StdHandle
    {
        Input = -10,
        Output = -11,
        Error = -12,
    }

    private enum ConsoleMode
    {
        ENABLE_ECHO_INPUT = 4
    }

    [DllImport("kernel32.dll", SetLastError = true)]
    private static extern IntPtr GetStdHandle(StdHandle nStdHandle);

    [DllImport("kernel32.dll", SetLastError = true)]
    [return: MarshalAs(UnmanagedType.Bool)]
    private static extern bool GetConsoleMode(IntPtr hConsoleHandle, out int lpMode);

    [DllImport("kernel32.dll", SetLastError = true)]
    [return: MarshalAs(UnmanagedType.Bool)]
    private static extern bool SetConsoleMode(IntPtr hConsoleHandle, int dwMode);

    public static string ReadPassword()
    {
        IntPtr stdInputHandle = GetStdHandle(StdHandle.Input);
        if (stdInputHandle == IntPtr.Zero)
        {
            throw new InvalidOperationException("No console input");
        }

        int previousConsoleMode;
        if (!GetConsoleMode(stdInputHandle , out previousConsoleMode))
        {
            throw new Win32Exception(Marshal.GetLastWin32Error(), "Could not get console mode.");
        }

        // disable console input echo
        if (!SetConsoleMode(stdInputHandle , previousConsoleMode & ~(int)ConsoleMode.ENABLE_ECHO_INPUT))
        {
            throw new Win32Exception(Marshal.GetLastWin32Error(), "Could not disable console input echo.");
        }

        // just read the password using standard Console.ReadLine()
        string password = Console.ReadLine();

        // reset console mode to previous
        if (!SetConsoleMode(stdInputHandle , previousConsoleMode))
        {
            throw new Win32Exception(Marshal.GetLastWin32Error(), "Could not reset console mode.");
        }

        return password;
    }
Rafał Kłys
źródło
9

To maskuje hasło czerwonym kwadratem, a następnie powraca do pierwotnych kolorów po wprowadzeniu hasła.

To nie powstrzymuje użytkownika przed użyciem funkcji kopiuj / wklej, aby uzyskać hasło, ale jeśli chodzi o zatrzymanie kogoś, kto patrzy przez ramię, jest to dobre szybkie rozwiązanie.

Console.Write("Password ");
ConsoleColor origBG = Console.BackgroundColor; // Store original values
ConsoleColor origFG = Console.ForegroundColor;

Console.BackgroundColor = ConsoleColor.Red; // Set the block colour (could be anything)
Console.ForegroundColor = ConsoleColor.Red;

string Password = Console.ReadLine(); // read the password

Console.BackgroundColor= origBG; // revert back to original
Console.ForegroundColor= origFG;
Rich S.
źródło
Jedynym problemem jest to, że jeśli zrobisz backspace, tło, w którym masz postacie, pozostanie czerwone. Wolę raczej ustawić ForegroundColor jako origBG, aby wprowadzić hasło w stylu Linux.
mababin
1
Możesz także zrobić to Console.CursorVisible=falsei przywrócić poprzednią wartość po. Zapobiegnie to osiągnięciu wartości szczytowej przy długości hasła.
mababin
6

Znalazłem błąd w waniliowym rozwiązaniu Shermana C # 3.5 .NET, które w innym przypadku działałoby na urok. Włączyłem tu również Damiana Leszczyńskiego - pomysł SecureString firmy Vash, ale jeśli wolisz, możesz użyć zwykłego ciągu.

BŁĄD: Jeśli naciśniesz klawisz Backspace podczas pytania o hasło, a bieżąca długość hasła wynosi 0, oznacza to, że gwiazdka jest niepoprawnie wstawiona do maski hasła. Aby naprawić ten błąd, zmodyfikuj następującą metodę.

    public static string ReadPassword(char mask)
    {
        const int ENTER = 13, BACKSP = 8, CTRLBACKSP = 127;
        int[] FILTERED = { 0, 27, 9, 10 /*, 32 space, if you care */ }; // const


        SecureString securePass = new SecureString();

        char chr = (char)0;

        while ((chr = System.Console.ReadKey(true).KeyChar) != ENTER)
        {
            if (((chr == BACKSP) || (chr == CTRLBACKSP)) 
                && (securePass.Length > 0))
            {
                System.Console.Write("\b \b");
                securePass.RemoveAt(securePass.Length - 1);

            }
            // Don't append * when length is 0 and backspace is selected
            else if (((chr == BACKSP) || (chr == CTRLBACKSP)) && (securePass.Length == 0))
            {
            }

            // Don't append when a filtered char is detected
            else if (FILTERED.Count(x => chr == x) > 0)
            {
            }

            // Append and write * mask
            else
            {
                securePass.AppendChar(chr);
                System.Console.Write(mask);
            }
        }

        System.Console.WriteLine();
        IntPtr ptr = new IntPtr();
        ptr = Marshal.SecureStringToBSTR(securePass);
        string plainPass = Marshal.PtrToStringBSTR(ptr);
        Marshal.ZeroFreeBSTR(ptr);
        return plainPass;
    }
Stephen Wilson
źródło
4

Oto wersja, która dodaje obsługę Escapeklucza (która zwraca nullciąg znaków)

public static string ReadPassword()
{
    string password = "";
    while (true)
    {
        ConsoleKeyInfo key = Console.ReadKey(true);
        switch (key.Key)
        {
            case ConsoleKey.Escape:
                return null;
            case ConsoleKey.Enter:
                return password;
            case ConsoleKey.Backspace:
                if (password.Length > 0) 
                {
                    password = password.Substring(0, (password.Length - 1));
                    Console.Write("\b \b");
                }
                break;
            default:
                password += key.KeyChar;
                Console.Write("*");
                break;
        }
    }
}
Sven Vranckx
źródło
2

(Mój) pakiet nuget, aby to zrobić, w oparciu o pierwszą odpowiedź:

install-package PanoramicData.ConsoleExtensions

Stosowanie:

using PanoramicData.ConsoleExtensions;

...

Console.Write("Password: ");
var password = ConsolePlus.ReadPassword();
Console.WriteLine();

Adres projektu: https://github.com/panoramicdata/PanoramicData.ConsoleExtensions

Witamy w zaproszeniach do ściągania.

David Bond
źródło
Użyłem tego do małego projektu. Działa zgodnie z oczekiwaniami. Dziękuję
gmalenko
1

Możesz dołączyć swoje klucze do kumulowanej listy połączonej.

Po otrzymaniu klawisza Backspace usuń ostatni klucz z listy.

Po otrzymaniu klawisza Enter zwiń listę w ciąg znaków i wykonaj resztę pracy.

Sarnold
źródło
Brzmi osiągalnie, ale jak mogę usunąć ostatnią postać z wyświetlacza.
Mohammad Nadeem,
1

Wprowadziłem kilka zmian dla backspace

        string pass = "";
        Console.Write("Enter your password: ");
        ConsoleKeyInfo key;

        do
        {
            key = Console.ReadKey(true);

            // Backspace Should Not Work
            if (key.Key != ConsoleKey.Backspace)
            {
                pass += key.KeyChar;
                Console.Write("*");
            }
            else
            {
                pass = pass.Remove(pass.Length - 1);
                Console.Write("\b \b");
            }
        }
        // Stops Receving Keys Once Enter is Pressed
        while (key.Key != ConsoleKey.Enter);

        Console.WriteLine();
        Console.WriteLine("The Password You entered is : " + pass);
użytkownik2346303
źródło
1

Oto moja prosta wersja. Za każdym razem, gdy naciśniesz klawisz, usuń wszystko z konsoli i narysuj tyle „*”, ile wynosi długość ciągu hasła.

int chr = 0;
string pass = "";
const int ENTER = 13;
const int BS = 8;

do
{
   chr = Console.ReadKey().KeyChar;
   Console.Clear(); //imediately clear the char you printed

   //if the char is not 'return' or 'backspace' add it to pass string
   if (chr != ENTER && chr != BS) pass += (char)chr;

   //if you hit backspace remove last char from pass string
   if (chr == BS) pass = pass.Remove(pass.Length-1, 1);

   for (int i = 0; i < pass.Length; i++)
   {
      Console.Write('*');
   }
} 
 while (chr != ENTER);

Console.Write("\n");
Console.Write(pass);

Console.Read(); //just to see the pass
Svicar
źródło
0

Zaktualizowałem wersję Ronniego po spędzeniu zbyt dużo czasu tym, jak spędziłem próbie wprowadzenia hasła, aby dowiedzieć się, że miałem włączony CAPS LOCK!

W tej wersji treść wiadomości _CapsLockMessagebędzie „unosić się” na końcu obszaru pisania i będzie wyświetlana na czerwono.

Ta wersja wymaga nieco więcej kodu i wymaga pętli odpytywania. Na moim komputerze użycie procesora około 3% do 4%, ale zawsze można dodać małą wartość Sleep (), aby w razie potrzeby zmniejszyć użycie procesora.

    private const string _CapsLockMessage = " CAPS LOCK";

    /// <summary>
    /// Like System.Console.ReadLine(), only with a mask.
    /// </summary>
    /// <param name="mask">a <c>char</c> representing your choice of console mask</param>
    /// <returns>the string the user typed in</returns>
    public static string ReadLineMasked(char mask = '*')
    {
        // Taken from http://stackoverflow.com/a/19770778/486660
        var consoleLine = new StringBuilder();
        ConsoleKeyInfo keyInfo;
        bool isDone;
        bool isAlreadyLocked;
        bool isCapsLockOn;
        int cursorLeft;
        int cursorTop;
        ConsoleColor originalForegroundColor;

        isDone = false;
        isAlreadyLocked = Console.CapsLock;

        while (isDone == false)
        {
            isCapsLockOn = Console.CapsLock;
            if (isCapsLockOn != isAlreadyLocked)
            {
                if (isCapsLockOn)
                {
                    cursorLeft = Console.CursorLeft;
                    cursorTop = Console.CursorTop;
                    originalForegroundColor = Console.ForegroundColor;
                    Console.ForegroundColor = ConsoleColor.Red;
                    Console.Write("{0}", _CapsLockMessage);
                    Console.SetCursorPosition(cursorLeft, cursorTop);
                    Console.ForegroundColor = originalForegroundColor;
                }
                else
                {
                    cursorLeft = Console.CursorLeft;
                    cursorTop = Console.CursorTop;
                    Console.Write("{0}", string.Empty.PadRight(_CapsLockMessage.Length));
                    Console.SetCursorPosition(cursorLeft, cursorTop);
                }
                isAlreadyLocked = isCapsLockOn;
            }

            if (Console.KeyAvailable)
            {
                keyInfo = Console.ReadKey(intercept: true);

                if (keyInfo.Key == ConsoleKey.Enter)
                {
                    isDone = true;
                    continue;
                }

                if (!char.IsControl(keyInfo.KeyChar))
                {
                    consoleLine.Append(keyInfo.KeyChar);
                    Console.Write(mask);
                }
                else if (keyInfo.Key == ConsoleKey.Backspace && consoleLine.Length > 0)
                {
                    consoleLine.Remove(consoleLine.Length - 1, 1);

                    if (Console.CursorLeft == 0)
                    {
                        Console.SetCursorPosition(Console.BufferWidth - 1, Console.CursorTop - 1);
                        Console.Write(' ');
                        Console.SetCursorPosition(Console.BufferWidth - 1, Console.CursorTop - 1);
                    }
                    else
                    {
                        Console.Write("\b \b");
                    }
                }

                if (isCapsLockOn)
                {
                    cursorLeft = Console.CursorLeft;
                    cursorTop = Console.CursorTop;
                    originalForegroundColor = Console.ForegroundColor;
                    Console.ForegroundColor = ConsoleColor.Red;
                    Console.Write("{0}", _CapsLockMessage);
                    Console.CursorLeft = cursorLeft;
                    Console.CursorTop = cursorTop;
                    Console.ForegroundColor = originalForegroundColor;
                }
            }
        }

        Console.WriteLine();

        return consoleLine.ToString();
    }
Jim
źródło
-1

Jeśli dobrze to rozumiem, próbujesz sprawić, że backspace usunie zarówno widoczny * znak na ekranie, jak i buforowany znak w zmiennej pass?

Jeśli tak, to po prostu zmień blok else na to:

            else
            {
                Console.Write("\b");
                pass = pass.Remove(pass.Length -1);
            }

źródło
1
Działa to dobrze, z tym wyjątkiem, że usunięcie znaku przez backspace nie będzie wyświetlane.
Mohammad Nadeem,
-2
 string pass = "";
 Console.WriteLine("Enter your password: ");
 ConsoleKeyInfo key;

 do {
  key = Console.ReadKey(true);

  if (key.Key != ConsoleKey.Backspace) {
   pass += key.KeyChar;
   Console.Write("*");
  } else {
   Console.Write("\b \b");
   char[] pas = pass.ToCharArray();
   string temp = "";
   for (int i = 0; i < pass.Length - 1; i++) {
    temp += pas[i];
   }
   pass = temp;
  }
 }
 // Stops Receving Keys Once Enter is Pressed
 while (key.Key != ConsoleKey.Enter);

 Console.WriteLine();
 Console.WriteLine("The Password You entered is : " + pass);
Neeraj Kumar
źródło
1
Ta odpowiedź nie dodaje niczego poza istniejącymi odpowiedziami. Ponadto dobre odpowiedzi powinny zazwyczaj wyjaśniać kod, a nie tylko wklejać kod w polu odpowiedzi. Proszę przeczytać Jak odpowiedzieć
durron597