Skip to content

Conversation

@forabi
Copy link
Contributor

@forabi forabi commented Apr 10, 2018

This PR adds support for wildcards in the binaryMimeTypes parameter. This allows for more flexibility when defining which responses should be base64-encoded.

forabi added 3 commits April 10, 2018 21:10
This PR adds support for wildcards in the `binaryMimeTypes` parameter. This allows for more flexibility when defining which responses should be base64-encoded.
@forabi forabi changed the title feat: support wildcards and extensions in binary Types feat: support wildcards and extensions in binary types Apr 10, 2018
@brettstack
Copy link
Collaborator

Nice! I was going to add this this month. Could you add tests for this?

@forabi
Copy link
Contributor Author

forabi commented Apr 10, 2018

@brettstack I added two tests for wildcards and extensions.

Copy link
Collaborator

@brettstack brettstack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the contribution!

@brettstack brettstack merged commit 8879cf3 into CodeGenieApp:master Apr 10, 2018
brettstack pushed a commit that referenced this pull request Apr 10, 2018
<a name="3.2.0"></a>
# [3.2.0](v3.1.3...v3.2.0) (2018-04-10)

### Bug Fixes

* **example:** add ca-central-1 to list of regions available ([#133](#133)) ([8cfd094](8cfd094))

### Features

* **binary:** allow wildcards and extensions in binary types ([#136](#136)) ([8879cf3](8879cf3))
OneDev0411 added a commit to OneDev0411/serverless-express that referenced this pull request Mar 16, 2023
<a name="3.2.0"></a>
# [3.2.0](CodeGenieApp/serverless-express@v3.1.3...v3.2.0) (2018-04-10)

### Bug Fixes

* **example:** add ca-central-1 to list of regions available ([#133](CodeGenieApp/serverless-express#133)) ([8cfd094](CodeGenieApp/serverless-express@8cfd094))

### Features

* **binary:** allow wildcards and extensions in binary types ([#136](CodeGenieApp/serverless-express#136)) ([8879cf3](CodeGenieApp/serverless-express@8879cf3))
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.

2 participants