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

Workaround instant disconnect issue for whispers #60

Merged
merged 2 commits into from
May 15, 2024
Merged

Workaround instant disconnect issue for whispers #60

merged 2 commits into from
May 15, 2024

Commits on May 15, 2024

  1. Cap target length to 47 bytes 

    This works around an instant disconnect bug with SendAddonMessage when
sending whispers to players whose combined "character-realm" name
appears to exceed this threshold, most easily noticed with players
from Russian realms.

As the issue is being actively fixed, hopefully we don't need this
workaround for too long - so a time bomb is included to disable it
after about two weeks in the event a user doesn't update.

This is only applied to "smart" messages, as those are the only
ones guaranteed to be "name-realm" qualified.
    @Meorawr
    Meorawr committed May 15, 2024
    Configuration menu
    Copy the full SHA
    cecf3c2 View commit details
    Browse the repository at this point in the history

  2. Appease the Luacheck Leprechaun

    @Meorawr
    Meorawr committed May 15, 2024
    Configuration menu
    Copy the full SHA
    99fe441 View commit details
    Browse the repository at this point in the history