Skip to content
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

refactor(bundler/nsis): replace nsis_process.dll usage with nsProcess.dll #11041

Closed
wants to merge 2 commits into from

Conversation

amrbashir
Copy link
Member

@amrbashir amrbashir commented Sep 18, 2024

nsis_process dll is written by us in https://github.com/tauri-apps/nsis-tauri-utils but anti virus tools picks it up as virus. Its code does as much as nsProcess but I guss we just don't have enough popularity as nsProcess has with anti-virus tools.

closes tauri-apps/nsis-tauri-utils#39
closes: tauri-apps/nsis-tauri-utils#37

`nsis_process` dll is written by us in https://github.com/tauri-apps/nsis-tauri-utils but anti virus tools picks it up as virus. Its code does as much as `nsProcess` but I guss we just don't have enough popularity as `nsProcess` has with anti-virus tools.

closes tauri-apps/nsis-tauri-utils#39
ref: tauri-apps/nsis-tauri-utils#37
@amrbashir amrbashir requested a review from a team as a code owner September 18, 2024 01:51
@amrbashir amrbashir changed the title refactor(bundler/nsis): replace nsis_process dll usage with nsProcess refactor(bundler/nsis): replace nsis_process.dll usage with nsProcess.dll Sep 18, 2024
Copy link
Contributor

github-actions bot commented Sep 18, 2024

Package Changes Through 1b45256

No changes.

Add a change file through the GitHub UI by following this link.


Read about change files or the docs at github.com/jbolda/covector

@Legend-Master
Copy link
Contributor

I don't like this to be honest, if this is a webroot only problem, can we contact them to white list us instead?

@amrbashir
Copy link
Member Author

I don't like it either but I am not sure how many more anti virus would have false positives.

Also weirdly enough, [email protected] passes all anti viruses https://www.virustotal.com/gui/url/e814a313d4ba77fd51c3d1408c3ba49329a52656a767df50bdcbb867bcf837f6?nocache=1

@amrbashir amrbashir marked this pull request as draft September 18, 2024 02:42
@amrbashir
Copy link
Member Author

Converting to a draft for now

@amrbashir amrbashir closed this Dec 8, 2024
@amrbashir
Copy link
Member Author

should be fixed with #11676 as well

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