Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: prevent circular structure exception #312

Merged
merged 1 commit into from
Jan 7, 2020
Merged

fix: prevent circular structure exception #312

merged 1 commit into from
Jan 7, 2020

Conversation

DanielRuf
Copy link
Contributor

Type:
bugfix

Scope:
local-storage

The following has been addressed in the PR:

  • There is a related issue? Yes
  • Unit or Functional tests are included in the PR: No

Description:

bunyan already sanitizes objects and prevents circular reference exceptions.

Resolves verdaccio/verdaccio#1639

@DanielRuf DanielRuf added bug Something isn't working plugin: local-storage labels Jan 6, 2020
@DanielRuf DanielRuf requested review from a team January 6, 2020 11:51
@codecov
Copy link

codecov bot commented Jan 6, 2020

Codecov Report

Merging #312 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #312   +/-   ##
=======================================
  Coverage   65.17%   65.17%           
=======================================
  Files          26       26           
  Lines        1407     1407           
  Branches      205      205           
=======================================
  Hits          917      917           
  Misses        487      487           
  Partials        3        3
Flag Coverage Δ
#core 88.18% <ø> (ø) ⬆️
#plugins 58.16% <100%> (ø) ⬆️
Impacted Files Coverage Δ
plugins/local-storage/src/local-database.ts 85.62% <100%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb4e48a...d8f44fa. Read the comment docs.

Copy link
Member

@anikethsaha anikethsaha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@juanpicado juanpicado merged commit f565461 into verdaccio:master Jan 7, 2020
@juanpicado
Copy link
Member

awesome @DanielRuf :)

@DanielRuf DanielRuf deleted the fix/1639-logger-circular-structure-exception branch January 7, 2020 05:09
@juanpicado
Copy link
Member

juanpicado added a commit to verdaccio/verdaccio that referenced this pull request Jan 8, 2020
Fixed at verdaccio/monorepo#312

Fixes #1639

Co-Authored-By: Daniel Ruf <danielruf@users.noreply.github.com>
juanpicado added a commit to verdaccio/verdaccio that referenced this pull request Jan 8, 2020
fix: converting circular structure to json (#1663)

Fixed at verdaccio/monorepo#312

Fixes #1639

Co-Authored-By: Daniel Ruf <danielruf@users.noreply.github.com>

Co-authored-by: Daniel Ruf <827205+DanielRuf@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Something isn't working plugin: local-storage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can not start verdaccio with docker-compose, version 4.4.0
4 participants