Skip to content

Conversation

@casperstr
Copy link
Contributor

Use safeStringify in favor of JSON.stringify to protect the user from crashes when some part of the options is circular.

@codecov
Copy link

codecov bot commented Jan 12, 2021

Codecov Report

Merging #47 (b75dfd6) into master (8f1f390) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #47   +/-   ##
=======================================
  Coverage   94.81%   94.81%           
=======================================
  Files           3        3           
  Lines         135      135           
  Branches       12       12           
=======================================
  Hits          128      128           
  Partials        7        7           
Impacted Files Coverage Δ
src/mongoose.ts 94.18% <100.00%> (ø)

@wdalmut wdalmut merged commit ee85f3d into wdalmut:master Jan 13, 2021
@wdalmut
Copy link
Owner

wdalmut commented Jan 13, 2021

👍 thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants