Comments plugin does not properly validate password hash. When addMessage endpoint is called, one can change loginData.username value to any existing username and impersonate any existing person in chat. While no important user data is stolen, this can certainly confuse people in the comment box. It is not reproducible consistently though, I wasn’t able to find out what exactly is causing this behavior. If you can’t reproduce, you can let me know and I will record a video.

  • @perchanceM
    link
    English
    2
    edit-2
    4 months ago

    It is not reproducible consistently though

    Please link to an example comment where you have replicated this, even just once (i.e. commenting using a username that you don’t own). My guess is that when you refresh the page, you’ll see that it didn’t actually submit the comment under the username - i.e. you “tricked yourself”, but didn’t trick other users. But if I’m wrong here, then I’d definitely like to know about it! Either way, thanks for your report & concern.

      • @perchanceM
        link
        English
        24 months ago

        Oh, wow, thanks!! Some stuff was getting cached in a way that it very obviously shouldn’t have been 😬 Can you please test again just to triple-check it’s fixed? (I’ve already double-checked the code…)

        • @perplexityOP
          link
          English
          1
          edit-2
          4 months ago

          Gave it a quick test just now, seems to be working properly. Thank you!