Elixir Phoenix variabile globale plug

Question

Sto cercando di ottenere l'identificatore di un sito Web in base al suo dominio, ma dopo averne scritto uno, sto correndo in un problema in cui tutti i link nel sistema stanno restituendo il contenuto dell'URL di root .

lib/frontend/spine/request_var.ex

defmodule Myapp.Plug.RequestVar do 
    import Plug.Conn 

    @doc false 
    def init(default), do: default 

    @doc false 
    def call(conn, router) do 
    host = conn.host 
    if host == "ll.com" || host == "domain1.com" do 
     slug = "domain1" 
    else 
     slug = "domain2" 
    end 

    conn |> put_private(:site_slug, slug) 
    end 
end 

in lib/frontend/endpoint.ex

plug Myapp.Plug.RequestVar, Myapp.Router 
plug Myapp.Router 

sia qualcosa di sbagliato con questo plug?

Modifica: Risolto il problema "se" in base alle risposte.

Answer 1

Il url è generato dal vostro endpoint.url e non host per il Plug.Conn

Da https://github.com/phoenixframework/phoenix/blob/8fe0538fd7be2adb05e2362b02fa8bd6bf3c6c46/lib/phoenix/router/helpers.ex#L13:

def url(_router, %Conn{private: private}) do 
    private.phoenix_endpoint.url 
    end 

    def url(_router, %Socket{endpoint: endpoint}) do 
    endpoint.url 
    end 

    def url(_router, %URI{} = uri) do 
    uri_to_string(uri) 
    end 

    def url(_router, endpoint) when is_atom(endpoint) do 
    endpoint.url 
    end 

È possibile ignorare questo utilizzando struct_url/0:

struct_url = update_in(Endpoint.struct_url.host, fn (_) -> "domain2" end) 
some_url(struct_url, :index) 

È inoltre possibile definire un secondo endpoint per il secondo dominio. Se i tuoi collegamenti sono interni, dovresti prendere in considerazione l'utilizzo delle funzioni _path invece delle funzioni _url. Gli helper _url vengono generalmente utilizzati quando è richiesto il dominio (come le e-mail).

Answer 2

Si è verificato un errore nella clausola if. Sarà sempre true.

iex(1)> host = "l2.com" 
"l2.com" 
iex(2)> host == "ll.com" || "domain1.com" 
"domain1.com" 

Per domini validi e non validi.

iex(3)> host = "ll.com"     
"ll.com" 
iex(4)> host == "ll.com" || "domain1.com" 
true 

prova:

iex(6)> if host == "ll.com" || "domain1.com" do 
...(6)> IO.puts "if" 
...(6)> end 
if 
:ok 

Devi cambiare la tua clausola if host == "ll.com" || host == "domain1.com" do. Ma. Non è idiota usare questo tipo di clausole. In genere è meglio usare pattern-matching.