Change WebHookLog `url` usage
Extracted from #382357 (closed) to simplify the issue flow.
Problem
Described in the related issue.
web_hook_logs.url
column usage
3. Change the Proposal
-
Column refactor:
- Keep the
url
column - Refactor the files to stop using the
url
column, except theapp/views/shared/hook_logs/_content.html.haml
partial
- Keep the
-
Column removal:
- Remove the
url
column fromweb_hook_logs
table using DB migration - Update the
WebHookService
- Remove the relevant usage of
url
attribute - Remove
before_save :obfuscate_basic_auth
- Update the
app/views/shared/hook_logs/_content.html.haml
partial - Update the specs
- Remove the
Edited by Bojan Marjanovic