tiQit

Merge lp:~samstoll1/tiqit/cookie into lp:tiqit

  1. cookie
  2. Merge into trunk
Proposed by Sam Stoll
Status: Merged
Approved by: Matthew Earl
Approved revision: 6
Merged at revision: 8
Proposed branch: lp:~samstoll1/tiqit/cookie
Merge into: lp:tiqit
Diff against target: 12 lines (+1/-1)
1 file modified
scripts/tiqit/__init__.py (+1/-1)
To merge this branch: bzr merge lp:~samstoll1/tiqit/cookie
Reviewer Review Type Date Requested Status
Matthew Earl Approve
Review via email: mp+282932@code.launchpad.net

Commit message

Changing plugin get cookie API to want whole set-cookie string, rather than name-value pair. This allows plugins more control over cookies' other morsel values.

Description of the change

Changing plugin get cookie API to want whole set-cookie string, rather than name-value pair. This allows plugins more control over cookies' other morsel values. (See changes in plugin repo)

To post a comment you must log in.
Revision history for this message
Matthew Earl (matthew-earl) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'scripts/tiqit/__init__.py'
2--- scripts/tiqit/__init__.py 2015-11-27 11:42:48 +0000
3+++ scripts/tiqit/__init__.py 2016-01-18 11:54:01 +0000
4@@ -1117,7 +1117,7 @@
5
6 # Print the HTTP header, including any cookies.
7 for c in plugins.getOutgoingCookies():
8- print "%s" % cookie.getSetCookie(c)
9+ print c
10 print "Set-Cookie: update=; expires=Sat, 01-Jan-2000 00:00:00 GMT; path=%s" % getBasePath()
11 print "Content-Type: text/html; charset=utf-8"
12 print """

Subscribers

People subscribed via source and target branches