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

Support for metadata in S3 and POST:ing to a bucket #31

Merged
merged 3 commits into from
May 17, 2013

Conversation

cheif
Copy link
Contributor

@cheif cheif commented May 15, 2013

I have added support for metadata in S3 and support for uploading by POST:ing.

@spulec
Copy link
Collaborator

spulec commented May 16, 2013

Cool stuff.

Looks like the tests aren't passing: https://travis-ci.org/spulec/moto/builds/7178740

Can you also write some more tests for the new functionality?

Any reason you switched HTTPretty versions? I'm pretty sure that we need master.

@cheif
Copy link
Contributor Author

cheif commented May 16, 2013

Oh, sure, I will try to fix the tests during the day.

I think we switched the version of HTTPretty to get the server running, it
didn't work with 0.6.0a for us, but 0.6.0 did work.

On Thu, May 16, 2013 at 4:49 AM, Steve Pulec notifications@github.comwrote:

Cool stuff.

Looks like the tests aren't passing:
https://travis-ci.org/spulec/moto/builds/7178740

Can you also write some more tests for the new functionality?

Any reason you switched HTTPretty versions? I'm pretty sure that we need
master.


Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-17978036
.

Dan Berglund
0703-358911
berglund.dan@gmail.com

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 7de4399 on Memoto:master into fc02faa on spulec:master.

@cheif
Copy link
Contributor Author

cheif commented May 17, 2013

Now the test are passing, for me at least, and new tests are added for the new functionallity.

spulec added a commit that referenced this pull request May 17, 2013
Support for metadata in S3 and POST:ing to a bucket
@spulec spulec merged commit c14b8ca into getmoto:master May 17, 2013
@spulec
Copy link
Collaborator

spulec commented May 17, 2013

Looks great.

(Going to release a new minor version with this in a minute)

@cheif
Copy link
Contributor Author

cheif commented May 17, 2013

Oh, nice.
I'm currently trying to get real files working, they work if you just run
the server, but HTTPretty seems to lack support for files.
Will open a new pull request when i'm happy with it :)

On Fri, May 17, 2013 at 3:48 PM, Steve Pulec notifications@github.comwrote:

Looks great.

(Going to release a new minor version with this in a minute)


Reply to this email directly or view it on GitHubhttps://github.com//pull/31#issuecomment-18062257
.

Dan Berglund
0703-358911
berglund.dan@gmail.com

@spulec
Copy link
Collaborator

spulec commented May 17, 2013

Cool, looking forward to it.

I'm actually a committer for HTTPretty too so let me know if I can be any help there.

spulec pushed a commit that referenced this pull request Mar 5, 2020
macnev2013 pushed a commit to macnev2013/moto that referenced this pull request Jul 20, 2021
bblommers added a commit that referenced this pull request Jul 26, 2021
* fix OPTIONS requests on non-existing API GW integrations

* add cloudformation models for API Gateway deployments

* bump version

* add backdoor to return CloudWatch metrics

* Updating implementation coverage

* Updating implementation coverage

* add cloudformation models for API Gateway deployments

* Updating implementation coverage

* Updating implementation coverage

* Implemented get-caller-identity returning real data depending on the access key used.

* bump version

* minor fixes

* fix Number data_type for SQS message attribute

* fix handling of encoding errors

* bump version

* make CF stack queryable before starting to initialize its resources

* bump version

* fix integration_method for API GW method integrations

* fix undefined status in CF FakeStack

* Fix apigateway issues with terraform v0.12.21
* resource_methods -> add handle for "DELETE" method
* integrations -> fix issue that "httpMethod" wasn't included in body request (this value was set as the value from refer method resource)

* bump version

