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

Microsoft.Azure.Cosmos.Table 1.0.5 brings in Newtonsoft 10.x, and Linqpad 7 pops up a Warning #22

Closed
Noctis- opened this issue Oct 12, 2022 · 7 comments

Comments

@Noctis-
Copy link

Noctis- commented Oct 12, 2022

The specific warning is :

Warning: LINQPad has identified the following vulnerabilties in NuGet package Newtonsoft.Json 10.0.2:

Level 2 (High): https://github.com/advisories/GHSA-5crp-9r3c-p9vr
Try updating NuGet packages to latest. You can suppress this warning in Edit | Preferences > Advanced > Execution.

I've tried pulling the code and updating newtonsoft to version 13, but, i have no idea how to build / publish / test this :(

There's a newer version of the Microsoft.Azure.Cosmos.Table (1.0.8) , which it fails to install (doubt it'll fix the issue though, as that vulnerability was fixed in the newtosoft v 13, which was released in Mach '21, and the 1.0.8 was done in august '20).

The whole Microsoft.Azure.Cosmos.Table is deprecated as well ...
not sure if it's an easy fix, or worth fixing, but figured i'll post it in case you do want to give it a go .

(oh, and cheers for you work BTW, regardless 🙏 )

@madd0
Copy link
Owner

madd0 commented Oct 12, 2022

@Noctis- thanks for the heads up. I'll look into it.

@Noctis-
Copy link
Author

Noctis- commented Oct 12, 2022 via email

@Noctis-
Copy link
Author

Noctis- commented Nov 13, 2022

Any thoughts / updates ? (asking from pure curiosity, as i've seen something about forcing a newer version of a package by including it in the csproj. I tried doing it, but i have no idea how to make linqpad use the dll, as it only accepts linqpad drivers and stuff)

@madd0
Copy link
Owner

madd0 commented Nov 13, 2022

Hi,
I looked into it (just now 😅) and the short-term work-around is simply reference a more recent version of Json.NET in your query's references and properties (F4).
I'll look into updating the Nuget package so that it includes a version without the vulnerability, but it looks like I'm quite a bit behind, so there's a bit of work before I can publish that.
I'll keep the issue open while I work on it.

@Noctis-
Copy link
Author

Noctis- commented Nov 13, 2022 via email

@madd0
Copy link
Owner

madd0 commented Nov 13, 2022

v2.1.2 has been published to Nuget, it includes the latest version of Json.Net

Thanks

@madd0 madd0 closed this as completed Nov 13, 2022
@Noctis-
Copy link
Author

Noctis- commented Nov 14, 2022 via email

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

No branches or pull requests

2 participants