Uchu icon indicating copy to clipboard operation
Uchu copied to clipboard

New Verify API | Changed "account/list"

Open ArkShocer opened this issue 3 years ago • 3 comments

These changes add the functionality to create a web-app for users / administrators to login and manage the servers.

Tasks:

  • [x] Add API to verify password
  • [x] Change output of account/list call usernames instead of ids
  • [ ] Add API Key in Header to instances
    • [ ] Instances can't connect to master server due to missing api key in header
  • [x] Deny access to api calls without key
  • [x] Exception for /verify
    • [x] Respond with 401 if no key
    • [x] Verify Response
      • [ ] Respond with key from config.xml
  • [x] Add New XmlElement Key to UchuConfiguration
    • [x] Add Key from config.xml to ApiManager.cs
    • [ ] Add Key from config.xml to AccountCommands.cs
  • [x] Set Access-Control-Allow-Origin, * for all responses
  • [ ] Fix https protocol not working
    • [ ] All instances stop working if changing http to https in config.xml

ArkShocer avatar Jan 06 '22 16:01 ArkShocer

What is the justification/use case for these changes?

TheNexusAvenger avatar Jan 07 '22 23:01 TheNexusAvenger

I think the verify API could be handy for logging into a web panel like the one Darkflame has.

teccheck avatar Jan 11 '22 12:01 teccheck

As @TecCheck mentioned, this API change adds the ability to verify user credentials over http which can then be used to create a web based interface for users/administrators to login to and see their stats/manage the server.

ArkShocer avatar Jan 14 '22 13:01 ArkShocer