-
-
Notifications
You must be signed in to change notification settings - Fork 360
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
Changing SendBufferSize for sockets #441
Comments
Offtopic: when do we get a new release to include all the fixes and such that have been merged since May? Possibly an automated nightly build even? |
GetConfigValue confirms that it is not set globally, at least not to the right value. |
@rlabrecque any clue? I've tried allocating memory with |
GetConfigValue is invalid. According to docs: https://partner.steamgames.com/doc/api/ISteamNetworkingUtils#GetConfigValue ESteamNetworkingGetConfigValueResult GetConfigValue(
ESteamNetworkingConfigValue eValue,
ESteamNetworkingConfigScope eScopeType,
intptr_t scopeObj,
ESteamNetworkingConfigDataType *pOutDataType,
void *pResult,
size_t *cbResult); cbResult: IN: the size of your buffer. OUT: the number of bytes filled in or required Which means cbResult should be |
There's still some issue though likely, because I am setting the value, when I use GetConfigValue it is correct (~1 MB), but when I try to connect and send data it says that: src/steamnetworkingsockets/clientlib/steamnetworkingsockets_connections.cpp (1915) : Assertion Failed: Message size 732253 is too big. Max is 524288 |
Hey there, I haven't had a chance to look more indepth and honestly it might be a few weeks still, I suspect there's something weird going on here:
We'll probably need something like this around here: https://docs.microsoft.com/en-us/dotnet/api/system.runtime.interopservices.marshal.structuretoptr?view=net-5.0 |
Hey, thank you for taking a look. On the other side, with fixed Get method I managed to set it (and get returns the right value), but Steam still shows the message above. Do you think Steam could be reading the value wrong due to what you've mentioned? |
Interesting how with new Get method it all is set properly for connection, socket and global. BUT I am still getting this: |
Fix ISteamNetworkingUtils_GetConfigValue parameter from out -> ref (#441)
Hmm are you still setting the This /may/ help too, but I haven't dugg in deeper yet |
Using an empty array for options atm, it returns OKInherited and the right value for connection. Could try options if you think that would change anything. |
Tested options array, just using an array does change the listener options so now it returns OK instead of OKInherited and a value I've provided. |
Steam support have responded to me:
|
SEE LAST COMMENT
I am trying to set SendBufferSize for my Steam networking sockets in two ways. First would be global options and second the parameters you pass to CreateListenSocketIP:
But neither of the config helpers seem to work for me - the limit stays the default (512 * 1024). How may I debug the issue and could it be my values being GCed or something to do with pointers? Not sure.
The text was updated successfully, but these errors were encountered: