Skip to content

Conversation

opaduchak
Copy link
Collaborator

@opaduchak opaduchak commented Jul 15, 2025

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
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

Copy link
Contributor

@cslzchen cslzchen left a comment

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

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
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
3 participants