Skip to content

Commit

Permalink
COPPA: Remove parental setting on update
Browse files Browse the repository at this point in the history
During the update we were setting the parental restriction in order to
comply with COPPA. This was causing problems with process ownership and
the updater hanging so remove it for this update and resolve the issue
for COPPA go-live date.
  • Loading branch information
tombettany committed Sep 3, 2019
1 parent e9d6f4f commit 5b47f7a
Showing 1 changed file with 5 additions and 8 deletions.
13 changes: 5 additions & 8 deletions kano_updater/scenarios.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@

import os
import shutil
import time
import traceback

from kano.logging import logger
Expand Down Expand Up @@ -1271,10 +1270,8 @@ def beta_4_3_1_to_beta_4_3_2(self, dummy_progress):

# Set Parental Controls to Ultimate for all existing users. COPPA.
# FIXME: The parental command sets the parental level and then
# restarts the sentry server, blocking on the process. We kick
# it off in the background here and wait a bit for things to
# happen but the settings command shouldn't block
run_for_every_user(
'sudo kano-settings-cli set parental --level=3 "kano" &'
)
time.sleep(10)
# restarts the sentry server, blocking on the process. For now
# just disable this and we will re-enable later.
# run_for_every_user(
# 'sudo kano-settings-cli set parental --level=3 "kano"'
# )

0 comments on commit 5b47f7a

Please sign in to comment.