Merge pull request #89 from grant/patch-1

Salesforce Auth Nits

Author: Eric Koleda

samples/Salesforce.gs

@@ -1,5 +1,6 @@
 /**
- * Saleforce's Auth flow https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/quickstart_oauth.htm
+ * Saleforce Auth flow
+ * @see https://developer.salesforce.com/docs/atlas.en-us.api_rest.meta/api_rest/quickstart_oauth.htm
  */
 
 var CLIENT_ID = '...';
@@ -12,8 +13,8 @@ function run() {
   var service = getService();
   if (service.hasAccess()) {
     // GET requests require access_token parameter
-
-    var response = UrlFetchApp.fetch(service.getToken().instance_url + '/services/data/v24.0/chatter/users/me', {
+    var url = service.getToken().instance_url + '/services/data/v24.0/chatter/users/me';
+    var response = UrlFetchApp.fetch(url, {
       headers: {
         Authorization: 'Bearer ' + service.getAccessToken()
       }
@@ -22,8 +23,7 @@ function run() {
     Logger.log(JSON.stringify(result, null, '  '));
   } else {
     var authorizationUrl = service.getAuthorizationUrl();
-    Logger.log('Open the following URL and re-run the script: %s',
-        authorizationUrl);
+    Logger.log('Open the following URL and re-run the script: ' + authorizationUrl);
   }
 }
 
@@ -65,14 +65,6 @@ function authCallback(request) {
   if (authorized) {
     return HtmlService.createHtmlOutput('Success!');
   } else {
-    return HtmlService.createHtmlOutput('Denied');
+    return HtmlService.createHtmlOutput('Denied.');
   }
 }
-
-/**
- * Logs the redict URI to register in the Saleforce Apps Page.
- */
-function logRedirectUri() {
-  var service = getService();
-  Logger.log(service.getRedirectUri());
-}