Skip to content
This repository has been archived by the owner on Mar 27, 2024. It is now read-only.

add content type to webhook request #3159

Merged
merged 1 commit into from
Jan 30, 2023

Conversation

MajdT51
Copy link
Contributor

@MajdT51 MajdT51 commented Feb 10, 2022

Title:
Webhook message has no content type

Description:
Add the content type to the request header in webhook
Issue: #3158

@codecov
Copy link

codecov bot commented Feb 10, 2022

Codecov Report

Merging #3159 (a1f4697) into main (4b0d40d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #3159   +/-   ##
=======================================
  Coverage   87.98%   87.98%           
=======================================
  Files         329      329           
  Lines       45261    45263    +2     
=======================================
+ Hits        39821    39823    +2     
  Misses       4024     4024           
  Partials     1416     1416           
Impacted Files Coverage Δ
pkg/controller/webnotifier/webhook.go 93.75% <100.00%> (+0.27%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@sudeshrshetty
Copy link
Contributor

@MajdT51 thanks for your contribution, please rebase your PR

@MajdT51
Copy link
Contributor Author

MajdT51 commented Aug 16, 2022

@MajdT51 thanks for your contribution, please rebase your PR

Thanks. It's done

@fqutishat fqutishat merged commit 9b1e5b7 into hyperledger-archives:main Jan 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants