Skip to content

Conversation

@yaninyzwitty
Copy link

  • Fixed incorrect use of fmt.Scanf("%s", &answers[i]) which prevented proper input reading
  • Switched to bufio.Reader for echoed input to support multi-word answers
  • Continued using term.ReadPassword(int(os.Stdin.Fd())) for hidden input for portability
  • Trimmed whitespace from all answers for cleaner results

This ensures askStdin correctly handles both normal inputs (with spaces) and secure password inputs across platforms (even windows).

@CLAassistant
Copy link

CLAassistant commented Aug 23, 2025

CLA assistant check
All committers have signed the CLA.

@yaninyzwitty
Copy link
Author

Please review my commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants