Webhook doesn't fire on creation or deletion of a repository that does not belong to an organisation

[JustAnotherArchivist]

Steps to reproduce

1. Set up a default webhook; in my case, I used a Gitea webhook with JSON POST, but I would be surprised if the details mattered.
2. Create a new repository under a user, i.e. outside of an organisation.

Expected behaviour

A repository webhook event is sent to signal the creation of the new repository.

Actual behaviour

No webhook call.

Repository creation and deletion events are only executed when the repo belongs to an organisation:

gitea/modules/notification/webhook/webhook.go

Line 102 in 2c5b3c7

if u.IsOrganization() {

gitea/modules/notification/webhook/webhook.go

Line 117 in 2c5b3c7

if u.IsOrganization() {

While that probably made sense in the past, it doesn't mix well with default webhooks nor with the new system-wide webhooks (#10546). Those should fire on all repositories.

(I have not tested system-wide webhooks, but given the code mentioned above, it should behave the same.)

System information

• Gitea version (or commit ref): 1.11.6
• Git version: not relevant
• Operating system: unknown, but I can ask if it matters
• Database (use [x]):
  • PostgreSQL
  • MySQL
  • MSSQL
  • SQLite
• Can you reproduce the bug at https://try.gitea.io:
  • Yes (provide example URL)
  • No
  • Not relevant
  • Can't test currently
• Log gist: not relevant

[JustAnotherArchivist]

#9180 is a partial duplicate of this but only mentions deletion, not creation.

[stale]

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

[JustAnotherArchivist]

Yes, this is still valid.

[jamesorlakin]

Default webhooks themselves don't do anything, they just are webhooks that get automatically added to fresh repos as far as I'm aware. System webhooks logically should, seeing as they're wider than just a repo in scope. I'll have a poke around to solve it for them.

However, this begs the question, should individual repos also fire a create (only if a default is defined) and delete event for their webhooks? I can't really come up with an argument for no in such a case, but I might be overlooking something.