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

astore: Split out auth main.go #966

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mhallmark
Copy link
Contributor

Previous PR moved the bulk of the files from astore to auth directory with the intention of splitting out the auth server from the astore binary.

Some files were moved to the auth directory that astore still needed like assets and templates, so those directories were copied back to astore. This makes duplicate auth/server/assets and astore/server/assets directories. Some of these assets may be removed from either directory if they arent needed.

The main part of this PR creates a new main.go under auth/server so auth and astore can be deployed separately.

Testing - neither of these binaries have been fully tested. I was able to get the astore server running locally, but wasn't able to start auth (was missing some flag values).

Previous PR moved the bulk of the files from astore to auth directory
with the intention of splitting out the auth server from the astore
binary.

Some files were moved to the auth directory that astore still needed
like assets and templates, so those directories were copied back to
astore. This makes duplicate auth/server/assets and astore/server/assets
directories. Some of these assets may be removed from either directory
if they arent needed.

The main part of this PR creates a new main.go under auth/server so auth
and astore can be deployed separately.

Testing - neither of these binaries have been fully tested. I was able
to get the astore server running locally, but wasn't able to start auth
(was missing some flag values).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants