Skip to content

feat(core): implement SecurityIntegration ENABLED filter.#14839

Merged
ammeek merged 6 commits intodevelopfrom
feat/security-integration-filters
Mar 6, 2026
Merged

feat(core): implement SecurityIntegration ENABLED filter.#14839
ammeek merged 6 commits intodevelopfrom
feat/security-integration-filters

Conversation

@ammeek
Copy link
Contributor

@ammeek ammeek commented Feb 26, 2026

✨ Description

What does this PR change?
Add SecurityIntegration ENABLED filter

🔗 Related Issue

relates to kestra-io/kestra-ee#6256

🛠️ Backend Checklist

  • Code compiles successfully and passes all checks
  • All unit and integration tests pass

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

🐋 Docker image

ghcr.io/kestra-io/kestra-pr:14839
docker run --pull=always --rm -it -p 8080:8080 --user=root -v /var/run/docker.sock:/var/run/docker.sock -v /tmp:/tmp ghcr.io/kestra-io/kestra-pr:14839 server local

🧪 Java Unit Tests

TestsPassed ✅Skipped ⚠️FailedTime ⏱
Java Tests Report4055 ran4042 ✅13 ⚠️0 ❌43m 12s 413ms

@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Tests report quick summary:

success ✅ > tests: 4055, success: 4042, skipped: 13, failed: 0

unfold for details
Project Status Success Skipped Failed
cli success ✅ 80 0 0
core success ✅ 2003 0 0
executor success ✅ 4 0 0
jdbc success ✅ 11 0 0
jdbc-h2 success ✅ 473 0 0
jdbc-mysql success ✅ 476 0 0
jdbc-postgres success ✅ 476 0 0
processor success ✅ 7 0 0
runner-memory success ✅ 1 0 0
scheduler success ✅ 23 0 0
script success ✅ 12 0 0
storage-local success ✅ 64 0 0
webserver success ✅ 421 0 0
worker success ✅ 4 0 0

Develocity build scan: https://develocity.kestra.io/s/ylrmq4ent7ktg

Copy link
Contributor

@AcevedoR AcevedoR left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldnt SECURITY_INTEGRATION be present in the openapi spec ?

@ammeek
Copy link
Contributor Author

ammeek commented Mar 5, 2026

shouldnt SECURITY_INTEGRATION be present in the openapi spec ?

Yea it should be, will regenerate the openapi spec.

steven meek added 3 commits March 5, 2026 13:47
# Conflicts:
#	jdbc/src/main/java/io/kestra/jdbc/repository/AbstractJdbcRepository.java
# Conflicts:
#	core/src/main/java/io/kestra/core/models/QueryFilter.java
@ammeek
Copy link
Contributor Author

ammeek commented Mar 5, 2026

@AcevedoR As mentioned here SECURITY_INTEGRATION should not be in the open api spec file.

@AcevedoR AcevedoR self-requested a review March 5, 2026 16:53
Copy link
Contributor

@nkwiatkowski nkwiatkowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ammeek ammeek merged commit 782a6d6 into develop Mar 6, 2026
14 checks passed
@ammeek ammeek deleted the feat/security-integration-filters branch March 6, 2026 10:42
@github-project-automation github-project-automation bot moved this from To review to Done in Pull Requests Mar 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants