fix(ReverseProxy): handle forwarding

[Betree]

No description provided.

[sourcery-ai]

Hey - I've left some high level feedback:

• Consider turning service_subdomain_regex/0 into a module attribute (e.g. @service_subdomain_regex) to avoid recompiling the same regex on every call to routing_host/1 and get_domain_from_host/1.
• In routing_host/1, you currently prefer conn.host whenever it matches the service subdomain regex; if an internal host ever accidentally matches that pattern (e.g. rest.internal.local), x-forwarded-host will be ignored—consider tightening the condition or explicitly preferring a valid x-forwarded-host when present and different from conn.host.

Prompt for AI Agents

Please address the comments from this code review:

## Overall Comments
- Consider turning `service_subdomain_regex/0` into a module attribute (e.g. `@service_subdomain_regex`) to avoid recompiling the same regex on every call to `routing_host/1` and `get_domain_from_host/1`.
- In `routing_host/1`, you currently prefer `conn.host` whenever it matches the service subdomain regex; if an internal host ever accidentally matches that pattern (e.g. `rest.internal.local`), `x-forwarded-host` will be ignored—consider tightening the condition or explicitly preferring a valid `x-forwarded-host` when present and different from `conn.host`.

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}