aboutsummaryrefslogtreecommitdiff
path: root/src/api
Commit message (Collapse)AuthorAgeFilesLines
* K2V: Fix `end` parameter and add tests (fix #305)fix-k2v-305Alex Auvolat2022-05-171-1/+5
|
* First implementation of K2V (#293)Alex2022-05-1030-981/+2401
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | **Specification:** View spec at [this URL](https://git.deuxfleurs.fr/Deuxfleurs/garage/src/branch/k2v/doc/drafts/k2v-spec.md) - [x] Specify the structure of K2V triples - [x] Specify the DVVS format used for causality detection - [x] Specify the K2V index (just a counter of number of values per partition key) - [x] Specify single-item endpoints: ReadItem, InsertItem, DeleteItem - [x] Specify index endpoint: ReadIndex - [x] Specify multi-item endpoints: InsertBatch, ReadBatch, DeleteBatch - [x] Move to JSON objects instead of tuples - [x] Specify endpoints for polling for updates on single values (PollItem) **Implementation:** - [x] Table for K2V items, causal contexts - [x] Indexing mechanism and table for K2V index - [x] Make API handlers a bit more generic - [x] K2V API endpoint - [x] K2V API router - [x] ReadItem - [x] InsertItem - [x] DeleteItem - [x] PollItem - [x] ReadIndex - [x] InsertBatch - [x] ReadBatch - [x] DeleteBatch **Testing:** - [x] Just a simple Python script that does some requests to check visually that things are going right (does not contain parsing of results or assertions on returned values) - [x] Actual tests: - [x] Adapt testing framework - [x] Simple test with InsertItem + ReadItem - [x] Test with several Insert/Read/DeleteItem + ReadIndex - [x] Test all combinations of return formats for ReadItem - [x] Test with ReadBatch, InsertBatch, DeleteBatch - [x] Test with PollItem - [x] Test error codes - [ ] Fix most broken stuff - [x] test PollItem broken randomly - [x] when invalid causality tokens are given, errors should be 4xx not 5xx **Improvements:** - [x] Descending range queries - [x] Specify - [x] Implement - [x] Add test - [x] Batch updates to index counter - [x] Put K2V behind `k2v` feature flag Co-authored-by: Alex Auvolat <alex@adnab.me> Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/293 Co-authored-by: Alex <alex@adnab.me> Co-committed-by: Alex <alex@adnab.me>
* fix clippyfix-multipart-fragmentationAlex Auvolat2022-04-191-1/+1
|
* Prettier code for defragmentationAlex Auvolat2022-04-191-104/+102
|
* Defragmentation in UploadPartCopy: first pass (not pretty but it compiles)Alex Auvolat2022-04-191-63/+122
|
* remove size limitation in UploadPartCopy (#280)withinboredom2022-04-191-12/+0
| | | | | | | | | | | This removes the >1mb s3_copy restriction. This restriction doesn't seem to be documented anywhere (I could be wrong). It also causes some software to fail (such as #248). Co-authored-by: Rob Landers <landers.robert@gmail.com> Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/280 Co-authored-by: withinboredom <landers.robert@gmail.com> Co-committed-by: withinboredom <landers.robert@gmail.com>
* fix a clippy lintAlex Auvolat2022-03-281-1/+1
|
* Move block manager to separate moduleAlex Auvolat2022-03-232-1/+2
|
* add test framework for arbitraty S3 requeststrinity-1686a2022-03-232-9/+8
| | | | and implement some basic test with it
* garage_api: Update streaming payload stream unit testsKokaKiwi2022-03-231-1/+1
|
* garage_api: Handle streaming payload early in request handlingKokaKiwi2022-03-233-57/+61
|
* Fix some new clippy lintsfix-resyncAlex Auvolat2022-03-144-12/+6
|
* Bump version to 0.7 because of incompatible NetappAlex Auvolat2022-03-141-4/+4
|
* Add metrics to web endpointAlex Auvolat2022-03-141-10/+11
|
* Add metrics to API endpointAlex Auvolat2022-03-141-19/+94
|
* Remove strum crate dependency; add protobuf nix dependencyAlex Auvolat2022-03-143-7/+2
|
* Update to Netapp 0.4 which supports distributed tracingAlex Auvolat2022-03-141-3/+7
|
* Add tracing integration with opentelemetryAlex Auvolat2022-03-144-4/+33
|
* More permissive OPTIONS on S3 APIv0.6.1better-corsAlex Auvolat2022-03-012-18/+48
|
* Add tracing output to signature calculationAlex Auvolat2022-02-281-0/+3
|
* Process CORS earlier in pipelineAlex Auvolat2022-02-283-6/+38
|
* Support for PostObject (#222)trinity-1686a2022-02-2110-69/+642
| | | | | | | | | | | | | | | | | | Add support for [PostObject](https://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectPOST.html) - [x] routing PostObject properly - [x] parsing multipart body - [x] validating signature - [x] validating policy - [x] validating content length - [x] actually saving data Co-authored-by: trinity-1686a <trinity@deuxfleurs.fr> Co-authored-by: Trinity Pointard <trinity.pointard@gmail.com> Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/222 Reviewed-by: Alex <alex@adnab.me> Co-authored-by: trinity-1686a <trinity.pointard@gmail.com> Co-committed-by: trinity-1686a <trinity.pointard@gmail.com>
* Add restriction on part ordering in CompleteMultipartUploadv0.6.0-rc1get-head-part-numberAlex Auvolat2022-01-251-0/+11
|
* Fix partnumberAlex Auvolat2022-01-251-91/+117
|
* Implement part_number for GetObjectAlex Auvolat2022-01-241-73/+146
|
* Multipart improvementsAlex Auvolat2022-01-244-31/+126
| | | | | - support part_number for HeadObject - add checks in complete_multipart_upload
* Handle OPTIONS on website endpointbucket-corsAlex Auvolat2022-01-243-33/+6
|
* Implement {Put,Get,Delete}BucketCors and CORS in generalAlex Auvolat2022-01-246-14/+474
| | | | | | - OPTIONS request against API endpoint - Returning corresponding CORS headers on API calls - Returning corresponding CORS headers on website GET's
* use clamp instead of min(max())Trinity Pointard2022-01-241-4/+3
|
* Implement ListPartsQuentin Dufour2022-01-212-4/+389
|
* Add ListPartsResult structureQuentin Dufour2022-01-201-6/+121
|
* Add date verification to presigned urls (#196)trinity-1686a2022-01-185-30/+66
| | | | | | | | | | fix #96 fix #162 by returning Forbidden instead Bad Request Co-authored-by: Trinity Pointard <trinity.pointard@gmail.com> Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/196 Co-authored-by: trinity-1686a <trinity.pointard@gmail.com> Co-committed-by: trinity-1686a <trinity.pointard@gmail.com>
* refactor s3_router and api_server to make unused Endpoint parameters more ↵trinity-1686a2022-01-172-239/+129
| | | | obvious
* Fix Multipart Upload with WinSCP (#164) (#193)Jill2022-01-171-2/+7
| | | | | | | | Closes #164. Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/193 Co-authored-by: Jill <kokakiwi@deuxfleurs.fr> Co-committed-by: Jill <kokakiwi@deuxfleurs.fr>
* Support STREAMING-AWS4-HMAC-SHA256-PAYLOAD (#64) (#156)Jill2022-01-179-75/+484
| | | | | | | | Closes #64. Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/156 Co-authored-by: Jill <kokakiwi@deuxfleurs.fr> Co-committed-by: Jill <kokakiwi@deuxfleurs.fr>
* Make use of website config, return error document on errorAlex Auvolat2022-01-131-1/+1
|
* Implement GetBucketWebsiteAlex Auvolat2022-01-132-19/+55
|
* Add quotes in returned etagsimprove-copiesAlex Auvolat2022-01-134-53/+45
|
* Testing for UploadPartCopies and bugfixes in AWS signaturesAlex Auvolat2022-01-132-3/+7
|
* Implement UploadPartCopyAlex Auvolat2022-01-134-54/+399
|
* Implement x-amz-copy-if-xxx copy preconditions and return more headers on ↵Alex Auvolat2022-01-132-0/+107
| | | | copy (fix #187)
* Implement ListMultipartUploads (#171)Quentin2022-01-126-253/+1087
| | | | | | | | | | | | | | | | | | | | | | | Implement ListMultipartUploads, also refactor ListObjects and ListObjectsV2. It took me some times as I wanted to propose the following things: - Using an iterator instead of the loop+goto pattern. I find it easier to read and it should enable some optimizations. For example, when consuming keys of a common prefix, we do many [redundant checks](https://git.deuxfleurs.fr/Deuxfleurs/garage/src/branch/main/src/api/s3_list.rs#L125-L156) while the only thing to do is to [check if the following key is still part of the common prefix](https://git.deuxfleurs.fr/Deuxfleurs/garage/src/branch/feature/s3-multipart-compat/src/api/s3_list.rs#L476). - Try to name things (see ExtractionResult and RangeBegin enums) and to separate concerns (see ListQuery and Accumulator) - An IO closure to make unit tests possibles. - Unit tests, to track regressions and document how to interact with the code - Integration tests with `s3api`. In the future, I would like to move them in Rust with the aws rust SDK. Merging of the logic of ListMultipartUploads and ListObjects was not a goal but a consequence of the previous modifications. Some points that we might want to discuss: - ListObjectsV1, when using pagination and delimiters, has a weird behavior (it lists multiple times the same prefix) with `aws s3api` due to the fact that it can not use our optimization to skip the whole prefix. It is independant from my refactor and can be tested with the commented `s3api` tests in `test-smoke.sh`. It probably has the same weird behavior on the official AWS S3 implementation. - Considering ListMultipartUploads, I had to "abuse" upload id marker to support prefix skipping. I send an `upload-id-marker` with the hardcoded value `include` to emulate your "including" token. - Some ways to test ListMultipartUploads with existing software (my tests are limited to s3api for now). Co-authored-by: Quentin Dufour <quentin@deuxfleurs.fr> Reviewed-on: https://git.deuxfleurs.fr/Deuxfleurs/garage/pulls/171 Co-authored-by: Quentin <quentin@dufour.io> Co-committed-by: Quentin <quentin@dufour.io>
* Clarify error messageAlex Auvolat2022-01-102-9/+10
|
* Fix unit testsAlex Auvolat2022-01-052-12/+44
|
* Fix some error codesAlex Auvolat2022-01-058-36/+53
|
* Implement DeleteBucketAlex Auvolat2022-01-053-6/+116
|
* Implement CreateBucketAlex Auvolat2022-01-052-16/+151
|
* Small changes in key model and refactoringnew-bucketsAlex Auvolat2022-01-042-5/+6
|
* New buckets for 0.6.0: make bucket id a SK and not a HK, CLI updatesAlex Auvolat2022-01-042-14/+10
|
* New buckets for 0.6.0: small changesAlex Auvolat2022-01-042-7/+5
| | | | | | | | | | | | - Fix bucket delete - fix merge of bucket creation date - Replace deletable with option in aliases Rationale: if two aliases point to conflicting bucket, resolving by making an arbitrary choice risks making data accessible when it shouldn't be. We'd rather resolve to deleting the alias until someone puts it back.