Skip to content

[RSS]XMLパースエラー発生時にシステムエラーとなる問題を修正しました - #2209

Merged
gakigaki merged 2 commits into
masterfrom
fix-rss-not-found
Jul 9, 2025
Merged

[RSS]XMLパースエラー発生時にシステムエラーとなる問題を修正しました#2209
gakigaki merged 2 commits into
masterfrom
fix-rss-not-found

Conversation

@gakigaki

@gakigaki gakigaki commented Jun 10, 2025

Copy link
Copy Markdown
Contributor

概要

RSSフィードの取得時に、XMLのパースエラーが原因でシステムエラーが発生する問題を修正しました。

変更の目的

RSSフィードの提供元URLがリンク切れになっている場合や、不正な形式のXMLが返された場合に、フレーム内にシステムエラーが表示される問題がありました。これを防ぎ、安定した運用ができるようにすることが目的です。

変更内容

XMLのパース処理に例外処理を追加しました。パースエラーが発生した際にはシステムエラーとせず、エラー内容をログに出力します。

また、エラー発生時に表示するメッセージを権限に応じて変更しました。

  • 管理者向け: 原因調査のヒントとなるよう「RSSフィードの取得に失敗しました。URLが正しいか、またはフィードが有効であるかご確認ください。」というメッセージを表示します。
  • 一般ユーザー向け: シンプルに「RSSフィードの読み込みに失敗しました。しばらくしてから再度お試しください。」というメッセージを表示します。

テスト

  • 意図的にリンク切れのURLを設定し、システムエラーが発生しないことを確認しました。
  • 不正な形式のXMLを返すURLを設定し、システムエラーが発生しないことを確認しました。
  • 管理者アカウントでログイン時、専用のエラーメッセージが表示されることを確認しました。
  • 一般ユーザー(非ログイン含む)で、専用のエラーメッセージが表示されることを確認しました。

特記事項

  • エラー発生時、管理者には原因の調査に役立つメッセージを、一般ユーザーにはシンプルなメッセージを表示するように制御しています。

レビュー完了希望日

関連Pull requests/Issues

参考

DB変更の有無

無し

チェックリスト

パースエラー時に例外キャッチ
@gakigaki gakigaki added the bug バグ・不具合連絡 label Jun 10, 2025
@gakigaki
gakigaki marked this pull request as ready for review June 10, 2025 09:18
@gakigaki
gakigaki merged commit 4c67549 into master Jul 9, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug バグ・不具合連絡 plugin(rsses)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant