fix(server) - upload media handling http error

[dnkbln]

Bei großen Uploads kam es zu Mint:Http Fehlern. Ich umgehe das Problem jetzt in dem ich für die Media Uploads vorsorglich das Protokoll auf http1 umstelle.

[eteubert]

Zwei Stil-Anmerkungen:

1. In den Req.new calls ist viel Dopplung. Ich würde da nur die connect_options abhängig vom http1 Parameter anders bauen. So in der Art:

default_connect_options = [transport_opts: [verify: :verify_none]]

connect_options = if http1 do 
  Keyword.put(default_connect_options, :protocols, [:http1])
else
  default_connect_options
end

Req.new(
  base_url: site <> "/wp-json/",
  headers: [{"Content-Type", "application/json"}],
  auth: {:basic, user <> ":" <> password},
  connect_options: connect_options
)

2. Unbenannte Parameter, vor allem booleans, sind in Elixir unüblich. Man sieht beim Aufruf nicht, was der Parameter macht. Üblich sind keyword lists, auch schon wenn es nur eine einzige Option ist. API.new(headers, http1: true) ist verständlicher als API.new(headers, true). Kannst du auch optional machen, dann musst du den Parameter von außen auch nur setzen, wenn du ihn brauchst.

  def new(headers, opts \\ []) do
    http1 = Keyword.get(opts, :http1, false)
    # ...