libretro / RetroArch

Cross-platform, sophisticated frontend for the libretro API. Licensed GPLv3.
http://www.libretro.com
GNU General Public License v3.0
10.39k stars 1.84k forks source link

Passwords are not hidden #8103

Closed Pysis868 closed 5 years ago

Pysis868 commented 5 years ago

First and foremost consider this:

Description

UI does not conceal stored password(s), at least for the achievement system (Cheevos?) setting.

Expected behavior

Entering password, shows stars. Complete, value entry in UI shows stars. Achievements work as expected.

Actual behavior

Entering password, shows actual characters. Complete, value entry in UI shows characters. Achievements work as expected. (Untested)

Steps to reproduce the bug

  1. Open RetroArch via HBMenu
  2. Navigate to Settings > Achievements in default XMB theme.
  3. Enable Acheivements
  4. Enter Username
  5. Enter Password

Bisect Results

Right when I installed the program, opened, during loading content, going through various menu options viewing most and configuring some.

Version/Commit

You can find this information under Information/System Information

Environment information

Pysis868 commented 5 years ago

Don't know why I couldn't find the other issue(s) before. Probably a duplicate now...

orbea commented 5 years ago

I agree, but since you filled out the template and explained the issue well lets keep this one instead.

andres-asm commented 5 years ago

the password is blanked after a reload (because it's no longer needed, a token is used instead)

orbea commented 5 years ago

So this is a non-issue? I haven't used retroachievements before.

andres-asm commented 5 years ago

it's minor in any case, after you have logged in once it will be hidden

natinusala commented 5 years ago

It's a very easy fix. I can have a shot at it.

orbea commented 5 years ago

The referenced PR was merged so I am going to close this, please let us know if there are any further issues!

Pysis868 commented 5 years ago

Great! I can't do much with C, so it's nice to see this response!