asherkin / connect

https://forums.alliedmods.net/showthread.php?t=162489
14 stars 7 forks source link

[Any/all] No detours getting hit + OnClientPreConnectEx not firing #19

Closed DARKSIDE99199 closed 6 months ago

DARKSIDE99199 commented 6 months ago

After the new TF2 update, this extension causes an error in the console: [S_API FAIL] Tried to access Steam interface SteamNetworkingUtils 004 before SteamAPI_Init succeeded.

The server continues to work, but when connecting, the error "Steam validation rejected" appears

sapphonie commented 6 months ago

40 dollar paypal bounty on anyone who wants to PR something that fixes this 👍

asherkin commented 6 months ago

Seems likely this is a change to the layout of CSteam3Server? Not sure why I never put that in gamedata, it was in SteamTools2...

sapphonie commented 6 months ago

Not resolved by #21, still an issue.

sapphonie commented 6 months ago

Info I've found:

Natanel-Shitrit commented 6 months ago

Info I've found:

  • None of the detours are getting hit in gdb
  • Extension loads fine, throws no errors in OnLoad, returns true

Have you checked if it works after commit fa6bf9c44ca8a10bcb292860f3b54aa6e0f02dc6? It changed the AMBuildScript to use CDetours from SM instead of the included one. (no idea if it really fixes anything)

sapphonie commented 6 months ago

Info I've found:

  • None of the detours are getting hit in gdb
  • Extension loads fine, throws no errors in OnLoad, returns true

Have you checked if it works after commit fa6bf9c44ca8a10bcb292860f3b54aa6e0f02dc6? It changed the AMBuildScript to use CDetours from SM instead of the included one. (no idea if it really fixes anything)

Yes, this was on a recent commit, double check me though if you'd like in case I'm misremembering...

Natanel-Shitrit commented 6 months ago

Info I've found:

  • None of the detours are getting hit in gdb
  • Extension loads fine, throws no errors in OnLoad, returns true

Have you checked if it works after commit fa6bf9c? It changed the AMBuildScript to use CDetours from SM instead of the included one. (no idea if it really fixes anything)

Yes, this was on a recent commit, double check me though if you'd like in case I'm misremembering...

I have no TF2 setup 😢 It fixed a crash in #21 I thought it might fix other things here too

khalik38 commented 6 months ago

40 dollar paypal bounty on anyone who wants to PR something that fixes this 👍

If someone fixes this, I shall contribute towards this.

No detours getting hit + OnClientPreConnectEx not firing

sapphonie commented 6 months ago

@Kenzzer