Skip to content

fix: missing property on objectColorizer - #625

Merged
jsumners merged 1 commit into
pinojs:masterfrom
IronGeek:fix/custom-colors
Oct 10, 2025
Merged

fix: missing property on objectColorizer#625
jsumners merged 1 commit into
pinojs:masterfrom
IronGeek:fix/custom-colors

Conversation

@IronGeek

@IronGeek IronGeek commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

When customColors option is specified to pino-pretty, the following error is thrown while prettifying object:

TypeError: objectColorizer.property is not a function
                    at .../pino-pretty/lib/utils/prettify-object.js:95:44
                    at Array.forEach (<anonymous>)
                    at prettifyObject (.../pino-pretty/lib/utils/prettify-object.js:83:27)
                    at TestContext.<anonymous> (.../pino-pretty/lib/utils/prettify-object.test.js:170:15)
                    at Test.runInAsyncScope (node:async_hooks:214:14)
                    at Test.run (node:internal/test_runner/test:1106:25)
                    at Test.processPendingSubtests (node:internal/test_runner/test:788:18)
                    at Test.postRun (node:internal/test_runner/test:1235:19)
                    at Test.run (node:internal/test_runner/test:1163:12)
                    at async Test.processPendingSubtests (node:internal/test_runner/test:788:7)

It seems the property field is not properly set on the objectColorizer.
This PR addresses this issue.

Fixes #617

jsumners
jsumners approved these changes Oct 10, 2025
View reviewed changes

@jsumners jsumners left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you.

