Skip to content

Commit

Permalink
bugfix: fix sniffer include url rule match not working problem becaus…
Browse files Browse the repository at this point in the history
…e of missing return statement in predicate lambda (#56)
  • Loading branch information
fanjianquan authored Jan 13, 2025
1 parent d8b6ee4 commit a3eb9c7
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/api-sniffer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,9 @@ export class ApiSniffer {
const doesIncludeRuleMatches = !this.config.include
? true
: this.config.include.some((config) => {
log.info(`Matching include url ${request.url} with request ${config.url}`);
doesUrlMatch(config.url, request.url)
const doesMatch = doesUrlMatch(config.url, request.url)
log.info(`Matching include url ${config.url} with request ${request.url} => ${doesMatch ? 'YES' : 'NO'}`);
return doesMatch
});
const doesExcludeRuleMatches = !this.config.exclude
? true
Expand Down

0 comments on commit a3eb9c7

Please sign in to comment.