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 raw data support #14

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

adamhjk
Copy link

@adamhjk adamhjk commented Feb 4, 2020

Thank you so much for the library! It was (almost) exactly what we needed. Let us know if we can help further.

This commit adds the ability to return the raw topic and message buffer,
allowing easier handling of binary data from mqtt inside the graphql endpoint
itself.

Enable by setting the rawData option. If it is not set, it behaves
identically to the existing JSON/String path.

It also upgrades Mocha to 7.0.1, to resolve multiple security
vulnerabilities with Mocha.

This commit adds the ability to return the raw topic and message buffer,
allowing easier handling of binary data from mqtt inside the graphql endpoint
itself.

Enable by setting the `rawData` option. If it is not set, it behaves
identically to the existing JSON/String path.

It also upgrades Mocha to 7.0.1, to resolve multiple security
vulnerabilities with Mocha.
@adamhjk
Copy link
Author

adamhjk commented Feb 4, 2020

It looks like node 6 fails on modern versions of Mocha, but I suspect it is reasonable to no longer have support for it. Happy to do whatever makes sense for you.

It also appears like we are the first people to check in the package-lock - happy to remove it if you would rather not.

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.

None yet

1 participant