Skip to content
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

Account system #14

Merged
merged 3 commits into from
Aug 29, 2024
Merged

Account system #14

merged 3 commits into from
Aug 29, 2024

Conversation

seionmoya
Copy link
Collaborator

@seionmoya seionmoya commented Aug 29, 2024

It's quite a big change and does a couple of things:

  • Accounts store login info, game saves
  • Session info binds to accounts
  • Session info can be obtained from PHPSESSID cookie in a request
  • Using session info you can obtain the account info

This PR also comes with EFT profile wipe functionality;

  • A profile marked for wipe will be reset on first startup of EFT

Due to the changes above, it became significantly easier to decouple some request from embedded data. This will make updating the project easier.

Note

Future changes

A login system still has to be implemented. On successful login, two options:

  • The server should return session info to launch the game with.
  • The launcher also sends the session indo to use when logging in, and gets stored on successful login.

A logged in account should also be able to mark a save for wipe, including edition selection.

Changes to namespace

I moved some code around in order to support other services later on, like Arena and Fuyu internal server logic.

@seionmoya seionmoya added the enhancement New feature or request label Aug 29, 2024
@seionmoya seionmoya requested a review from nexus4880 August 29, 2024 10:24
@seionmoya seionmoya self-assigned this Aug 29, 2024
@seionmoya seionmoya changed the title [WIP] Account system WIP: Account system Aug 29, 2024
@seionmoya seionmoya changed the title WIP: Account system Account system Aug 29, 2024
@seionmoya seionmoya merged commit f314b37 into project-fika:main Aug 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants