Skip to content

Commit 3099b61

Browse files
chadwhitacrechadwhitacre
committed
Default opt-out on error reporting for now
1 parent 31767d2 commit 3099b61

File tree

2 files changed

+76
-34
lines changed

2 files changed

+76
-34
lines changed

‎install/_lib.sh‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,13 @@ umask 002
88
log_file="sentry_install_log-`date +'%Y-%m-%d_%H-%M-%S'`.txt"
99
exec &> >(tee -a "$log_file")
1010

11+
# Thanks to https://tldp.org/LDP/abs/html/intandnonint.html
12+
if [[ "$(tty)" == "not a tty" ]]; then
13+
PROMPTABLE=0
14+
else
15+
PROMPTABLE=1
16+
fi
17+
1118
# Work from /install/ for install.sh, project root otherwise
1219
if [[ "$(basename $0)" = "install.sh" ]]; then
1320
cd "$(dirname $0)/install/"

‎install/error-handling.sh‎

Lines changed: 69 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -22,42 +22,77 @@ if [[ -f $reporterrors ]]; then
2222
export REPORT_ERRORS=0
2323
fi
2424
else
25-
echo
26-
echo "Hey, so ... we would love to find out when you hit an issue with this here"
27-
echo "installer you are running. Turns out there is an app for that, called Sentry."
28-
echo "Are you okay with us sending info to Sentry when you run this installer?"
29-
echo
30-
echo " y / yes / 1"
31-
echo " n / no / 0"
32-
echo
33-
echo "(Btw, we send this to our own self-hosted Sentry instance, not to Sentry SaaS,"
34-
echo "so that we can be in this together.)"
35-
echo
36-
echo "Here's the info we may collect in order to help us improve the installer:"
37-
echo
38-
echo " - OS username"
39-
echo " - IP address"
40-
echo " - install log"
41-
echo " - performance data"
42-
echo
43-
echo "Thirty (30) day retention. No marketing. Privacy policy at sentry.io/privacy."
44-
echo
25+
if [[ $PROMPTABLE == "0" ]]; then
26+
echo
27+
echo "Hey, so ... we would love to find out when you hit an issue with this here"
28+
echo "installer you are running. Turns out there is an app for that, called Sentry."
29+
echo "Would you be willing to let us automatically send data to Sentry from this "
30+
echo "installer? If so, add this to your automation:"
31+
echo
32+
echo " echo yes > /path/to/sentry/.reporterrors"
33+
echo
34+
echo "(Btw, we send this to our own self-hosted Sentry instance, not to Sentry SaaS,"
35+
echo "so that we can be in this together.)"
36+
echo
37+
echo "Here's the info we may collect in order to help us improve the installer:"
38+
echo
39+
echo " - OS username"
40+
echo " - IP address"
41+
echo " - install log"
42+
echo " - performance data"
43+
echo
44+
echo "Thirty (30) day retention. No marketing. Privacy policy at sentry.io/privacy."
45+
echo
46+
echo "For now we are defaulting to opt-out, but our plan is to hard-require a choice"
47+
echo "from you starting in version 22.12.0, because let's be honest, none of you will"
48+
echo "act on this otherwise. To avoid disruption you can:"
49+
echo
50+
echo " echo no > /path/to/sentry/.reporterrors"
51+
echo
52+
echo "We'll probably also add an env var and/or CLI flag before then, too. See:"
53+
echo
54+
echo " https://github.com/getsentry/team-ospo/issues/36"
55+
echo
56+
echo "Thanks for using Sentry."
57+
echo
58+
else
59+
echo
60+
echo "Hey, so ... we would love to find out when you hit an issue with this here"
61+
echo "installer you are running. Turns out there is an app for that, called Sentry."
62+
echo "Are you okay with us sending info to Sentry when you run this installer?"
63+
echo
64+
echo " y / yes / 1"
65+
echo " n / no / 0"
66+
echo
67+
echo "(Btw, we send this to our own self-hosted Sentry instance, not to Sentry SaaS,"
68+
echo "so that we can be in this together.)"
69+
echo
70+
echo "Here's the info we may collect in order to help us improve the installer:"
71+
echo
72+
echo " - OS username"
73+
echo " - IP address"
74+
echo " - install log"
75+
echo " - performance data"
76+
echo
77+
echo "Thirty (30) day retention. No marketing. Privacy policy at sentry.io/privacy."
78+
echo
4579

46-
yn=""
47-
until [ ! -z "$yn" ]
48-
do
49-
read -p "y or n? " yn
50-
case $yn in
51-
y | yes | 1) export REPORT_ERRORS=1; echo "yes" > $reporterrors; echo; echo -n "Thank you.";;
52-
n | no | 0) export REPORT_ERRORS=0; echo "no" > $reporterrors; echo; echo -n "Understood.";;
53-
*) yn="";;
54-
esac
55-
done
80+
yn=""
81+
until [ ! -z "$yn" ]
82+
do
83+
read -p "y or n? " yn
84+
case $yn in
85+
y | yes | 1) export REPORT_ERRORS=1; echo "yes" > $reporterrors; echo; echo -n "Thank you.";;
86+
n | no | 0) export REPORT_ERRORS=0; echo "no" > $reporterrors; echo; echo -n "Understood.";;
87+
*) yn="";;
88+
esac
89+
done
5690

57-
echo " Your answer is cached in '.reporterrors', remove it to see this"
58-
echo "prompt again."
59-
echo
60-
sleep 5
91+
echo " Your answer is cached in '.reporterrors', remove it to see this"
92+
echo "prompt again."
93+
echo
94+
sleep 5
95+
fi
6196
fi
6297

6398
# Make sure we can use sentry-cli if we need it.

0 commit comments

Comments
 (0)