Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Misleading setting name when restricting secrets to images #3748

Closed
3 tasks done
picsel2 opened this issue Jun 2, 2024 · 1 comment
Closed
3 tasks done

Misleading setting name when restricting secrets to images #3748

picsel2 opened this issue Jun 2, 2024 · 1 comment
Labels
enhancement improve existing features good first issue Likely to be an easy fix ui frontend related

Comments

@picsel2
Copy link

picsel2 commented Jun 2, 2024

Component

web-ui

Describe the bug

I wanted to use a secret within a step with commands. When setting the secret I thought that the image filter also applies to such steps. Now I found out the hard way, that it is apparently supposed only for plugins.

I had some trouble debugging that because the error message of the pipeline said

secret "registry_grabel_username" only allowed to be used by plugins by step "Package"

I suggest to use the expression "Plugin filter" instead of "Image filter" in the docs and UI.

Steps to reproduce

  1. Set a secret with an image filter set.
  2. Include a commands step with a matching image in the workflow.
  3. Run the pipeline.
  4. Notice the error message.

Expected behavior

Less confusing naming 😅

System Info

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

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]
@picsel2 picsel2 added the bug Something isn't working label Jun 2, 2024
@qwerty287 qwerty287 added ui frontend related enhancement improve existing features good first issue Likely to be an easy fix and removed bug Something isn't working labels Jun 3, 2024
@zc-devs
Copy link
Contributor

zc-devs commented Oct 5, 2024

Fixed by #4028

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement improve existing features good first issue Likely to be an easy fix ui frontend related
Projects
None yet
Development

No branches or pull requests

3 participants