-
Notifications
You must be signed in to change notification settings - Fork 1
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
Парочка изменений #1
Comments
Привет, спасибо)
|
|
Я как не виндовс юзеров, не шарю за это) |
На windows во время получения появляется 4 окна powershell ) |
Прикол) let csproduct_output = run_shell_comand(
"powershell",
[
"-WindowStyle",
"Hidden",
"-command",
r#"Get-WmiObject Win32_ComputerSystemProduct | Select-Object -ExpandProperty UUID"#,
],
)
.unwrap_or("".into()); Тут до сих пор открыто issue, но пишут, что обязательно сначала |
|
Отлично, а проверь, данные отдает такая команда? Если да, то поправлю и обновлю сегодня |
Отдаёт |
Готово, оптимизировал зависимости и добавил WindowStyle Hidden |
Всё работает, но всё-равно очень медленно. Минимальное время полученное в ходе 3 запусков приложения: 2.89s. |
Наверное надо будет асинхронно вызывать команды powershell, чтобы ускорить |
Загрузи крейт к себе на пк, замени код в #[cfg(target_os = "windows")]
use crate::errors::MIDError;
#[cfg(target_os = "windows")]
use crate::utils::run_shell_comand;
#[cfg(target_os = "windows")]
pub(crate) fn get_mid_result() -> Result<String, MIDError> {
let combined_output = run_shell_comand(
"powershell",
[
"-WindowStyle",
"Hidden",
"-command",
r#"
$csproduct = Get-WmiObject Win32_ComputerSystemProduct | Select-Object -ExpandProperty UUID;
$bios = Get-WmiObject Win32_BIOS | Select-Object -ExpandProperty SerialNumber;
$baseboard = Get-WmiObject Win32_BaseBoard | Select-Object -ExpandProperty SerialNumber;
$cpu = Get-WmiObject Win32_Processor | Select-Object -ExpandProperty ProcessorId;
"$csproduct|$bios|$baseboard|$cpu"
"#,
],
)
.unwrap_or("".into());
if combined_output.is_empty() {
return Err(MIDError::ResultMidError);
}
let combined_string = combined_output
.trim()
.trim_start_matches('|')
.trim_end_matches('|');
Ok(combined_string.to_string())
} И да, перед этим сними mid текущей версии и сравни его с тем, что получится с этим кодом, должно совпасть. |
Если нужно часто дергать mid, то например, имеет смысл сохранять его в state |
Извиняюсь за долгий ответ. Hwid я записывал в sessionStorage. Сейчас я отказался от получения hwid из-за проблем с уникальностью получаемого hwid. На Linux и macos может и сработает подобный вариант получения hwid (который у Вас в crate), но на windows постоянно попадаются повторения получаемого hwid. |
Проблемы с уникальностью в целом на windows или в текущем крейте?
На Linux практически нет надежных вариантов, она вся секьюрная, самый идеальный вариант для macos.
Можно подробней пожалуйста, как вы это отследили, запустили трафик на свое приложение? Странно, используется 4 не связанных между собой параметра, у разных людей вы заметили одинаковый хеш MID? |
Проблемы с уникальностью в целом на windows. Другие платформы протестировать не было возможности. Я разрабатываю приватный лаунчер для Minecraft проектов. На бэке организовал добавление в таблицу к пользователю hwid. В итоге тестирования на аудитории ~10к человек у многих вылетала ошибка, которая гласила, что данный ПК уже привязан к другому аккаунту. Я пробовал и другие параметры использовать для вычисления hwid, но повторения всё равно были. |
А сколько конкретно людей из 10к, можно посчитать? Интересен %.
|
|
У некоторых пользователей были проблемы с получением части параметров. Почему на одной системе все параметры получаются, а на другой нет - для меня загадка |
Да, я только начал про это писать, опередили) |
"А сколько конкретно людей из 10к, можно посчитать? Интересен %." - процент подсчитать не смогу. Ещё хотел добавить, что процессор лучше не использовать для получения hwid. Если разработчик хочет сделать привязку ПК к аккаунту, то значение будет меняться. Я думаю, что стоит идти в сторону получения каких-то данных от материнской платы, чтобы при замене процессора не происходила смена hwid. |
В ближайшие дни верну проверку по HWID и отпишусь о результатах более подробно |
Имейте ввиду, что cpu_output может меняться?
Проблема в том, что ПК не винде, как конструктор, по идее там можно заменить всё и тогда отпечаток изменится, но надеюсь это не частый случай и в таком случае пользователю просто надо будет перепривязать свой аккаунт на обновленном устройстве. |
Подобная защита по HWID защищает только от школьников, посмотревших гайды на ютубе, но когда аудитория состоит только из них - это хорошее решение))) (на винде) |
А как еще иначе лицензировать софт кроме как создания MID?) Поэтому и используются идентификаторы железок, которые программно не подменить, только если железо перебирать, что становится нецелесообразным для тех, кто хочет ваш софт использовать бесплатно (если есть триал версия на каждый уникальный MID) |
@michitta а потестируй пожалуйста этот вариант #1 (comment) |
Выпустил версию 1.1.3 с этим кодом, который быстрее чуть больше х2. |
@doroved использую mid 2.0.0 на Виндах (10 и 11) при каждом запросе к get_mid_result вылезает окно powershell. Похоже что "-WindowStyle" не работает. Может стоит перейти на Command::new("powershell") ? |
@doroved вот я нашел комбинацию которая вроде как не открывает отдельное окно powershell на windows const CREATE_NO_WINDOW: u32 = 0x08000000;
use std::os::windows::process::CommandExt;
...
let combined_output = Command::new("powershell")
.args([
"-WindowStyle",
"Hidden",
"-command",
r#"
$csproduct = Get-WmiObject Win32_ComputerSystemProduct | Select-Object -ExpandProperty UUID;
$bios = Get-WmiObject Win32_BIOS | Select-Object -ExpandProperty SerialNumber;
$baseboard = Get-WmiObject Win32_BaseBoard | Select-Object -ExpandProperty SerialNumber;
$cpu = Get-WmiObject Win32_Processor | Select-Object -ExpandProperty ProcessorId;
"$csproduct|$bios|$baseboard|$cpu"
"#,
])
.creation_flags(CREATE_NO_WINDOW)
.stderr(Stdio::piped())
.stdout(Stdio::piped())
.spawn();
match combined_output {
Ok(output) => {
let output = output.wait_with_output().unwrap();
if !output.status.success() {
return Err("Failed to get device ID".to_string());
}
let combined_output = String::from_utf8_lossy(&output.stdout);
if combined_output.is_empty() {
return Err("Failed to get device ID".to_string());
}
println!("Device ID: {}", combined_output.trim());
Ok(
combined_output
.trim()
.trim_start_matches('|')
.trim_end_matches('|')
.to_lowercase(),
)
}
Err(e) => Err(e.to_string()),
} |
Похоже винда у всех "своя" и все работает по разному, вот у человека окно пропало после -WindowStyle В след версии я уберу powershell и буду использовать WMI, там практически мгновенно инфа получается + надежнее надеюсь будет) |
@doroved когда планируешь выпустить новую версию ? |
может на выходных выделю на это время, чтобы не отвлекаться больше. Я пингану тут, потестим, т.к. я не на винде. |
@doroved Да, напиши я потестирую на Виндах. |
ох, если честно под винду писать на маке это нереально) Попробуй потестировать у себя mid = { git = "https://github.com/doroved/mid", branch = "dev" } Если заработает, то сверь хеши текущей версии и дев, нужно чтобы совпадали |
@doroved поставь на Маке utm с Windows 11, там все есть для rust development -а Я попробую как время будет на Виндах. Я тоже на маке сижу в основном и я Parallel использую для Windows 11 Arm |
@doroved протестирвал на Windows, все работает. Решение использовать
По крайней мере в моем случае, мне нужен уникальный номер машины клиента что положить его в базу вместе с лицензией которую он купит для софта и проверять его при активации этой лицензии. Я не хотел бы чтоб например при обновлении процессора на компьютере у меня получался новый Machine ID поэтому в моем случае мне вполне будет достаточно получения Machine ID только на основе computer_uuid_base. В случае если его не удалось достать, можно проверять уже на bios_serial_base[0].SerialNumber или baseboard_serial_base[0].SerialNumber. Но комбинация всех этих параметров, опять же для моего кейса скорей всего не подходит. Если ты сделаешь опцию по каким параметрам генерить Machine ID то наверное тогда мне будет целесообразно использовать твою либу а так скорей всего мне достаточно сделать запросы на Windows
let computer_uuid_base: Vec<UUIDGeneric> = wmi_connection
.raw_query("SELECT UUID from Win32_ComputerSystemProduct WHERE UUID IS NOT NULL")
.unwrap(); и на Маке let system_profiler_output = run_shell_comand(
"sh",
[
"-c",
r#"system_profiler SPHardwareDataType"#,
],
)?;
// Target only the Hardware UUID
let target = "Hardware UUID"; (у меня только Мак и Windows) |
@kurdin на текущий момент концепция либы такая, что ничего настраивать не нужно, а просто сразу получаешь хеш. Может в будущем, как доп опция будет, но не планировал. |
У меня на виртуалке вообще только отдает csproduct через wmic, через poweshell команду - терминал просто закрывается. |
@doroved У меня все работает нормально на Mac с Parallel и Windows 11 Arm, а также на обычных Windows 10 и Windows 11 с Intel. Я пробовал твой dev код, и все работало. Но для себя я выбрал более простой вариант с получением только UUID, и это тоже работает хорошо. Вот мой код: Windows Для моей задачи этого вполне достаточно. Честно говоря, я не понимаю, зачем нужно что-то еще, если у тебя есть доступ к Hardware UUID. Зачем добавлять что-то еще к уже достаточно уникальному UUID? Конечно, этот ID может быть не 100% уникальным среди всех существующих компьютеров, но, как мне кажется, этого вполне достаточно для большинства задач. В общем, я выбрал более простой вариант, и он вполне подходит для моих нужд в плане уникальности. Ошибка, которую ты видишь, думаю, говорит о том, что в UTM не реализовали нормальную поддержку WMI. Я давно перестал использовать UTM и перешел на платный Parallel — там проблем меньше, но тоже есть. Ты можешь попробовать еще поставить VMware Fusion Pro — его сейчас сделали бесплатным для личного пользования. Спасибо тебе за библиотеку. |
Ну это самое главное.
🤝 |
@doroved я почитал и поспрашивал ИИ насчет целесообразности добавления чего либо еще кроме Hardware UUID для получения уникального номера машины. Ты прав, что есть кейсы когда одного UUID мало и есть смысл добавить номера материнской платы и серийный номер процессора. Было бы неплохо сделать это опциями в будущем. Где по умолчанию будет добавляться все а если не нужно, то можно выбрать какие параметры использовать для формировании уникального номера машины. Но я бы сделал чуть проще и оставил только UUID, SerialNumber и ProcessorId. Думаю этого будет достаточно. Спасибо еще раз. |
@kurdin да, я в обнимку с GPT изучал все + гугл и т.п. и выбрал самые лучшие варианты для создания хеша.
С виндой и тем более линуксом в этом плане все не просто, самый идеальный вариант для мак. Смысл брать один показатель из того же SPHardwareDataType, когда можно взять другие, тем более доп запросы не нужны. + я изучил все параметры и опять же выбрал те, которые не меняются и не совпадают с другими устройствами. + я это делал для себя, поэтому достаточно скрупулезно изучал вопрос. |
Здравствуйте! Мне очень понравился Ваш проект. Во время внедрения в своё приложение, я пересмотрел некоторые реализованные Вами способы для большей оптимизации и уменьшения размера итогового приложения.
Вот несколько моих изменений:
The text was updated successfully, but these errors were encountered: