Skip to content

feat(cmn): keep raw body in requests#3550

Merged
mergify[bot] merged 2 commits intodevelopfrom
sc/cmn-raw-body
Sep 26, 2022
Merged

feat(cmn): keep raw body in requests#3550
mergify[bot] merged 2 commits intodevelopfrom
sc/cmn-raw-body

Conversation

@smartcontracts
Copy link
Contributor

Description
Minor modification that puts the raw body as a string on the request object. Useful for applications where the raw body is required rather than the parsed JSON body as given by bodyParser. For example, this is useful when verifying GitHub webhooks.

Minor modification that puts the raw body as a string on the request
object. Useful for applications where the raw body is required rather
than the parsed JSON body as given by bodyParser. For example, this is
useful when verifying GitHub webhooks.
@changeset-bot
Copy link

changeset-bot bot commented Sep 24, 2022

🦋 Changeset detected

Latest commit: 29635de

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 5 packages
Name Type
@eth-optimism/common-ts Patch
@eth-optimism/data-transport-layer Patch
@eth-optimism/drippie-mon Patch
@eth-optimism/message-relayer Patch
@eth-optimism/replica-healthcheck Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@smartcontracts
Copy link
Contributor Author

I need this specifically so that the bot I'm writing for the token list repository can verify that webhooks are coming from GitHub and not somewhere else

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2022

This PR has been added to the merge queue, and will be merged soon.

@mergify
Copy link
Contributor

mergify bot commented Sep 26, 2022

This PR is next in line to be merged, and will be merged as soon as checks pass.

@mergify mergify bot merged commit ce7da91 into develop Sep 26, 2022
@mergify mergify bot deleted the sc/cmn-raw-body branch September 26, 2022 18:53
@mergify mergify bot removed the on-merge-train label Sep 26, 2022
This was referenced Sep 29, 2022
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.

3 participants