Skip to content

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

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

gakigaki
Copy link
Contributor

@gakigaki gakigaki commented Jun 10, 2025

概要

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
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