HearthOfHestia / Nebula

Hearth of Hestia's fork of NebulaSS13
GNU Affero General Public License v3.0
2 stars 40 forks source link

Psionics Coercion Mind Reading Character Limited #205

Open Menkhu opened 2 years ago

Menkhu commented 2 years ago

Description of issue

Replies to coercion mind reading questions seem to be severely limited by length. They also don't give any indicator to the mind reader or the person being read when the length is too long. The answers seems to be submitted by the person being read, and the mind reader receives the normal 'You receive nothing useful' message which is usually given when the person being read refuses to answer.

Difference between expected and actual behavior

I can't pin down the exact character limit, but it seems to be very small and impossible to ascertain when actually writing a response to a mind-reading question.

Steps to reproduce

In game, try to read someones mind with the coercion psionics faculty. Certain reply lengths will not be submitted as described above.

Specific information for locating

Length of time in which bug has been known to occur

Noticed 28/11/2021 during the scheduled high pop testing round.

Client version, Server revision & Game ID

Unknown (I forgot to check at the time, sorry)

Issue bingo

MistakeNot4892 commented 2 years ago

I don't believe that length should break input() like this - there are no length checks or sanitization. That failure message is also printed if you take more than sixty seconds to reply, if the user is unconscious, or the target dies. I'll slap in a commit to change as text to as message in case that helps though.

Menkhu commented 2 years ago

I don't believe that length should break input() like this - there are no length checks or sanitization. That failure message is also printed if you take more than sixty seconds to reply, if the user is unconscious, or the target dies. I'll slap in a commit to change as text to as message in case that helps though.

Oddly enough the window didn't seem to close in sixty seconds, but the replies were still ignored when submitted in favour of the 'You receive nothing useful' message. I didn't time it though so I could be wrong here. I'll try to check next round I can.

MistakeNot4892 commented 2 years ago

We can't make BYOND close an input() window, hence the sixty seconds check.

Menkhu commented 2 years ago

Might be reasonable to extend the check then? Almost every time I've tried to use the ability, the person on the receiving end hasn't been able to get their replies in on time, and they haven't been aware their messages weren't actually getting through, which leads to a lot of confusion.