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

Modify UI to avoid CORS issues when running production build #89

Draft
wants to merge 1 commit into
base: edge
Choose a base branch
from

Conversation

btmorr
Copy link
Owner

@btmorr btmorr commented Jul 7, 2020

#86 added a docker-compose yml file that successfully starts a 3-node cluster and a container for the UI. The UI is able to connect to nodes, but does not handle CORS or redirects correctly, so it can only write when actually connected to the current Leader.

When merged, this PR will:

  • Modify the UI app to include a node app for serving the prod build (expressly for use in docker-compose, but will be generally useful)
  • Remove the connect logic from the Admin page of the UI, and make this part of app launch configuration instead (using the same environment variable system for now that the database uses to determine configuration)
  • Modify Dockerfiles and docker-compose.yml in keeping with UI changes

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.

1 participant