"Breaking" change?
An option has been removed from the config, which apparently was never working. So this should not break anyone's IPP as the previous code never functioned:
Previously there was an option in IPP when serving from multiple domains, that you could "instead of setting the public URL in your docker-compose file, you can set it dynamically via a HTTP header in the request from your reverse proxy to IPP."
Thanks to @mFIND #234 for pointing it out - the code had a camelCase key, but NodeJS forces all keys to lowercase. This means that the existing code would have been non-functional.
Due to the cache poisoning risk pointed out by @mFIND #234 and since apparently no one was using this functionality, rather than fix the issue it has been removed entirely 79346d9
Fixes
- #230 slug album not downloading correctly
- #233 unable to unlock password protected link with cross-site context (thanks @MDornacher)
- Update the zip-creation code to match the way Immich does it, which seemed like a sensible idea