Jonathan Sumilang Ответов: 2

Как я могу реализовать шифрование логина и регистрационного пароля в C# winforms?


Поэтому я недавно создал функцию входа и регистрации в своей форме Windows c#, я реализовал шифрование SHA1, но ut потерпел неудачу в части дешифрования. Поэтому я искал другой способ шифрования пароля, который, как я обнаружил, является AES, но у меня есть трудная часть в его реализации, так как я использую базу данных (sql server) для сохранения данных из регистрации. Я надеюсь, что кто - нибудь сможет провести меня через это.

Что я уже пробовал:

У меня уже есть функции регистрации и входа в систему, единственное, чего мне не хватает в приложении, - это шифрование и дешифрование пароля.

Авторизоваться:
private void btn_Login_Click(object sender, EventArgs e)
        {
            SqlConnection con = new SqlConnection("Data Source=DESKTOP-2SSIMR1;Initial Catalog=Juan Carlo SCM;Integrated Security=True");
            SqlDataAdapter sda = new SqlDataAdapter("SELECT Username, Password FROM Employee_Table WHERE Username= '" + txtUsername.Text + "' AND Password = '" + txtPassword.Text + "' AND Status='Approved'", con);
            DataTable dt = new DataTable();
            sda.Fill(dt);

            if (dt.Rows.Count == 1)
            {
                this.Hide();
                Home hm = new Home(dt.Rows[0][0].ToString());
                hm.Show();
            }
            else
            {
                MessageBox.Show("Invalid username or Password", "Login Information", MessageBoxButtons.OK, MessageBoxIcon.Error);

            }
        }


Регистрация:
private void Register_Btn_Click(object sender, EventArgs e)
        {
            if (IsValidated())
            {
                try
                {
                    if (con.State == ConnectionState.Closed)
                        con.Open();
                    if (Register_Btn.Text == "Register")
                    {
                        SqlCommand cmd = new SqlCommand("RegisterAdd", con);
                        cmd.CommandType = CommandType.StoredProcedure;
                        cmd.Parameters.AddWithValue("@mode", "Add");
                        cmd.Parameters.AddWithValue("@EmployeeID", 0);
                        cmd.Parameters.AddWithValue("@LName", txtLastName.Text.Trim());
                        cmd.Parameters.AddWithValue("@FName", txtFirstName.Text.Trim());
                        cmd.Parameters.AddWithValue("@Contact", txtContact.Text.Trim());
                        cmd.Parameters.AddWithValue("@Email", txtEmail.Text.Trim());
                        cmd.Parameters.AddWithValue("@Username", txtUsername.Text.Trim());
                        cmd.Parameters.AddWithValue("@Password", txtPassword.Text.Trim());
                        cmd.Parameters.AddWithValue("@Position", txtPosition.Text.Trim());
                        cmd.Parameters.AddWithValue("@Status", txtStatus.Text.Trim());

                        if (txtPassword.Text != txtConfirmPassword.Text)
                        {
                            MessageBox.Show("Password not Match!");
                            txtPassword.Text = "";
                            txtConfirmPassword.Text = "";
                        }
                        if (txtUsername.Text.Length < 3 || txtPassword.Text.Length < 4)
                        {
                            MessageBox.Show("Username or Pasword is too short");
                        }
                        else
                        {

                            cmd.ExecuteNonQuery();
                            MessageBox.Show("Recorded Successfully.");
                            con.Close();
                        }
                    }
                }
                catch (Exception ex)
                {
                    MessageBox.Show(ex.Message, "Error Message");
                    return;
                }
                finally
                {
                    con.Close();
                }
            }
        }

2 Ответов

Рейтинг:
2

Patrice T

Цитата:
Я реализовал шифрование SHA1, но ut потерпел неудачу в части расшифровки.

Поскольку SHA не является шифрованием, его нельзя расшифровать.
SHA-1 - Википедия[^]
Ваш код использует обычный текстовый пароль в SQL-запросах, нет никакой попытки зашифровать его.

