Skip to content

feat: improve error output by showing which URL caused the error#1740

Merged
paulmelnikow merged 1 commit intonock:masterfrom
martinfrancois:patch-master
Oct 7, 2019
Merged

feat: improve error output by showing which URL caused the error#1740
paulmelnikow merged 1 commit intonock:masterfrom
martinfrancois:patch-master

Conversation

@martinfrancois
Copy link
Copy Markdown
Contributor

After getting an error in the interceptor, mentioning URL's need to start with a slash, it was difficult to find out which URL was causing the issue. With this change, since the URL is included in the error message, it's easier to troubleshoot the issue.

=> Reopening to master by request of @paulmelnikow

…k#1735)

After getting an error in the interceptor, mentioning URL's need to start with a slash, it was difficult to find out which URL was causing the issue. With this change, since the URL is included in the error message, it's easier to troubleshoot the issue.
@paulmelnikow paulmelnikow merged commit f4a208b into nock:master Oct 7, 2019
@paulmelnikow
Copy link
Copy Markdown
Member

Thank you!

@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Oct 7, 2019

🎉 This PR is included in version 11.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

juninmd pushed a commit to juninmd/nock that referenced this pull request Mar 21, 2026
…k#1735) (nock#1740)

After getting an error in the interceptor, mentioning URL's need to start with a slash, it was difficult to find out which URL was causing the issue. With this change, since the URL is included in the error message, it's easier to troubleshoot the issue.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants