Skip to content

Commit

Permalink
Require explicit answer in SH license prompt (#722)
Browse files Browse the repository at this point in the history
* Require explicit answer in SH license prompt

* add news
  • Loading branch information
jaimergp authored Sep 30, 2023
1 parent 18e97b2 commit 2302872
Show file tree
Hide file tree
Showing 2 changed files with 21 additions and 2 deletions.
4 changes: 2 additions & 2 deletions constructor/header.sh
Original file line number Diff line number Diff line change
Expand Up @@ -268,10 +268,10 @@ __LICENSE__
EOF
printf "\\n"
printf "Do you accept the license terms? [yes|no]\\n"
printf "[no] >>> "
printf ">>> "
read -r ans
ans=$(echo "${ans}" | tr '[:lower:]' '[:upper:]')
while [ "$ans" != "YES" ] && [ "$ans" != "NO" ] && [ "$ans" != "" ]
while [ "$ans" != "YES" ] && [ "$ans" != "NO" ]
do
printf "Please answer 'yes' or 'no':'\\n"
printf ">>> "
Expand Down
19 changes: 19 additions & 0 deletions news/722-license-yn
Original file line number Diff line number Diff line change
@@ -0,0 +1,19 @@
### Enhancements

* <news item>

### Bug fixes

* Fix a regression in the license prompt on SH installers to require a explicit answer instead of defaulting to `no` on Enter. (#721 via #722)

### Deprecations

* <news item>

### Docs

* <news item>

### Other

* <news item>

0 comments on commit 2302872

Please sign in to comment.