Principal Service: Add support for multiple user accounts

examples/bank_app/bank.cc

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "examples/bank_app/bank.mojom.h"
 #include "mojo/common/binding_set.h"
 #include "mojo/public/c/system/main.h"
 #include "mojo/public/cpp/application/application_connection.h"
 #include "mojo/public/cpp/application/application_delegate.h"
 #include "mojo/public/cpp/application/application_impl.h"
@@ skipping 15 matching lines @@
   void GetBalance(const GetBalanceCallback& callback) override {
     callback.Run(balance_);
   }
  private:
   int32_t balance_;
 };
 
 class BankUser {
  public:
   explicit BankUser(std::string* user) : user_(user) { }
-  void Run(const vanadium::BlessingPtr& b) const {
+  void Run(const vanadium::UserPtr& u) const {

[jamesr, 2015/10/28 21:45:51]

our style guide says to avoid one-letter variable names except for loop
counters. 'u' is also a valid modifier in C++ which is extra confusing. i'd say
'user' here

(C++ needs to be more verbose than a language like Go in cases like this, in
part because the language grammar is far less clean)

[ataly, 2015/10/28 21:59:14]

Ah, I see. Thanks for pointing that out. Done.

     user_->clear();
-    if (b && b->chain.size() > 0) {
-      user_->append(b->chain[0]->extension);
-      for (size_t i = 1; i < b->chain.size(); i++) {
-        user_->append(vanadium::ChainSeparator);
-        user_->append(b->chain[i]->extension);
-      }
+    if (u) {
+      user_->append(u->email);
     }
   }
  private:
   std::string *user_;
 };
 
 class BankApp : public mojo::ApplicationDelegate,
                 public mojo::InterfaceFactory<Bank> {
  public:
   BankApp() {
   }
   void Initialize(mojo::ApplicationImpl* app) override {
     app->ConnectToService("mojo:principal_service", &login_service_);
   }
 
   // From ApplicationDelegate
   bool ConfigureIncomingConnection(
       mojo::ApplicationConnection* connection) override {
     std::string url = connection->GetRemoteApplicationURL();
     if (url.length() > 0) {
       vanadium::AppInstanceNamePtr app(vanadium::AppInstanceName::New());
       app->url = url;
       std::string user;
-      login_service_->GetUserBlessing(app.Pass(), BankUser(&user));
+      login_service_->GetUser(app.Pass(), BankUser(&user));
       // Check and see whether we got a valid user blessing.
       if (!login_service_.WaitForIncomingResponse()) {
         MOJO_LOG(INFO) << "Failed to get a valid user blessing";
         return false;
       }
       // Record user access to the bank and reject customers that
       // don't have a user identity.
       if (user.empty()) {
         MOJO_LOG(INFO) << "Rejecting customer without a user identity";
         return false;
@@ skipping 14 matching lines @@
   mojo::BindingSet<Bank> bindings_;
   vanadium::PrincipalServicePtr login_service_;
 };
 
 }  // namespace examples
 
 MojoResult MojoMain(MojoHandle application_request) {
   mojo::ApplicationRunner runner(new examples::BankApp());
   return runner.Run(application_request);
 }