Nit: I have a minor preference against nested resources\u2013e.g. `xblock/{usage_key_str}` for the OLX and `file/{usage_key}/<path>`. Mostly because our usage key parsing is a bit wacky.    Not strongly attached to this however, and I'm fine with the approach you've taken.