Allow different key sizes in REST API #228
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
vovac12
force-pushed
the
227-allow-different-key-types-in-rest
branch
from
e829277
to
9a6ec44
Compare
Use uuid from rocket_contrib
Fix for escape sequences GUID without brackets
…o 227-allow-different-key-types-in-rest
idruzhitskiy
reviewed
Jul 6, 2021
…nt-key-types-in-rest
…o 227-allow-different-key-types-in-rest
piakushin
suggested changes
Jul 27, 2021
…nt-key-types-in-rest
…ollo/bob into 227-allow-different-key-types-in-rest
…nt-key-types-in-rest
…o 227-allow-different-key-types-in-rest
piakushin
approved these changes
Aug 13, 2021
piakushin
added a commit
that referenced
this pull request
Aug 13, 2021
* add key_size as constant * add bytes as blob key to proto * add keysize to bobc and bobp * change KEY_SIZE to BOB_KEY_SIZE * add key size to docker * fix typo * use tryinto instead of manual copy * Fix mod op for slice * Test build * Fix From<u64> trait implementation * update changelog * fix use of new bobkey in http * use le encoding * fix le impl * fix key saturation * update vdisk id retrieval * fix key formatting * fix smaller size behaviour in bobp and bobc * add panic in bobc and bobp in case of key truncation * fix usage of keysize in bobc * remove use of extend in favor of resize * Allow different key sizes in REST API (#228) * Different key sizes in REST API * Add key size validation * Big integer parsing * Remove manual number parsing Use uuid from rocket_contrib * Error explanation instead of 404 Fix for escape sequences GUID without brackets * Fix hex bytes order * Update CHANGELOG.md * Key overflow check before copying * Fix misleading function name Co-authored-by: pyakushin <jetpilot412@gmail.com> * pass keysize from command line * re-add env vars to bobp and bobc * Return parse error instead of Not found. Fix key parsing * Update pearl to 0.6.1 Co-authored-by: Pavel Perestoronin <p_perestoronin@mail.ru> Co-authored-by: Vladimir Stepanenko <vovac12@gmail.com> Co-authored-by: pyakushin <jetpilot412@gmail.com>
piakushin
added a commit
that referenced
this pull request
Oct 19, 2021
* add key_size as constant * add bytes as blob key to proto * add keysize to bobc and bobp * change KEY_SIZE to BOB_KEY_SIZE * add key size to docker * fix typo * use tryinto instead of manual copy * Fix mod op for slice * Test build * Fix From<u64> trait implementation * update changelog * fix use of new bobkey in http * use le encoding * fix le impl * fix key saturation * update vdisk id retrieval * fix key formatting * fix smaller size behaviour in bobp and bobc * add panic in bobc and bobp in case of key truncation * fix usage of keysize in bobc * remove use of extend in favor of resize * update gitgnore for complex filestructure * Allow different key sizes in REST API (#228) * Different key sizes in REST API * Add key size validation * Big integer parsing * Remove manual number parsing Use uuid from rocket_contrib * Error explanation instead of 404 Fix for escape sequences GUID without brackets * Fix hex bytes order * Update CHANGELOG.md * Key overflow check before copying * Fix misleading function name Co-authored-by: pyakushin <jetpilot412@gmail.com> * pass keysize from command line * re-add env vars to bobp and bobc * update gitignore * add matrics accumulator * refactor warnings * propagate shared metrics * split metrics exporter into accumulator and recorder * update changelog * fix active wait * update write lock in accumulator * reorganiza metrics collection * fix discussions * update http method * update http interface * disable preservation of mean time * reduce number of metric models * add timestamps to metrics * reduce timestamp calculation calls * add cap on metrics received in one period Co-authored-by: Pavel Perestoronin <p_perestoronin@mail.ru> Co-authored-by: Vladimir Stepanenko <vovac12@gmail.com> Co-authored-by: pyakushin <jetpilot412@gmail.com>
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.
Closes #227