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

set missing env vars #392

Merged
merged 1 commit into from
Aug 19, 2020
Merged

set missing env vars #392

merged 1 commit into from
Aug 19, 2020

Conversation

butonic
Copy link
Member

@butonic butonic commented Jul 16, 2020

REVA_FRONTEND_URL and REVA_DATAGATEWAY_URL need to be set to the OCIS_DOMAIN so upload is not using localhost ;-)
cc @jnweiger

@update-docs
Copy link

update-docs bot commented Jul 16, 2020

Thanks for opening this pull request! The maintainers of this repository would appreciate it if you would create a changelog item based on your changes.

@butonic butonic self-assigned this Jul 16, 2020
@butonic butonic added the bug label Jul 16, 2020
Copy link
Contributor

@jnweiger jnweiger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Definitly correct in this docker-compose.yml.

@felixboehm please double-check, if needed elsewhere (compose-playground?) too!

Copy link
Member

@kulmann kulmann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Both URLs are supposed to be reachable from outside, so the port 9200 should only be on the localhost fallback, not on the OCIS_DOMAIN

@felixboehm
Copy link
Contributor

felixboehm commented Jul 17, 2020

fine if it works. I don’t understand these configs...
showing clearly the design flaws in our architecture, ugh.

we need to update in

REVA_FRONTEND_URL and REVA_DATAGATEWAY_URL need to be set to the OCIS_DOMAIN so upload is not using localhost ;-)
cc @jnweiger

Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
@butonic butonic merged commit db6a384 into master Aug 19, 2020
@phil-davis phil-davis deleted the add-missing-env-vars branch August 19, 2020 14:29
refs pushed a commit that referenced this pull request Sep 18, 2020
Signed-off-by: Jörn Friedrich Dreyer <[email protected]>
arm build failures unrelated
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

4 participants