woodpecker-ci/woodpecker

Poorly formatted global registry can not be deleted.

Opened this issue · 3 comments

Component

server

Describe the bug

When adding a global registry, if you add the protocol (ie https://), the entry is not removable via the UI.

Steps to reproduce

create two global registry records, mytest.local, and https://mytest.local.

mytest.local will be removable.
https://mytest.local will say it's being removed, but will persist.

Expected behavior

Both records should be removed.

System Info

{"source":"https://github.com/woodpecker-ci/woodpecker","version":"2.7.3"}

Additional context

No response

Validations

  • Read the docs.
  • Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
  • Checked that the bug isn't fixed in the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]

@greenaar I cannot reproduce this. I can just delete it if I add a registry with protocol.

WP next-05e355915b, Postgres 16

Add two registries
Screenshot 2024-11-28 1

Delete https://mytest.local
Screenshot 2024-11-28 2

Delete mytest.local
Screenshot 2024-11-28 3

server.log

woodpecker=# select * from registries;
 id | repo_id |       address        | username | password | org_id
----+---------+----------------------+----------+----------+--------
  2 |       0 | https://mytest.local | u        | u        |      0
(1 row)

Still reproducing on next-21755bef4e