Summit

Merge lp:~mhall119/summit/api-hide-secret-key into lp:summit

  1. api-hide-secret-key
  2. Merge into trunk
Proposed by Michael Hall
Status: Merged
Approved by: James Westby
Approved revision: 186
Merged at revision: 186
Proposed branch: lp:~mhall119/summit/api-hide-secret-key
Merge into: lp:summit
Diff against target: 12 lines (+1/-1)
1 file modified
summit/services/views.py (+1/-1)
To merge this branch: bzr merge lp:~mhall119/summit/api-hide-secret-key
Reviewer Review Type Date Requested Status
James Westby (community) Approve
Review via email: mp+76073@code.launchpad.net

Commit message

Exclude attendee secret key from the API

Description of the change

Attendee secret keys, used for private schedule icals, was being exposed by the API. This patch excluded that field.

To post a comment you must log in.
Revision history for this message
James Westby (james-w) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'summit/services/views.py'
2--- summit/services/views.py 2011-09-17 16:26:40 +0000
3+++ summit/services/views.py 2011-09-19 17:16:24 +0000
4@@ -34,7 +34,7 @@
5 return model_service(Meeting, request, url, exclude=['private'], limit=limit)
6
7 def attendee_service(request, url):
8- return model_service(Attendee, request, url)
9+ return model_service(Attendee, request, url, exclude=['secret_key_id'])
10
11 def crew_service(request, url):
12 return model_service(Crew, request, url)

Subscribers

People subscribed via source and target branches