* Fix setting http method for API gateway integrations (#6)

* bump version

* remove duplicate methods

* add storage class to S3 Key when completing multipart upload (#7)

* fix SQS performance issues; bump version

* add pagination to SecretsManager list-secrets (#9)

* fix default parameter groups in RDS

* fix adding S3 metadata headers with names containing dots (#13)

* Updating implementation coverage

* Updating implementation coverage

* add cloudformation models for API Gateway deployments

* Updating implementation coverage

* Updating implementation coverage

* Implemented get-caller-identity returning real data depending on the access key used.

* make CF stack queryable before starting to initialize its resources

* bump version

* remove duplicate methods

* fix adding S3 metadata headers with names containing dots (#13)

* Update amis.json to support EKS AMI mocks (#15)

* fix PascalCase for boolean value in ListMultipartUploads response (#17); fix _get_multi_param to parse nested list/dict query params

* determine non-zero container exit code in Batch API

* support filtering by dimensions in CW get_metric_statistics

* fix storing attributes for ELBv2 Route entities; API GW refactorings for TF tests

* add missing fields for API GW resources

* fix error messages for Route53 (TF-compat)

* various fixes for IAM resources (tf-compat)

* minor fixes for API GW models (tf-compat)

* minor fixes for API GW responses (tf-compat)

* add s3 exception for bucket notification filter rule validation

* change the way RESTErrors generate the response body and content-type header

* fix lint errors and disable "black" syntax enforcement

* remove return type hint in RESTError.get_body

* add RESTError XML template for IAM exceptions

* add support for API GW minimumCompressionSize

* fix casing getting PrivateDnsEnabled API GW attribute

* minor fixes for error responses

* fix escaping special chars for IAM role descriptions (tf-compat)

* minor fixes and tagging support for API GW and ELB v2 (tf-compat)

* Merge branch 'master' into localstack

* add "AlarmRule" attribute to enable support for composite CloudWatch metrics

* fix recursive parsing of complex/nested query params

* bump version

* add API to delete S3 website configurations (#18)

* use dict copy to allow parallelism and avoid concurrent modification exceptions in S3

* fix precondition check for etags in S3 (#19)

* minor fix for user filtering in Cognito

* fix API Gateway error response; avoid returning empty response templates (tf-compat)

* support tags and tracingEnabled attribute for API GW stages

* fix boolean value in S3 encryption response (#20)

* fix connection arn structure

* fix api destination arn structure

* black format

* release 2.0.3.37

* fix s3 exception tests

see botocore/parsers.py:1002 where RequestId is removed from parsed

* remove python 2 from build action

* add test failure annotations in build action

* fix events test arn comparisons

* fix s3 encryption response test

* return default value "0" if EC2 availableIpAddressCount is empty

* fix extracting SecurityGroupIds for EC2 VPC endpoints

* support deleting/updating API Gateway DomainNames

* fix(events): Return empty string instead of null when no pattern is specified in EventPattern (tf-compat) (#22)

* fix logic and revert CF changes to get tests running again (#21)

* add support for EC2 customer gateway API (#25)

* add support for EC2 Transit Gateway APIs (#24)

* feat(logs): add `kmsKeyId` into `LogGroup` entity (#23)

* minor change in ELBv2 logic to fix tests

* feat(events): add APIs to describe and delete CloudWatch Events connections (#26)

* add support for EC2 transit gateway route tables (#27)

* pass transit gateway route table ID in Describe API, minor refactoring (#29)

* add support for EC2 Transit Gateway Routes (#28)

* fix region on ACM certificate import (#31)

* add support for EC2 transit gateway attachments (#30)

* add support for EC2 Transit Gateway VPN attachments (#32)

* fix account ID for logs API

* add support for DeleteOrganization API

* feat(events): store raw filter representation for CloudWatch events patterns (tf-compat) (#36)

* feat(events): add support to describe/update/delete CloudWatch API destinations (#35)

* add Cognito UpdateIdentityPool, CW Logs PutResourcePolicy

* feat(events): add support for tags in EventBus API (#38)

* fix parameter validation for Batch compute environments (tf-compat)

* revert merge conflicts in IMPLEMENTATION_COVERAGE.md

* format code using black

* restore original README; re-enable and fix CloudFormation tests

* restore tests and old logic for CF stack parameters from SSM

* parameterize RequestId/RequestID in response messages and revert related test changes

* undo LocalStack-specific adaptations

* minor fix

* Update CodeCov config to reflect removal of Py2

* undo change related to CW metric filtering; add additional test for CW metric statistics with dimensions

* Terraform - Extend whitelist of running tests

Co-authored-by: acsbendi <acsbendi28@gmail.com>
Co-authored-by: Phan Duong <duongpv@outlook.com>
Co-authored-by: Thomas Rausch <thomas@thrau.at>
Co-authored-by: Macwan Nevil <macnev2013@gmail.com>
Co-authored-by: Dominik Schubert <dominik.schubert91@gmail.com>
Co-authored-by: Gonzalo Saad <saad.gonzalo.ale@gmail.com>
Co-authored-by: Mohit Alonja <monty16597@users.noreply.github.com>
Co-authored-by: Miguel Gagliardo <migag9@gmail.com>
Co-authored-by: Bert Blommers <info@bertblommers.nl>
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

3 participants