Merge lp:~sergiusens/account-polld/token_less into lp:~ubuntu-push-hackers/account-polld/trunk

Proposed by Sergio Schvezov
Status: Merged
Approved by: Sergio Schvezov
Approved revision: 51
Merged at revision: 36
Proposed branch: lp:~sergiusens/account-polld/token_less
Merge into: lp:~ubuntu-push-hackers/account-polld/trunk
Prerequisite: lp:~sergiusens/account-polld/account_icon
Diff against target: 12 lines (+1/-1)
1 file modified
cmd/account-polld/main.go (+1/-1)
To merge this branch: bzr merge lp:~sergiusens/account-polld/token_less
Reviewer Review Type Date Requested Status
Roberto Alsina (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+229273@code.launchpad.net

Commit message

When logging account refresh limit to logging the corresponding account id only.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Roberto Alsina (ralsina) :
review: Approve
52. By Sergio Schvezov

Merged account_icon into token_less.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmd/account-polld/main.go'
2--- cmd/account-polld/main.go 2014-08-01 22:35:05 +0000
3+++ cmd/account-polld/main.go 2014-08-01 22:35:05 +0000
4@@ -83,7 +83,7 @@
5 for data := range watcher.C {
6 if account, ok := mgr[data.AccountId]; ok {
7 if data.Enabled {
8- log.Printf("New account data for %d - was %#v, now is %#v", data.AccountId, account.authData, data)
9+ log.Println("New account data for existing account with id", data.AccountId)
10 account.updateAuthData(data)
11 } else {
12 account.Delete()

Subscribers

People subscribed via source and target branches