Skip to content

Minor refactorings#4

Open
rydnr wants to merge 3 commits intojvdsandt:masterfrom
rydnr:master
Open

Minor refactorings#4
rydnr wants to merge 3 commits intojvdsandt:masterfrom
rydnr:master

Conversation

@rydnr
Copy link

@rydnr rydnr commented Apr 21, 2021

  • Moved AWSXmlObject to AWS-Core
  • Minor refactoring in AWSService to help subclasses know what to override.
  • Created a trait with the code to handle XML error messages.
  • AWSSQSService and AWSS3Service delegate in the trait the handling of error messages.

I'm working on supporting CloudFormation, and these changes make my life a bit easier.

rydnr added 2 commits April 21, 2021 05:39
- Minor refactoring in AWSService to help subclasses know what to override.
@rydnr
Copy link
Author

rydnr commented Apr 21, 2021

Error handling code in AWSSQSService and AWSS3Service is not identical. Closing this PR until I fix it.

@rydnr rydnr closed this Apr 21, 2021
@rydnr rydnr reopened this Apr 21, 2021
@rydnr
Copy link
Author

rydnr commented Apr 21, 2021

Reviewing it again, it seems they are compatible.

…it named AWSTFormUrlEncoding, so it can be used without adding a dependency to SQS package.
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.

1 participant