Woodpecker security issues #98

Closed
opened 2022-01-20 02:45:35 +00:00 by a · 4 comments
Owner

Currently, the woodpecker-agent user has write access to my deployed websites, but this is significant security issue. We could use environment variable secrets, but my PR only has very janky support for secrets.

Currently, the `woodpecker-agent` user has write access to my deployed websites, but this is significant security issue. We could use environment variable secrets, but my [PR only has very janky support for secrets](https://github.com/woodpecker-ci/woodpecker/pull/709#issuecomment-1017063016).
a added the
security
bug
labels 2022-01-20 02:45:35 +00:00
a self-assigned this 2022-01-20 02:45:35 +00:00
a added this to the (deleted) project 2022-01-20 02:45:35 +00:00
Author
Owner

Should be possible to do this with SSH keys but I'll also try to think of a way to improve the security of our current method, since it's much simpler than using environement variables and SSH keys.

Should be possible to do this with SSH keys but I'll also try to think of a way to improve the security of our current method, since it's much simpler than using environement variables and SSH keys.
Author
Owner

Alright so I was able to get SSH keys to work: https://ci.exozy.me/Ta180m/website/build/17

However, I think it just opens up more security issues, so I'm going to keep things as is for now.

Alright so I was able to get SSH keys to work: https://ci.exozy.me/Ta180m/website/build/17 However, I think it just opens up more security issues, so I'm going to keep things as is for now.
Author
Owner

We might also want to do some systemd security hardening for Woodpecker: https://www.freedesktop.org/software/systemd/man/systemd.exec.html

We might also want to do some systemd security hardening for Woodpecker: https://www.freedesktop.org/software/systemd/man/systemd.exec.html
Author
Owner

I disabled registration for Woodpecker since our setup with the bare-metal backend is inherently insecure.

I disabled registration for Woodpecker since our setup with the bare-metal backend is inherently insecure.
a added the
wontfix
label 2022-01-22 17:12:43 +00:00
a closed this issue 2022-01-22 17:13:08 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: exozyme/exozyme#98
No description provided.