Skip to content
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.

Sign up for GitHub

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

Jump to bottom

If MAM lookup code crashes, user gets an empty result set instead of an error #4191

Merged
merged 4 commits into from
Dec 14, 2023
Merged

If MAM lookup code crashes, user gets an empty result set instead of an error #4191

merged 4 commits into from
Dec 14, 2023

Commits on Dec 12, 2023

  1. Return internal-server-error IQ instead of an empty result set 

    In case there is an error when querying
Currently debugging case when one message in archive is encoded by other format (i.e. expected XML, but gets erlang term)
    @arcusfelis
    arcusfelis committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    ea9958c View commit details
    Browse the repository at this point in the history

  2. Disable pm_failed_to_decode_message_in_database for elastic

    @arcusfelis
    arcusfelis committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    4d1bf14 View commit details
    Browse the repository at this point in the history

  3. Return an error for each decoded message separately

    @arcusfelis
    arcusfelis committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    f274d4d View commit details
    Browse the repository at this point in the history

  4. Make fix work with cassandra 

    Move fix into mam_message module
    @arcusfelis
    arcusfelis committed Dec 12, 2023
    Configuration menu
    Copy the full SHA
    87d01ac View commit details
    Browse the repository at this point in the history