]> source.dussan.org Git - jquery.git/commitdiff
Tests: Recognize callbacks with dots in the Node.js mock server 4785/head
authorMichał Gołębiowski-Owczarek <m.goleb@gmail.com>
Wed, 2 Sep 2020 16:42:52 +0000 (18:42 +0200)
committerGitHub <noreply@github.com>
Wed, 2 Sep 2020 16:42:52 +0000 (18:42 +0200)
This aligns the Node.js server with the previous PHP one in sending `mock.php`
as a callback if there's no `callback` parameter in the query string which is
triggered by a recently added test. This prevents the request crashing on that
Node.js server and printing a JS error:
```
TypeError: Cannot read property '1' of null
```

Closes gh-4764
Ref gh-4754

test/middleware-mockserver.js

index d0329eb9bbe629ecd0cc08337bf97debd8ef553b..b3f630dd5413950dcdafd7d39925114d27261841 100644 (file)
@@ -101,7 +101,7 @@ var mocks = {
                } else if ( req.query.callback ) {
                        callback = Promise.resolve( req.query.callback );
                } else if ( req.method === "GET" ) {
-                       callback = Promise.resolve( req.url.match( /^.+\/([^\/?.]+)\?.+$/ )[ 1 ] );
+                       callback = Promise.resolve( req.url.match( /^.+\/([^\/?]+)\?.+$/ )[ 1 ] );
                } else {
                        callback = getBody( req ).then( function( body ) {
                                return body.trim().replace( "callback=", "" );