Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add exposeAdditionalProperties option #8

Open
wants to merge 1 commit into
base: master
from

Conversation

@LinusU
Copy link
Member

LinusU commented Aug 29, 2018

see #7 for motivation

@coveralls

This comment has been minimized.

Copy link

coveralls commented Aug 29, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling fe4ea9a on LinusU:expose-additional-properties into df765a5 on expressjs:master.

@kilhage

This comment has been minimized.

Copy link

kilhage commented Sep 24, 2018

+1

@LinusU

This comment has been minimized.

Copy link
Member Author

LinusU commented Nov 21, 2018

ping @jonathanong, what do you think about this? ☺️

@gillesdemey

This comment has been minimized.

Copy link

gillesdemey commented Dec 20, 2018

I think it would be useful to specify either a truthy/falsy value or an array of properties to whitelist.

Some packages add additional error properties that you might not want to propagate to the user.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.