Code review comment for lp://staging/~mardy/signon-ui/export-cookies

Revision history for this message
Ken VanDine (ken-vandine) wrote :

Looks fine, my only concern is security. We need to make sure this can only be used by the webapps container. I guess we need an apparmor profile for that. Has that been discussed yet?

review: Needs Information

« Back to merge proposal