CCF icon indicating copy to clipboard operation
CCF copied to clipboard

Follow-ups for redirection support

Open eddyashton opened this issue 1 year ago • 0 comments

#6050 provides a working implementation of forwarding, but there's a few things that I've delayed for future PRs:

  • [x] Removing the custom is_create flag. I think the special /create endpoint can be given appropriate flags so that it is always executed locally, and doesn't need a custom flag. #6096
  • [x] Revisit fallback to local execution. This was added to the redirection path for parity with the forwarding path. I believe hope it can be removed, but we may currently depend on it during unit tests or early-lifecycle execution. #6097
  • [x] Clarify interaction of set_forwarding_required and set_redirection_strategy. We currently derive the latter from the former, need to make sure we do that safely, as we move towards deprecation of the former.
  • [x] Improve documentation. The auto-generated host schema docs don't give much detail on the new fields, need to expand what they show. #6095
  • [ ] Add to_backup forwarding option. No explicit demand for this, but should be easy to add for balance.
  • [x] Add sample of JS app configuration.

eddyashton avatar Mar 21 '24 11:03 eddyashton