fix: improve webhook resilience

[hperl]

Changes:

• Decouple post-registration post-persistence context from request context
• Upgrade ory/x to use retrying in Jsonnet VM
• Add better logging for when webhooks fail

Related issue(s)

Checklist

• I have read the contributing guidelines.
• I have referenced an issue containing the design document if my change
  introduces a new feature.
• I am following the
  contributing code guidelines.
• I have read the security policy.
• I confirm that this pull request does not address a security
  vulnerability. If this pull request addresses a security vulnerability, I
  confirm that I got the approval (please contact
  [email protected]) from the maintainers to push
  the changes.
• I have added tests that prove my fix is effective or that my feature
  works.
• I have added or changed the documentation.

Further Comments

[codecov]

Codecov Report

Merging #3200 (1ba8e9f) into master (1180c05) will decrease coverage by 0.06%.
The diff coverage is 42.85%.

❗ Current head 1ba8e9f differs from pull request most recent head c1ac6f3. Consider uploading reports for the commit c1ac6f3 to get more accurate results

@@            Coverage Diff             @@
##           master    #3200      +/-   ##
==========================================
- Coverage   77.95%   77.89%   -0.06%     
==========================================
  Files         320      320              
  Lines       20428    20451      +23     
==========================================
+ Hits        15924    15931       +7     
- Misses       3301     3317      +16     
  Partials     1203     1203              

Impacted Files	Coverage Δ
selfservice/flow/registration/hook.go	74.28% <20.00%> (-8.26%)	⬇️
driver/registry_default.go	87.02% <100.00%> (+0.06%)	⬆️
selfservice/hook/web_hook.go	80.00% <100.00%> (+0.52%)	⬆️

... and 1 file with indirect coverage changes

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

[zepatrik]

Please correct me, but I don't see new retry logic added, only context timeout added.

[zepatrik]

I realized that we already had an update of ory/x in a different PR, so the retry logic there is already on master 👍
However, I do not agree that we want to decouple the request context from the webhook, at least not for blocking hooks. Non-blocking hooks already have their own context, so I think we should keep that as is and just pick the log improvements from this PR.

[aeneasr]

Looks good :)