Skip to content

feat(policy-devel): support providing allowed hostnames#2339

Merged
migmartri merged 2 commits into
chainloop-dev:mainfrom
migmartri:2335-fix-comment
Aug 11, 2025
Merged

feat(policy-devel): support providing allowed hostnames#2339
migmartri merged 2 commits into
chainloop-dev:mainfrom
migmartri:2335-fix-comment

Conversation

@migmartri

Copy link
Copy Markdown
Member

running a policy that calls to an invalid endpoint

$ chainloop policy develop eval --kind EVIDENCE --material ~/work/chainloop/playground/policies/test-http/empty.json -p ~/work/chainloop/playground/policies/test-http/test-http.yaml
ERR evaluating policy: failed to execute policy : failed to evaluate policy: test-http:36: eval_builtin_error: http.send: unallowed host: app.chainloop.dev

providing the hostname it works

chainloop policy develop eval --kind EVIDENCE --material ~/work/chainloop/playground/policies/test-http/empty.json -p ~/work/chainloop/playground/policies/test-http/test-http.yaml --allowed-hostnames app.chainloop.dev
[
   {
      "violations": [
         "Platform version violation: expected '1.2.3', got 'v0.256.0'"
      ],
      "skip_reasons": [],
      "skipped": false
   }
]

Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri requested review from Piskoo and jiparis August 11, 2025 09:58
Signed-off-by: Miguel Martinez <miguel@chainloop.dev>
@migmartri migmartri merged commit 277331d into chainloop-dev:main Aug 11, 2025
13 checks passed
@migmartri migmartri deleted the 2335-fix-comment branch August 11, 2025 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants