Merge lp:~mardy/ubuntu-system-settings-online-accounts/lp1234099 into lp:ubuntu-system-settings-online-accounts

Proposed by Alberto Mardegan
Status: Merged
Approved by: Ken VanDine
Approved revision: 51
Merged at revision: 51
Proposed branch: lp:~mardy/ubuntu-system-settings-online-accounts/lp1234099
Merge into: lp:ubuntu-system-settings-online-accounts
Diff against target: 20 lines (+3/-1)
1 file modified
src/module/OAuth.qml (+3/-1)
To merge this branch: bzr merge lp:~mardy/ubuntu-system-settings-online-accounts/lp1234099
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+188815@code.launchpad.net

Commit message

By default, only let unconfined apps access Online Accounts

Description of the change

By default, only let unconfined apps access Online Accounts

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
Ken VanDine (ken-vandine) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/module/OAuth.qml'
2--- src/module/OAuth.qml 2013-07-11 10:06:49 +0000
3+++ src/module/OAuth.qml 2013-10-02 12:20:46 +0000
4@@ -26,6 +26,8 @@
5
6 /* To override the parameters coming from the .provider file: */
7 property variant authenticationParameters: null
8+ /* To override the default access control list: */
9+ property variant accessControlList: ["unconfined"]
10
11 property variant authReply
12 property bool isNewAccount: false
13@@ -48,7 +50,7 @@
14 Credentials {
15 id: creds
16 caption: account.provider.id
17- acl: ["*"]
18+ acl: accessControlList
19 onCredentialsIdChanged: root.credentialsStored()
20 }
21

Subscribers

People subscribed via source and target branches