Repository navigation
Conversation
Proposal pending the owner's decision. - New "Unofficial components" setting, off by default. Turning it on needs a plain-language warning confirmed. While it is off, the DLSSNR runtime and MFG Unlock are never installed or updated, and MFG Unlock isn't even looked up on GitHub. - Existing users: settings from 1.5.x are migrated once. Anyone who already has either component (imported DLSSNR, imported or downloaded MFG Unlock) keeps it on. Everyone gets a one-time notice. Opting out never removes files from games, and an installed DLSSNR runtime stays enabled in OptiScaler.ini. - Remove the SHA-256 table that whitelisted the NVIDIA original and a modified DLSSNR build. Replace it with an Authenticode check: the file must carry a valid NVIDIA Corporation signature, anything else shows as unverified. - Reword "safe with anti-cheat" claims in the UI and README, and stop pointing at the modified compat build. - docs/NAMING.md: five name candidates without NVIDIA marks, checked against winget, GitHub and the web.
Owner
Author
|
Closing: this app is mainly for the owner's own use, and they want DLSS 5 / MFG Unlock and the known-build hash detection to keep working exactly as they do now. The branch is kept for reference. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Two features invite a DMCA or trademark complaint, and they're the most likely reason for a takedown or a store rejection:
nvngx_dlssnr.dll). It also shipped a SHA-256 table that whitelisted a modified NVIDIA build, which reads as the app endorsing a tampered NVIDIA binary.The app never redistributed either file, and that stays true. This PR goes further: the app no longer pushes either one, and the claims are now accurate.
What changes
1. Opt-in, off by default
Existing users
SettingsVersion0 → 2).nvngx_dlssnr.dll, or an imported or downloaded MFG Unlock.OptiScaler.ini. Uninstall still removes everything.2. Hash table removed, replaced by an Authenticode check
ComponentStore.KnownDlssNris gone.Core/Authenticode.csruns WinVerifyTrust (offline, no revocation fetch) and requires the signer to beNVIDIA Corporation.3. Wording
4. docs/NAMING.md
Tests
dotnet buildanddotnet test: 77/77 pass (56 existing + 21 new), Debug and Release. New tests:UnofficialComponentsTests:NVIDIA Corporationcounts as the signer.InstallerTests.Update_OptedOutOfUnofficial_LeavesInstalledOnesWorking.I also launched the app with fresh app data. It starts normally, and the log shows "MFG Unlock off". The Settings layout was not checked visually.
Overlap with #4 and #5
Both merge cleanly with this branch, with no textual conflicts. A test merge of this branch + #5 + #4 builds, and all 118 tests pass. I left one
AppSettingscomment unchanged ("OptiScaler-NR (the main purpose)") because it sits inside #5's diff context. It can be reworded after #5 merges.Not in this PR (audit follow-ups)
nvngx_dlss*.dll/sl.*.dll