🌐 AI搜索 & 代理 主页
Skip to content

Non-breaking workaround for BinaryFormatter deprecation, issue #2469 #2621

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.

Sign up for GitHub

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

Closed
Legion-ale wants to merge 2 commits into from
Closed

Non-breaking workaround for BinaryFormatter deprecation, issue #2469 #2621

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
50752e8
Non-breaking workaround for BinaryFormatter deprecation, issue #2469
Legion-ale Sep 23, 2025
f144d37
Update RuntimeData.cs
Legion-ale Sep 30, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/runtime/StateSerialization/NoopFormatter.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,11 @@
namespace Python.Runtime;

public class NoopFormatter : IFormatter {
public object Deserialize(Stream s) => throw new NotImplementedException();

public object Deserialize(Stream s) => throw new NotImplementedException(
"Cannot deserialize using 'NoopFormatter', implement 'Python.Runtime.RuntimeData.FormatterType' if needed"
);

public void Serialize(Stream s, object o) {}

public SerializationBinder? Binder { get; set; }
Expand Down
27 changes: 26 additions & 1 deletion src/runtime/StateSerialization/RuntimeData.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,41 @@ namespace Python.Runtime
{
public static class RuntimeData
{
//TODO: put it in a utility class instead of inside the RuntimeData Class.
private static Version? GetNetVersion(string verToCheck)
{
// Examples:
// ".NET 8.0.0"
// ".NET 9.0.0"
// ".NET Framework 4.8.1"
// ".NET Core 3.1.32"
string desc = RuntimeInformation.FrameworkDescription;
if (desc.StartsWith(verToCheck, StringComparison.OrdinalIgnoreCase))
{
var verStr = desc.Split(' ').Last();
if (Version.TryParse(verStr, out var ver))
return ver;
}
return null;
}

public readonly static Func<IFormatter> DefaultFormatterFactory = () =>
{
try
//The issue here is that just instancing BynaryFormatter doesn't solve the problem.
//you need to use it to generate the exception.
/*try
{
return new BinaryFormatter();
}
catch
{
return new NoopFormatter();
}*/
var ver = GetNetVersion(".NET");
if(ver != null && ver.Major >= 9) {
return new NoopFormatter();
} else {
return new BinaryFormatter();
}
};

Expand Down
Loading