Skip to content

Commit 2850470

Browse files
Removing other username entries from logs
1 parent 2986d54 commit 2850470

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

common/djangoapps/track/views/__init__.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,11 @@ def user_track(request):
9292
except ValueError:
9393
pass
9494

95+
try:
96+
data.pop('username')
97+
except KeyError:
98+
pass
99+
95100
context_override = contexts.course_context_from_url(page)
96101
context_override['user_id'] = user_id
97102
context_override['event_source'] = 'browser'
@@ -128,13 +133,13 @@ def server_track(request, event_type, event, page=None):
128133
return # don't log
129134

130135
try:
131-
username = request.user.username
136+
user_id = request.user.id
132137
except:
133-
username = "anonymous"
138+
user_id = 0
134139

135140
context_override = _get_course_context(page)
136141
context_override.update({
137-
'username': username,
142+
'user_id': user_id,
138143
'event_source': 'server',
139144
'page': page
140145
})
@@ -170,7 +175,6 @@ def task_track(request_info, task_info, event_type, event, page=None):
170175

171176
context_override = contexts.course_context_from_url(page)
172177
context_override.update({
173-
'username': request_info.get('username', 'unknown'),
174178
'ip': request_info.get('ip', 'unknown'),
175179
'agent': request_info.get('agent', 'unknown'),
176180
'host': request_info.get('host', 'unknown'),

0 commit comments

Comments
 (0)