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

Add reverse proxy for /admin #102

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

Conversation

tushar5526
Copy link
Contributor

Description

Reverse proxy setup to deploy admin app on /admin endpoint from 3000 port.

Changes

Added a proxy service in compose, which I have commented out to not hinder local development

PS: Added a base prefix of /admin for a production run to serve static files from /admin

@tushar5526
Copy link
Contributor Author

The Docker build was somehow taking a lot of time on my local setup so I tested things on the server. @ChakshuGautam please check if this was the intended behaviour http://128.199.26.225/admin

@tushar5526
Copy link
Contributor Author

Also as a side note, if you would start the admin in prod mode then it will not work fine on port 3000 because of the added assetPrefix. In dev mode, it will be fine.

See the console for this URL: http://128.199.26.225:3000/

@tushar5526
Copy link
Contributor Author

yarn start and yarn build locally will not work due to the assetPrefix addition

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