cloudfoundry-community / slack-notification-resource

Concourse CI resource for sending notifications to Slack.
MIT License
75 stars 82 forks source link

Use `curl -T` to avoid argument length limit #64

Closed drlau closed 5 years ago

drlau commented 5 years ago

Signed-off-by: Conor Nosal cnosal@pivotal.io

If the payload is too large, curl will fail with argument list too long: curl. Write the payload to a temporary file and use curl -T to avoid this limitation.

ohlookadollar commented 5 years ago

@jhunt could you review this PR? Thanks