Skip to content

Commit 9857d06

Browse files
committed
fix(server) - upload media handling http error
1 parent 925830e commit 9857d06

File tree

3 files changed

+27
-13
lines changed

3 files changed

+27
-13
lines changed

server/lib/publisher/wordpress/api.ex

+22-8
Original file line numberDiff line numberDiff line change
@@ -4,21 +4,35 @@ defmodule Publisher.WordPress.API do
44
55
## Example
66
7-
req = API.new(headers)
7+
req = API.new(headers, http1)
88
{:ok, response} = Req.get(req, url: "podlove/v2/episodes")
99
1010
"""
11-
def new(headers) do
11+
def new(headers, http1) do
1212
user = get_header_value(headers, "wordpress-user")
1313
password = get_header_value(headers, "wordpress-password")
1414
site = get_header_value(headers, "wordpress-site")
1515

16-
Req.new(
17-
base_url: site <> "/wp-json/",
18-
headers: [{"Content-Type", "application/json"}],
19-
auth: {:basic, user <> ":" <> password},
20-
connect_options: [transport_opts: [verify: :verify_none]]
21-
)
16+
if (http1) do
17+
Req.new(
18+
base_url: site <> "/wp-json/",
19+
headers: [{"Content-Type", "application/json"}],
20+
auth: {:basic, user <> ":" <> password},
21+
connect_options: [
22+
transport_opts: [verify: :verify_none]
23+
]
24+
)
25+
else
26+
Req.new(
27+
base_url: site <> "/wp-json/",
28+
headers: [{"Content-Type", "application/json"}],
29+
auth: {:basic, user <> ":" <> password},
30+
connect_options: [
31+
protocols: [:http1],
32+
transport_opts: [verify: :verify_none]
33+
]
34+
)
35+
end
2236
end
2337

2438
defp get_header_value(headers, header_item) do

server/lib/publisher/wordpress/episode.ex

+1-1
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ defmodule Publisher.WordPress.Episode do
55
alias Publisher.WordPress.Media
66

77
def save(conn, params) do
8-
req = API.new(conn.req_headers)
8+
req = API.new(conn.req_headers, true)
99

1010
with episode_id <- find_or_create_episode(req, params["guid"]),
1111
post_id <- fetch_post_id(req, episode_id),

server/lib/publisher/wordpress/podcast.ex

+4-4
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ defmodule Publisher.WordPress.Podcast do
3232
|> reject_empty_values()
3333
|> Enum.into(%{})
3434

35-
req = API.new(headers)
35+
req = API.new(headers, false)
3636

3737
with {:ok, response} <- Req.post(req, url: "podlove/v2/onboarding/setup", json: payload),
3838
{:ok, _} <- extract_status(response) do
@@ -58,7 +58,7 @@ defmodule Publisher.WordPress.Podcast do
5858
explicit: body.explicit
5959
}
6060

61-
req = API.new(headers)
61+
req = API.new(headers, false)
6262

6363
with {:ok, response} <- Req.post(req, url: "podlove/v2/podcast", json: payload),
6464
{:ok, _} <- extract_status(response) do
@@ -76,7 +76,7 @@ defmodule Publisher.WordPress.Podcast do
7676
# Logger.log(:info, "user: #{user}, endpoint: #{site}/wp-json/wp/v2/media")
7777
Logger.log(:info, "body { name: #{image_name}, type: #{image_type} }")
7878

79-
req = API.new(headers)
79+
req = API.new(headers, true)
8080

8181
with {:ok, source_url} <- Media.upload_image(req, base64_image, image_name, image_type),
8282
{:ok, info} <- save_podcast_image_url(req, source_url) do
@@ -93,7 +93,7 @@ defmodule Publisher.WordPress.Podcast do
9393
# Logger.log(:info, "user: #{user}, endpoint: #{site}/wp-json/wp/v2/media")
9494
Logger.log(:info, "body { name: #{image_name}, url: #{image_url} }")
9595

96-
req = API.new(headers)
96+
req = API.new(headers, true)
9797

9898
with {:ok, source_url} <- Media.upload_media_from_url(req, image_url, image_name),
9999
{:ok, info} <- save_podcast_image_url(req, source_url) do

0 commit comments

Comments
 (0)