default req.proxy destination to req.uri if not provided (common case)

davehorton · Aug 2, 2017 · 56079cf · 56079cf
1 parent 33fb3a4
commit 56079cf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/lib/request.js b/lib/request.js
@@ -91,12 +91,10 @@ class Request extends Emitter {
     if( this.source !== 'network' ) { 
       throw new Error('Request#proxy can only be used for incoming requests') ; 
     }
-    if( typeof opts !== 'object' || !opts.destination ) { 
-      throw new Error('Request#proxy: opts.destination is required') ; 
-    }
+    opts = opts || {} ;
 
     //TODO: throw error if req.res.send has already been called (i.e. can't start off as UAS and then become a proxy)
-    let destination = opts.destination;
+    let destination = opts.destination || this.uri ;
     if( typeof destination === 'string') { opts.destination = [destination] ; }
 
     Object.assign( opts, {

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "drachtio",
-  "version": "3.0.2",
+  "version": "3.0.4",
   "description": "node.js middleware for sip applications",
   "keywords": [
     "framework",