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

Refactor 'dispatch' server/Update plugin authentication API #868

Merged
13 commits merged into from
May 14, 2022

Conversation

KingRainbow44
Copy link
Member

@KingRainbow44 KingRainbow44 commented May 13, 2022

Description

The very clean dispatch server rewrite.
Since the dispatch server can be accessed by plugins, its more of an HTTP server, hence the rename.
Additionally, an authentication system rewrite was put in place. (to clean it up and add more expandability for plugins)

Type of changes

  • Bug fix
  • New feature
  • Enhancement
  • Documentation

Checklist:

  • My code follows the style guidelines of this project
  • My pull request is unique and no other pull requests have been opened for these changes
  • I have read the Contributing note and Code of conduct
  • I am responsible for any copyright issues with my code if it occurs in the future.

@KingRainbow44 KingRainbow44 added enhancement New feature or request dispatch Related to the dispatch server breaking A breaking change to other PRs or plugins labels May 13, 2022
@KingRainbow44 KingRainbow44 marked this pull request as ready for review May 14, 2022 03:22
@ghost ghost merged commit 155a720 into development May 14, 2022
@KingRainbow44 KingRainbow44 deleted the plugin-auth branch May 15, 2022 23:19
Birdulon pushed a commit to Birdulon/Grasscutter that referenced this pull request Aug 21, 2022
Refactor 'dispatch' server/Update plugin authentication API
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking A breaking change to other PRs or plugins dispatch Related to the dispatch server enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants