Jack

> What about the following case: > > Player uses the console command notarget. player.GetNoTarget() -> true > Player uses phase shift. player.GetNoTarget() -> true > Player exists phase shift....

My PR is based on your PR, so if you merge mine itll appear here as a commit

Just tested, can confirm that this PR both fixes the bug mentioned above, as well as not removing `notarget` from a player when it shouldn't

The error clients get is the Invalid/Expired Masterserver Token

More details on the errors that we get `[error] Northstar origin authentication failed [error] {"success":false,"error":{"enum":"STRYDER_RESPONSE","msg":"Couldn't parse stryder response"}} [error] Failed reading masterserver response: got fastify error response [error] {"success":false,"error":{"enum":"INVALID_MASTERSERVER_TOKEN","msg":"Invalid or...

The error seems to be happening in one of two places: https://github.com/R2Northstar/NorthstarMasterServer/blob/main/client/clientauth.js#L64 or https://github.com/R2Northstar/NorthstarMasterServer/blob/main/client/clientauth.js#L74 So we are either failing to parse the JSON that we get back from stryder (unlikely...

Since my PR has been merged, the error I am getting has changed from `[error] Northstar origin authentication failed [error] {"success":false,"error":{"enum":"STRYDER_RESPONSE","msg":"Couldn't parse stryder response"}}` to `[error] Failed reading origin auth...

Ok so, basic reading and writing is functional (kinda), clients dont send mod data when they auth with themselves, so I can't send them any pdiff data Also a good...

Current progress: reading and writing basic pdiff is functional, however I am yet to split the pdiff from the rest of the pdata in a way that conserves the remaining...

pdiff now seems to be functional, for pdiffs that do not add to enums and stuff like that, idk where its going wrong in that regard basically pdiffs like this...