fix: webhook tracing and missing defers

[alnr]

Found by @jonas-jonas here (thanks!)

The webhook tracing was broken in the last refactoring. This change restores the correct passing of span contexts, and also re-enables retries. Previously, all attempts were aborted after a total of 30s. Now, we get 30s for each attempt.

[codecov]

Codecov Report

Merging #3145 (514d177) into master (ceb5cc2) will decrease coverage by 0.03%.
The diff coverage is 93.33%.

❗ Current head 514d177 differs from pull request most recent head 46677a3. Consider uploading reports for the commit 46677a3 to get more accurate results

@@            Coverage Diff             @@
##           master    #3145      +/-   ##
==========================================
- Coverage   77.58%   77.56%   -0.03%     
==========================================
  Files         316      316              
  Lines       19908    19906       -2     
==========================================
- Hits        15446    15440       -6     
- Misses       3273     3276       +3     
- Partials     1189     1190       +1     

Impacted Files	Coverage Δ
persistence/sql/identity/persister_identity.go	76.64% <92.85%> (ø)
selfservice/hook/web_hook.go	78.70% <100.00%> (-0.17%)	⬇️
session/test/persistence.go	99.05% <0.00%> (-0.95%)	⬇️

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

[jonas-jonas]

Nice! Thanks for fixing this so quickly! :)