SqlDataAdapter sda = new SqlDataAdapter("SELECT Username, Password FROM Employee_Table WHERE Username= '" + txtUsername.Text + "' AND Password = '" + txtPassword.Text + "' AND Status='Approved'", con);

Никогда не создавайте SQL-запрос путем объединения строк. Рано или поздно вы сделаете это с помощью пользовательских вводов, и это откроет дверь к уязвимости под названием "SQL-инъекция", она опасна для вашей базы данных и подвержена ошибкам.
Одна кавычка в имени - и ваша программа выйдет из строя. Если пользователь вводит имя, например "Брайан О'Коннер", может привести к сбою вашего приложения, это уязвимость SQL-инъекции, и сбой-это наименьшая из проблем, вредоносный пользовательский ввод, и он продвигается к командам SQL со всеми учетными данными.
SQL-инъекция - Википедия[^]
SQL-инъекция[^]
Атаки SQL-инъекций на примере[^]
PHP: SQL-инъекция - руководство пользователя[^]
Шпаргалка по предотвращению инъекций SQL - OWASP[^]
Как я могу объяснить SQL-инъекцию без технического жаргона? - Обмен Стеками Информационной Безопасности[^]


Рейтинг:
0

OriginalGriff

SHA - это не алгоритм шифрования, это алгоритм хеширования, и они очень разные. Шифрование может быть отменено, чтобы восстановить исходный ввод, хеширование не может. Вы не можете "расшифровать" хэш-значение SHA1, чтобы получить обратно исходный пароль.

Сказав это, это именно то, что нужно использовать для пароля!
Нет, серьезно: шифрование паролей небезопасно, потому что ключ дешифрования должен быть частью вашего приложения, чтобы использовать его - а это значит, что вы храните ключ с данными там, где его легко найти. В результате зашифрованные пароли примерно так же безопасны, как и хранение их в обычном тексте!

Чтобы использовать хэшированные пароли, вы получаете хэшированное значение из БД и сравниваете его с хэшем, сгенерированным из того, что ввел пользователь. Если они совпадают, он вводит правильный пароль. Если они этого не делают, то и он тоже. Здесь есть некоторая информация о том, как это сделать: Хранение паролей: как это сделать.[^]

Но у вас есть более серьезная, опасная и насущная проблема: никогда не объединяйте строки для построения SQL-команды. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы.

Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:

SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.

Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
Вам нужно пройти через все ваше приложение и исправить везде, где вы это делаете - оставьте один, и ваша БД находится под угрозой. И делать это на странице входа в систему? Это не просто оставить входную дверь незапертой, это вручить грабителям ключи на выходе и сказать им код сигнализации...


Jonathan Sumilang

Я планирую сделать его параметризованным запросом до тех пор, пока он работает, я только сделал его таким, чтобы показать, как работают мой логин и регистрация. Мне просто нужно поставить функцию, которая бы шифровала пароль при регистрации и расшифровывала его по мере входа пользователя в систему

OriginalGriff

Никогда, никогда не делай этого. Когда вы это делаете, это забывается. Это проще, чище и проще - не говоря уже о том, что быстрее - сделать это правильно с самого начала, и это делает ваш код более читабельным.

Jonathan Sumilang

или в любом случае, чтобы я мог сделать свой логин и зарегистрироваться безопасно, мне просто нужно это сделать

OriginalGriff

Неправильное отношение: безопасность важна, отчасти поэтому ЕС ввел GDPR - и вы можете быть привлечены к ответственности за то, что бросаете ее вместе и надеетесь ...

Jonathan Sumilang

Итак Хм, Что же мне тогда делать?

OriginalGriff

Сначала исправьте параметризованные запросы, а затем перейдите по ссылке в моем первоначальном ответе (он использовал MD5, который устарел, но принципы точно такие же для SHA).
Сольте свой пароль с идентификатором пользователя (а не с именем пользователя, которое может измениться) и хэшируйте его. Затем сравните хэши.

Jonathan Sumilang

спасибо вам за это, извините, если я был немного упрям в этом вопросе

OriginalGriff

Всегда пожалуйста!