shell_yes?/1 was not showing the correct message and always defaults to yes which is dangerous

2020-11-27 13:42:28 -06:00 · 2020-11-27 13:42:28 -06:00 · 50aadc3d5c
commit 50aadc3d5c
parent 6adad96c79
2 changed files with 1 additions and 7 deletions
--- a/lib/mix/tasks/pleroma/user.ex
+++ b/lib/mix/tasks/pleroma/user.ex
@ -60,7 +60,7 @@ defmodule Mix.Tasks.Pleroma.User do
      - admin: #{if(admin?, do: "true", else: "false")}
    """)

-    proceed? = assume_yes? or shell_yes?("Continue?")
+    proceed? = assume_yes? or shell_prompt("Continue?", "n") in ~w(Yn Y y)

    if proceed? do
      start_pleroma()