NetworkMessagesSessionRequest event expects a long argument instead of a ulong

[Avantir-Chaosfire]

Describe the bug
Using the .NET 6 version of GodotSteam, the NetworkMessagesSessionRequest is expecting a method with a long argument, but the value is a Steam ID, which should be a ulong. This means the data is corrupted and can't be used, rendering the event useless.

To Reproduce
In a C# project, hook up a method to the NetworkMessagesSessionRequest event. Notice that the argument type has to be long.

Expected behavior
The NetworkMessagesSessionRequest should expect and pass a ulong argument.

Screenshots

Desktop (please complete the following information):

• OS: macOS 14.5

Version of Godot:
4.2.2

Version of GodotSteam:
4.8, module

Additional context
Steamworks 159

[Gramps]

Hey there! We don't technically support C# currently or, more specifically, have no C# enabled stuff. How did you get the C# stuff working?

[Avantir-Chaosfire]

@Gramps Oh - I didn't realize. I just did a custom build of Godot with GodotSteam. The glue generated fine (aside from this issue). I haven't run the project yet, I'm still busy converting it to Godot 4 & fixing compilation errors.

[Gramps]

Oh, very cool! Have you checked out: https://github.com/LauraWebdev/GodotSteam_CSharpBindings These we made specifically for GodotSteam and may help.

[Avantir-Chaosfire]

@Gramps Yes, that worked! (well, for the build. It will take some time to establish if it works at runtime...) Also interestingly, when I generated the glue for GodotSteam, it generated with a string identityReference in Godot 3 and a ulong steamId in Godot 4 (for the P2P networking APIs). But that project you linked swapped it back to string identityReference in Godot 4. So I'll stick with that for now.

[Gramps]

Hmm, seems the string stuff is still in there then. It got changed recently so Laura may not have updated it for that then. Might want to put in an issue about it. Thanks for letting me know about all this!