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
HELP-40835: storage updates #5370
Merged
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
lazedo
pushed a commit
that referenced
this pull request
Jan 3, 2019
* remove UI metadata * start to define http handler schema * update spec and naming * log the handler being used * update the attachments to accept HTTP handler * handle decoding error response * make sure the verb is properly created * create per-test IDs * remove redundant call * start to exercise the storage API * add take_value/{2,3} to kz_json * add httpd server for recv http requests from kazoo * start httpd * verify returned contents match * add come cleanup help. start httpd with log id * add logging metadata to help correlate logs * refactor compose voicemail into smaller functions * some dialyzing * separate storing meta from store url creation * move encoding multipart to kz_http_util * handle undefined owner_id * add a blocking wait for requests to come in * create a vmbox and message to trigger storage * handle storage put_attachment return * fix binding key for storage plan cache * add http storage backend doc * vmbox API test module * test that we recv the MP3 on our http server * make sure vmboxes are started * fix code complaint * update schema doc and what not * dialyzer updates * add empty response headers on error * formatting * spelling updates * handle multipart response * build multipart body if configured to do so * ask for multipart bodies * use cowlib boundary creator * validate multipart response * document multipart * update schema generated docs * separate concerns a bit * set default headers if not supplied by the handler module * detect the crossbar url/port better * accept a port argument * log successful store * when fetching owner's plan, if missing, try account's * add default resp_headers
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.
Specifically centered around the HTTP storage backend but some fixes that apply to all storage backends as well.