/settingのAllow Originを設定しても正しく認識しない

[whiteball]

内容

Allow Originの設定としてSettingsに書いたものを使う場合に、正しくホスト名を認識していませんでした。
コードを確認したところ、settings.allow_originが文字列なのにgenerate_appのList[str]な引数にそのまま渡しているため、1文字ずつに分解されているようでした。
そのため、引数のAllow Originと、Settingsのものをどちらか使うかを判定する部分で、settings.allow_originをリストに変換するようにしました。
この変換は/settingページの説明で、半角スペースで複数のオリジンを複数指定できるとありましたので、settings.allow_origin.split(" ")で分割・リスト化しています。

関連 Issue

スクリーンショット・動画など

その他

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
voicevox_engine/init.py	1	0
voicevox_engine/acoustic_feature_extractor.py	75	0
voicevox_engine/dev/synthesis_engine/init.py	2	0
voicevox_engine/dev/synthesis_engine/mock.py	36	2
voicevox_engine/full_context_label.py	162	3
voicevox_engine/kana_parser.py	86	1
voicevox_engine/metas/Metas.py	33	0
voicevox_engine/metas/MetasStore.py	29	14
voicevox_engine/metas/init.py	2	0
voicevox_engine/model.py	145	9
voicevox_engine/mora_list.py	4	0
voicevox_engine/part_of_speech_data.py	5	0
voicevox_engine/preset/Preset.py	12	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	81	2
voicevox_engine/preset/init.py	4	0
voicevox_engine/synthesis_engine/init.py	5	0
voicevox_engine/synthesis_engine/core_wrapper.py	206	166
voicevox_engine/synthesis_engine/make_synthesis_engines.py	57	49
voicevox_engine/synthesis_engine/synthesis_engine.py	130	11
voicevox_engine/synthesis_engine/synthesis_engine_base.py	67	9
voicevox_engine/user_dict.py	129	6
voicevox_engine/utility/init.py	3	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/path_utility.py	26	6
TOTAL	1339	278

[y-chan]

ロジック的にはLGTMですが、コードの可読性が低くなってしまったように感じます。
allow_origin変数を用意し、条件に合わせてそこへ代入して、最後それを引数として渡してあげる方が見栄えが良くなるかなと思いました...!

[whiteball]

レビューありがとうございます！
三項演算子のifで書いていた部分を、普通のif-else文で書き直してみました。

[y-chan]

LGTM！

[Hiroshiba]

LGTM！！

[Hiroshiba]

（メンテナ判断メモです）
もし0.14.1が出るとしたらこの変更があったほうが良さそうなので、release-0.14ブランチ側でcherry-pickしたいと思います。