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

Why is the newtonsoft.json.dll modified? #128

Closed
5 tasks done
jobrien78 opened this issue Sep 6, 2022 · 7 comments
Closed
5 tasks done

Why is the newtonsoft.json.dll modified? #128

jobrien78 opened this issue Sep 6, 2022 · 7 comments
Labels
bug Something isn't working

Comments

@jobrien78
Copy link

Prerequisites

  • Verify that this is not a Windows issue;
  • Refer to the System requirements;
  • I do not use a homebrew Windows image;
  • I did not tweak Windows before that could cause system instability;
  • Attach a created log. You may find it in the Logs folder.

Steps To Reproduce

Hi, I had a look at the dlls and in particular newtonsoft.json.dll which comes packed with the app and am finding its hash doesnt match the one that is supposedly have been downloaded from the official JamesNK/Newtonsoft.Json repo during the build process (

Uri = "https://api.github.com/repos/JamesNK/Newtonsoft.Json/releases/latest"
UseBasicParsing = $true
}
$NewtonsoftJsonLatestVersion = ((Invoke-RestMethod @Parameters).assets | Where-Object -FilterScript {$_.browser_download_url -match "Json"}).browser_download_url
)

image

Can you explain why the dll is modified?

Also why isn't this dll at least be downloaded via NuGet?

Expected behavior

N/A

Actual behaviour

N/A

Windows Version

Win 10

SophiApp version

Current

SophiApp release type

SophiApp

Logs

N/A

@jobrien78 jobrien78 added the bug Something isn't working label Sep 6, 2022
@farag2
Copy link
Member

farag2 commented Sep 6, 2022

Hm... I/we do not modify any of DLLs. Everything is automated. Let me check...

@farag2
Copy link
Member

farag2 commented Sep 6, 2022

Yeah, they're really not equal. But how?.. O_O

изображение

@farag2
Copy link
Member

farag2 commented Sep 6, 2022

All that I understood is that DLL in the SophiApp folder is equal to the Nuget one, but not equal to the GitHub one at the same time. Also the GitHub dll isn't equal to the Nuget dll. They even have different timestamp.

@jobrien78
Copy link
Author

I haven't tried the one from NuGet, but running your get binary script I am also able to get the expected hash which should be in the SophiApp.

@farag2
Copy link
Member

farag2 commented Sep 6, 2022

I will ask my colleague tomorrow, how it could happen when we download one dll but get another after compilation. I'm really puzzled.

@jobrien78
Copy link
Author

OK, thank you.

@farag2
Copy link
Member

farag2 commented Sep 11, 2022

Fixed with #131

@farag2 farag2 closed this as completed Sep 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants