search

stijnwop / guidanceSteering

Guidance Steering (AutoTrack) for Farming Simulator 22.
381 stars 94 forks source link

[BUG] A+Direction locks up Client and Dedi #317

Closed ghost closed 2 years ago

ghost commented 2 years ago

Using A+Direction on a dedicated server locks up the game and the server freezes.

On server setting A+Direction opens the A+Heading "desired cardinal" window. Confirming the cardinal will lock the control of the Client and lag out the server.

Changing the line method using the ">" button works correctly but in the other direction it seems to mess up. As also A+Direction has a "Set B" button. In Singleplayer "Set B" doesn't have any effect and "Set A" work as intended and orients the track with the vehicle. On Dedi it locks the client controls and may lock or even crash the server.

Client Log Error: Error: Running LUA method 'mouseEvent'. C:/Users/USER/Documents/My Games/FarmingSimulator2022/mods/FS22_guidanceSteering/src/misc/ABPoint.lua:88: attempt to index a nil value Warning: Could not decrypt received packet.

Server Log Error: (Spammed) If the bugged player is online: Warning: Could not decrypt received packet.

If bugged player disconnects: Warning: Send called with unknown target address.

yrots300 commented 2 years ago

I have the same problem yesterday. After the update it starts freezing the server. We lost about 2h of gameplay there :/ I went to GPS settings and try to set A+direction, I am using it frequently, the box, where I sould write the direction stays on the screen. I was pressing ESC to cancel that, but nothing happend. Then I hold the ESC buttong and the game went to ESC, bud the GPS diagram stays there. After while, the GPS went away, but the server freezed. Also after sereting the server, some errors occured and I have to restart whole server, not just the game.

I would really appreciate, if you fix that ASAP or load the old working version back to modhub. But also I really appreciate this mod!

stijnwop commented 2 years ago

Pushed a fix that should avoid the client sided spam when using the new direction strategy. I tested it on a host + client MP setup, could you please verify it on dedi?

ghost commented 2 years ago

Ok. Seems to work now and we tried hard. Only thing thats still not working 100% is if you use the the left arrow to change the Line Method then changing vom A+B to A+Direction still displays "Set A" and "Set B" in the A+ Direction menu, using right arrow button only shows "Set A" (Possibly displaying what it showed last?)

stijnwop commented 2 years ago

Good catch will do a fix for that 👍

stijnwop commented 2 years ago

Resolved that little issue too @711Soul