New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add application/xml into MIME_TYPES #10

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
2 participants
@Akkiesoft

Akkiesoft commented Jan 13, 2018

Content-Type: application/xmlで配信されているフィードが読み込めなかったので追加しました。

@Akkiesoft

This comment has been minimized.

Show comment
Hide comment
@Akkiesoft

Akkiesoft Jan 13, 2018

feed_searcher_spec.rb を読んでいませんでした……。
application/xmlを読まないようにしてあるのは何か意図があるのでしょうか。

読み込もうとしていたRSSは http://www.tokyu.co.jp/rss_infomation.xmlhttp://www.tokyu.co.jp/rss_news.xml の2つでした。

Akkiesoft commented Jan 13, 2018

feed_searcher_spec.rb を読んでいませんでした……。
application/xmlを読まないようにしてあるのは何か意図があるのでしょうか。

読み込もうとしていたRSSは http://www.tokyu.co.jp/rss_infomation.xmlhttp://www.tokyu.co.jp/rss_news.xml の2つでした。

@r7kamura

This comment has been minimized.

Show comment
Hide comment
@r7kamura

r7kamura Jan 13, 2018

Member

@Akkiesoft Pull Request ありがとうございます。

application/xml を読み込まないようにしているのは、意図があると思います。明らかに RSS あるいは Atom フィードの MIME Type であるものだけを選択し、XML ではあるもののフィードではないようなものを誤って選択しないような実装になっているのではないでしょうか。

デフォルトの挙動は現在のままで、application/xml を読み込めるようにする、あるいは任意の MIME Type を指定可能にするようなオプションを提供する、という実装への変更はどうでしょうか。この場合、勿論 Fastladder 側でも、どの MIME Type をフィードとして認識するかというオプションを与えられるようにする変更が必要になります。しかしこのプランには、多くの既存の利用者が影響を受けないというメリットがあります。

Member

r7kamura commented Jan 13, 2018

@Akkiesoft Pull Request ありがとうございます。

application/xml を読み込まないようにしているのは、意図があると思います。明らかに RSS あるいは Atom フィードの MIME Type であるものだけを選択し、XML ではあるもののフィードではないようなものを誤って選択しないような実装になっているのではないでしょうか。

デフォルトの挙動は現在のままで、application/xml を読み込めるようにする、あるいは任意の MIME Type を指定可能にするようなオプションを提供する、という実装への変更はどうでしょうか。この場合、勿論 Fastladder 側でも、どの MIME Type をフィードとして認識するかというオプションを与えられるようにする変更が必要になります。しかしこのプランには、多くの既存の利用者が影響を受けないというメリットがあります。

@Akkiesoft

This comment has been minimized.

Show comment
Hide comment
@Akkiesoft

Akkiesoft Jan 13, 2018

@r7kamura さん、ありがとうございます。

フィードではないXMLを誤って選択しないという意図について理解しました。
オプションの追加は魅力的ですが、私のスキルが乏しく自分で作るのは難しいかなぁ……と言う印象です。すみません。

該当のフィードをfeed_searcherで読み込めるように加工・出力するスクリプトを作成して回避したので、リクエストを取り下げたいと思います。

Akkiesoft commented Jan 13, 2018

@r7kamura さん、ありがとうございます。

フィードではないXMLを誤って選択しないという意図について理解しました。
オプションの追加は魅力的ですが、私のスキルが乏しく自分で作るのは難しいかなぁ……と言う印象です。すみません。

該当のフィードをfeed_searcherで読み込めるように加工・出力するスクリプトを作成して回避したので、リクエストを取り下げたいと思います。

@Akkiesoft Akkiesoft closed this Jan 13, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment