Skip to content
This repository

Fixes a bug with the "Try It" functions, the API endpoint path being generated had an extra colon #28

Open
wants to merge 1 commit into from

1 participant

Mike Lawlor
Mike Lawlor

When building the URL for the API endpoint, the value for options.protocol already had a colon in it (was added elsewhere in app.js), and so the resulting URL looked like http:://example.com/api

Mike Lawlor Fixes a bug with the "Try It" functions, the API endpoint path being …
…generated had a superfluous colon in it (eg. http:://example.com/api)
efd7872
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Showing 1 unique commit by 1 author.

May 31, 2012
Mike Lawlor Fixes a bug with the "Try It" functions, the API endpoint path being …
…generated had a superfluous colon in it (eg. http:://example.com/api)
efd7872
This page is out of date. Refresh to see the latest.

Showing 1 changed file with 2 additions and 1 deletion. Show diff stats Hide diff stats

  1. 3  app.js
3  app.js
@@ -395,7 +395,8 @@ function processRequest(req, res, next) {
395 395
                                null,
396 396
                                apiConfig.oauth.crypt);
397 397
 
398  
-            var resource = options.protocol + '://' + options.host + options.path,
  398
+            // options.protocol already has a colon (:) appended to it on line 313
  399
+            var resource = options.protocol + '//' + options.host + options.path,
399 400
                 cb = function(error, data, response) {
400 401
                     if (error) {
401 402
                         if (error.data == 'Server Error' || error.data == '') {
Commit_comment_tip

Tip: You can add notes to lines in a file. Hover to the left of a line to make a note

Something went wrong with that request. Please try again.