Skip to content

[ENG-8370] Fix for local setup#385

Closed
opaduchak wants to merge 1 commit into
CenterForOpenScience:feature/buff-wormsfrom
opaduchak:fix/local_setup
Closed

[ENG-8370] Fix for local setup#385
opaduchak wants to merge 1 commit into
CenterForOpenScience:feature/buff-wormsfrom
opaduchak:fix/local_setup

Conversation

@opaduchak

@opaduchak opaduchak commented Jul 15, 2025

Copy link
Copy Markdown
Collaborator

Ticket

https://openscience.atlassian.net/browse/ENG-8370

Purpose

Fix MFR not being able to render files on webpage in docker-compose

Changes

Added host substitution in osf provider to silently forward all requests from localhost to 192.168.168.167

Side effects

None, but local config should contain a host to point to

QA Notes

Deployment Notes

@opaduchak opaduchak changed the title impplemented fix for local domain [DRAFT] Fix for local setup Jul 15, 2025
@opaduchak opaduchak changed the title [DRAFT] Fix for local setup [ENG-8370] Fix for local setup Jul 15, 2025
MFR_IDENTIFYING_HEADER = config.get('MFR_IDENTIFYING_HEADER', 'X-Cos-Mfr-Render-Request')
MFR_ACTION_HEADER = config.get('MFR_ACTION_HEADER', 'X-Cos-Mfr-Request-Action')

OSF_DOMAIN = config.get('MFR_OSF_DOMAIN', None)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

None should be changed to '192.168.168.167' locally

@cslzchen cslzchen left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A nice fix. I will test it locally after I get pass WB first. For now, I will defer to @felliott to handle the merge.

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 73.07% (-0.09%) from 73.156%
when pulling 7cc6565 on opaduchak:fix/local_setup
into 7ca4004 on CenterForOpenScience:feature/buff-worms.

@opaduchak opaduchak closed this Jul 18, 2025
@opaduchak

Copy link
Copy Markdown
Collaborator Author

Closed as CenterForOpenScience/osf.io#11233 provides better solution

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.

3 participants