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

Go Backend #13

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Go Backend #13

wants to merge 11 commits into from

Conversation

WillFantom
Copy link

Why?

So, the reason for switching to a GoLang backend is extensibility. This provides a better base for quickly adding features. To demonstrate this, I have added another page at the /admin endpoint that takes a password just as an example.
Furthermore, this can now run as a standalone binary, not requiring the use of an apache or Nginx server, making it far more simple to instantiate. But of course, the docker file provided also allow it to run easily atop the docker engine.
More features can be added to this, however, to ensure the pull request is focused, I have simply left the login page there as an example, thus, it does not provide any real benefit in its current format.
As a final benefit, the parsing of the JSON config is done statically, improving performance slightly.

More...

The scope of this is limited to the change of language, not new features. I do intend to push and merge other features that make use of the new backend program.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant