[OnlineAccounts] Ship and enable SASL authentication plugin
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Canonical System Image |
Fix Released
|
High
|
David Barth | ||
| Ubuntu Online Accounts API |
Fix Released
|
High
|
Alberto Mardegan | ||
| signon-plugin-sasl |
Fix Released
|
High
|
Alberto Mardegan | ||
| webapps-sprint |
Fix Released
|
High
|
Alberto Mardegan | ||
Bug Description
E-mail, XMPP and other services often uses SASL as authentication method. In the Online Accounts project there is an authentication plugin for SASL, from the old Nokia times (2011) and hasn't been maintained since.
The first step in order to bring this plugin to Ubuntu is building the plugin and packaging it.
Then, it appears that there are some changes to be done either in the plugin or in the OA APIs, due to the dynamic nature of SASL (the list of mechanisms is known only at runtime, because it's coming from the remote server). Nor the older QML API from online-
To me, it seems that the best way to solve this is actually to modify the SASL plugin, and create an input parameter in which the client can specify the initial list of mechanisms.
The third thing to do is create a simple QML application which QA can use to test the plugin and catch possible regressions.
Related branches
- Online Accounts: Pending requested
-
Diff: 1609 lines (+671/-717)15 files modified.bzr-builddeb/default.conf (+2/-0)
.gitignore (+8/-0)
.qmake.conf (+2/-0)
common-project-config.pri (+6/-16)
debian/changelog (+3/-2)
debian/control (+4/-2)
signon-plugin-sasl.pro (+0/-10)
src/sasldata.h (+11/-2)
src/saslplugin.cpp (+9/-6)
src/saslplugin.h (+2/-1)
src/src.pro (+4/-2)
tests/saslplugintest.cpp (+0/-559)
tests/saslplugintest.h (+0/-100)
tests/tests.pro (+9/-17)
tests/tst_plugin.cpp (+611/-0)
- Online Accounts: Pending requested
-
Diff: 431 lines (+217/-9)11 files modifiedsrc/lib/OnlineAccounts/SaslData (+1/-0)
src/lib/OnlineAccounts/authentication_data.cpp (+68/-1)
src/lib/OnlineAccounts/authentication_data.h (+48/-1)
src/lib/OnlineAccounts/authentication_reply.cpp (+41/-1)
src/lib/OnlineAccounts/global.h (+1/-0)
src/lib/OnlineAccountsDaemon/authenticator.cpp (+15/-3)
src/lib/OnlineAccountsDaemon/dbus_constants.h (+14/-1)
src/lib/OnlineAccountsDaemon/manager.cpp (+3/-1)
src/lib/Ubuntu/OnlineAccounts.2/account.cpp (+1/-0)
src/lib/Ubuntu/OnlineAccounts.2/account.h (+1/-0)
tests/lib/OnlineAccounts/functional_tests/functional_tests.cpp (+24/-1)
| Changed in signon-plugin-sasl: | |
| status: | Confirmed → In Progress |
| Changed in webapps-sprint: | |
| status: | New → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Alberto Mardegan (mardy) |
| milestone: | none → sprint-17 |
| Changed in canonical-devices-system-image: | |
| status: | New → Fix Committed |
| importance: | Undecided → High |
| assignee: | nobody → David Barth (dbarth) |
| milestone: | none → ww02-2016 |
| Changed in webapps-sprint: | |
| status: | In Progress → Fix Committed |
| Changed in signon-plugin-sasl: | |
| status: | In Progress → Fix Committed |
| Changed in online-accounts-api: | |
| status: | New → In Progress |
| importance: | Undecided → High |
| assignee: | nobody → Alberto Mardegan (mardy) |
| Changed in canonical-devices-system-image: | |
| status: | Fix Committed → Fix Released |
| Changed in online-accounts-api: | |
| status: | In Progress → Fix Released |
| Changed in signon-plugin-sasl: | |
| status: | Fix Committed → Fix Released |
| Changed in webapps-sprint: | |
| status: | Fix Committed → Fix Released |
