Skip to content

client(bootstrap): bootstrap webhook secret on init#652

Merged
bobheadxi merged 1 commit into
masterfrom
client/set-webhook-on-init
Mar 19, 2020
Merged

client(bootstrap): bootstrap webhook secret on init#652
bobheadxi merged 1 commit into
masterfrom
client/set-webhook-on-init

Conversation

@bobheadxi

Copy link
Copy Markdown
Member

🎟️ Ticket(s): Closes #394


👷 Changes

Client now provides the daemon with a webhook secret upon bootstrapping so that the daemon can start accept webhooks immediately.

🔦 Testing Instructions

inertia [remote] init

@bobheadxi bobheadxi added the pr: finalized needs review and final approval label Mar 19, 2020
@bobheadxi bobheadxi requested a review from a team March 19, 2020 20:52
@bobheadxi bobheadxi changed the title client(bootstrap): provide daemon with webhook secret on init client(bootstrap): bootstrap webhook secret on init Mar 19, 2020
@codecov

codecov Bot commented Mar 19, 2020

Copy link
Copy Markdown

Codecov Report

Merging #652 into master will decrease coverage by 0.02%.
The diff coverage is 33.34%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #652      +/-   ##
==========================================
- Coverage   55.58%   55.56%   -0.01%     
==========================================
  Files          67       67              
  Lines        3392     3393       +1     
==========================================
  Hits         1885     1885              
- Misses       1264     1265       +1     
  Partials      243      243              
Impacted Files Coverage Δ
daemon/inertiad/daemon/up.go 0.00% <0.00%> (ø)
client/sshc.go 35.22% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 818dbde...21b64df. Read the comment docs.

@bobheadxi bobheadxi merged commit 1dc8d02 into master Mar 19, 2020
@bobheadxi bobheadxi deleted the client/set-webhook-on-init branch March 19, 2020 21:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: finalized needs review and final approval

Projects

None yet

Development

Successfully merging this pull request may close these issues.

daemon: set webhook secret on init

1 participant