Transports, streams and buffers #8
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added - Support for Buffer and Stream parameters and results - init() method is now optional: initialization will occur at first call if needed, whatever transport used - Usage of Yarn Fixed - client.exposed contains the exposed descriptor, even with local transport (only http transport used to have one) - Http errors are now wrapped in Boom errors with http transport (only local transport was propagating them) Changed - Reformat internals to introduce transports: better extensibility - Breaking: buildClient() throws only asynchronous validation errors (name and version when using local service). Previously those errors were thrown synchronously. - Breaking: when called, init() always refresh the exposed APIs. This allows to force initialization proactively when suspecting a remote service change. Previously it was feching exposed APIs only once. - Breaking: use got instead of request-promise-native. Thrown Error codes and message may be slighlty different. - Reformat CHANGELOG to follow Keep a Changelog recommandations - New documentation with latest docma v2.0.0 - Dependencies update
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added
getClient()'stransport.typeoption (default tolocaltransport)init()method is now optional: initialization will occur at first call if needed, whatever transport usedFixed
client.exposedcontains the exposed descriptor, even with local transport (only http transport used to have one)Changed
getClient()always throws synchronous validation errors.When called,
init()always refresh the exposed APIs.This allows to force initialization proactively when suspecting a remote service change.
Previously it was feching exposed APIs only once.
Use got instead of request-promise-native.
Thrown Error codes and message may be slighlty different.
Reformat CHANGELOG to follow Keep a Changelog recommandations
New documentation with latest docma v2.0.0
Dependencies update