Open joshtriplett opened 2 years ago
Should this be a cargo option? Or do you think it's better on Config
like you have in this PR?
@Fishrock123 I definitely don't think it should be a cargo feature; that'd be backwards-incompatible for anyone not expecting that behavior, including other users elsewhere in the dependency tree that didn't set that feature.
I think it makes sense to set this on a config-by-config basis.
Add
Config::set_mandatory_base_origin()
to enable this behavior. Useful if theConfig
includes authentication headers or similar.