@jsumners
jsumners merged commit f037355 into pinojs:master Oct 10, 2025
5 checks passed
Blankll added a commit to geek-fun/serverlessinsight that referenced this pull request Dec 8, 2025
@Blankll @snyk-bot
[Snyk] Upgrade pino-pretty from 13.1.1 to 13.1.2 (#90)
b84c530 
![snyk-top-banner](https://res.cloudinary.com/snyk/image/upload/r-d/scm-platform/snyk-pull-requests/pr-banner-default.svg)


<h3>Snyk has created this PR to upgrade pino-pretty from 13.1.1 to
13.1.2.</h3>

:information_source: Keep your dependencies up-to-date. This makes it
easier to fix existing vulnerabilities and to more quickly identify and
fix newly disclosed vulnerabilities when they affect your project.

<hr/>


- The recommended version is **1 version** ahead of your current
version.

- The recommended version was released **a month ago**.




<details>
<summary><b>Release notes</b></summary>
<br/>
  <details>
    <summary>Package name: <b>pino-pretty</b></summary>
    <ul>
      <li>
<b>13.1.2</b> - <a
href="https://redirect.github.com/pinojs/pino-pretty/releases/tag/v13.1.2">2025-10-10</a></br><h2>What's
Changed</h2>
<ul>
<li>build(deps-dev): bump typescript from 5.8.3 to 5.9.2 by <a
class="user-mention notranslate" data-hovercard-type="organization"
data-hovercard-url="/orgs/dependabot/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/dependabot">@ dependabot</a>[bot] in
<a class="issue-link js-issue-link" data-error-text="Failed to load
title" data-id="3283150445" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#609"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/609/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/609">#609</a></li>
<li>build(deps): bump actions/checkout from 4 to 5 by <a
class="user-mention notranslate" data-hovercard-type="organization"
data-hovercard-url="/orgs/dependabot/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/dependabot">@ dependabot</a>[bot] in
<a class="issue-link js-issue-link" data-error-text="Failed to load
title" data-id="3472667455" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#621"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/621/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/621">#621</a></li>
<li>build(deps-dev): bump tsd from 0.32.0 to 0.33.0 by <a
class="user-mention notranslate" data-hovercard-type="organization"
data-hovercard-url="/orgs/dependabot/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/dependabot">@ dependabot</a>[bot] in
<a class="issue-link js-issue-link" data-error-text="Failed to load
title" data-id="3472670715" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#622"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/622/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/622">#622</a></li>
<li>fix: allow esm import for isColorSupported by <a class="user-mention
notranslate" data-hovercard-type="user"
data-hovercard-url="/users/JoeCap08055/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/JoeCap08055">@ JoeCap08055</a> in <a
class="issue-link js-issue-link" data-error-text="Failed to load title"
data-id="3311704460" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#616"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/616/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/616">#616</a></li>
<li>Use neostandard and remove pre-commit by <a class="user-mention
notranslate" data-hovercard-type="user"
data-hovercard-url="/users/jsumners/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/jsumners">@ jsumners</a> in <a
class="issue-link js-issue-link" data-error-text="Failed to load title"
data-id="3494895695" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#624"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/624/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/624">#624</a></li>
<li>fix: missing <code>property</code> on objectColorizer by <a
class="user-mention notranslate" data-hovercard-type="user"
data-hovercard-url="/users/IronGeek/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/IronGeek">@ IronGeek</a> in <a
class="issue-link js-issue-link" data-error-text="Failed to load title"
data-id="3501922582" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#625"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/625/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/625">#625</a></li>
</ul>
<h2>New Contributors</h2>
<ul>
<li><a class="user-mention notranslate" data-hovercard-type="user"
data-hovercard-url="/users/JoeCap08055/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/JoeCap08055">@ JoeCap08055</a> made
their first contribution in <a class="issue-link js-issue-link"
data-error-text="Failed to load title" data-id="3311704460"
data-permission-text="Title is private"
data-url="pinojs/pino-pretty#616"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/616/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/616">#616</a></li>
<li><a class="user-mention notranslate" data-hovercard-type="user"
data-hovercard-url="/users/IronGeek/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/IronGeek">@ IronGeek</a> made their
first contribution in <a class="issue-link js-issue-link"
data-error-text="Failed to load title" data-id="3501922582"
data-permission-text="Title is private"
data-url="pinojs/pino-pretty#625"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/625/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/625">#625</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a class="commit-link"
href="https://redirect.github.com/pinojs/pino-pretty/compare/v13.1.1...v13.1.2"><tt>v13.1.1...v13.1.2</tt></a></p>
      </li>
      <li>
<b>13.1.1</b> - <a
href="https://redirect.github.com/pinojs/pino-pretty/releases/tag/v13.1.1">2025-07-30</a></br><h2>What's
Changed</h2>
<ul>
<li>Revert "chore: upgrade dateformat" by <a class="user-mention
notranslate" data-hovercard-type="user"
data-hovercard-url="/users/Uzlopak/hovercard"
data-octo-click="hovercard-link-click"
data-octo-dimensions="link_type:self"
href="https://redirect.github.com/Uzlopak">@ Uzlopak</a> in <a
class="issue-link js-issue-link" data-error-text="Failed to load title"
data-id="3276723338" data-permission-text="Title is private"
data-url="pinojs/pino-pretty#607"
data-hovercard-type="pull_request"
data-hovercard-url="/pinojs/pino-pretty/pull/607/hovercard"
href="https://redirect.github.com/pinojs/pino-pretty/pull/607">#607</a></li>
</ul>
<p><strong>Full Changelog</strong>: <a class="commit-link"
href="https://redirect.github.com/pinojs/pino-pretty/compare/v13.1.0...v13.1.1"><tt>v13.1.0...v13.1.1</tt></a></p>
      </li>
    </ul>
from <a
href="https://redirect.github.com/pinojs/pino-pretty/releases">pino-pretty
GitHub release notes</a>
  </details>
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with
your project.
> - This PR was automatically created by Snyk using the credentials of a
real user.

---

**Note:** _You are seeing this because you or someone else with access
to this repository has authorized Snyk to open upgrade PRs._

**For more information:** <img
src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiIxZjZlNzJhZS04NGE5LTRiNjgtYTFjMS02OGZmMDcyMjU0NGIiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjFmNmU3MmFlLTg0YTktNGI2OC1hMWMxLTY4ZmYwNzIyNTQ0YiJ9fQ=="
width="0" height="0"/>

> - 🧐 [View latest project
report](https://app.snyk.io/org/blankll/project/9c72c875-e7a2-4e68-85a9-7b26a5bc5b32?utm_source&#x3D;github&amp;utm_medium&#x3D;referral&amp;page&#x3D;upgrade-pr)
> - 📜 [Customise PR
templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates?utm_source=&utm_content=fix-pr-template)
> - 🛠 [Adjust upgrade PR
settings](https://app.snyk.io/org/blankll/project/9c72c875-e7a2-4e68-85a9-7b26a5bc5b32/settings/integration?utm_source&#x3D;github&amp;utm_medium&#x3D;referral&amp;page&#x3D;upgrade-pr)
> - 🔕 [Ignore this dependency or unsubscribe from future upgrade
PRs](https://app.snyk.io/org/blankll/project/9c72c875-e7a2-4e68-85a9-7b26a5bc5b32/settings/integration?pkg&#x3D;pino-pretty&amp;utm_source&#x3D;github&amp;utm_medium&#x3D;referral&amp;page&#x3D;upgrade-pr#auto-dep-upgrades)

[//]: #
'snyk:metadata:{"breakingChangeRiskLevel":null,"FF_showPullRequestBreakingChanges":false,"FF_showPullRequestBreakingChangesWebSearch":false,"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"pino-pretty","from":"13.1.1","to":"13.1.2"}],"env":"prod","hasFixes":false,"isBreakingChange":false,"isMajorUpgrade":false,"issuesToFix":[],"prId":"1f6e72ae-84a9-4b68-a1c1-68ff0722544b","prPublicId":"1f6e72ae-84a9-4b68-a1c1-68ff0722544b","packageManager":"npm","priorityScoreList":[],"projectPublicId":"9c72c875-e7a2-4e68-85a9-7b26a5bc5b32","projectUrl":"https://app.snyk.io/org/blankll/project/9c72c875-e7a2-4e68-85a9-7b26a5bc5b32?utm_source=github&utm_medium=referral&page=upgrade-pr","prType":"upgrade","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":[],"type":"auto","upgrade":[],"upgradeInfo":{"versionsDiff":1,"publishedDate":"2025-10-10T10:20:40.911Z"},"vulns":[]}'

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsumners jsumners jsumners approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Unhandled 'error' event when using pino-pretty 13.1.1

2 participants

@IronGeek @jsumners