Skip to content

[injectors] feat(scv): add domains management (#4266) #127

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.

Sign up for GitHub

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

Jump to bottom
Merged
gabriel-peze merged 4 commits into from
Dec 16, 2025
Merged

[injectors] feat(scv): add domains management (#4266) #127

gabriel-peze merged 4 commits into from
Dec 16, 2025

Conversation

@gabriel-peze
Copy link

@gabriel-peze gabriel-peze commented Dec 3, 2025

Proposed changes

  • Add domains management for all injectors.

Testing Instructions

  1. Run the injectors, domains should be present into the platform

@gabriel-peze gabriel-peze linked an issue Dec 4, 2025 that may be closed by this pull request
[SCV] Assign Security Domains to Injectors Contract OpenAEV-Platform/openaev#4477
Closed
@gabriel-peze gabriel-peze marked this pull request as ready for review December 4, 2025 14:06
@gabriel-peze gabriel-peze self-assigned this Dec 4, 2025
@gabriel-peze gabriel-peze added the filigran team use to identify PR from the Filigran team label Dec 4, 2025
@gabriel-peze
Copy link
Author

gabriel-peze commented Dec 4, 2025

Need this to be validated to have the tests ok, and merge after

@gabriel-peze gabriel-peze linked an issue Dec 5, 2025 that may be closed by this pull request
[SCV] Categorize existing payloads into domains OpenAEV-Platform/openaev#4266
Closed
antoinemzs
antoinemzs approved these changes Dec 16, 2025
View reviewed changes
Copy link
Member

@antoinemzs antoinemzs left a comment

Choose a reason for hiding this comment

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

this LGTM

@gabriel-peze gabriel-peze changed the title [all] feat(SCV): add domains management (#4266) [injectors] feat(SCV): add domains management (#4266) Dec 16, 2025
@gabriel-peze gabriel-peze changed the title [injectors] feat(SCV): add domains management (#4266) [injectors] feat(scv): add domains management (#4266) Dec 16, 2025
@gabriel-peze gabriel-peze merged commit a30d965 into release/current Dec 16, 2025
4 of 6 checks passed
@gabriel-peze gabriel-peze deleted the issue/4266 branch December 16, 2025 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@antoinemzs antoinemzs antoinemzs approved these changes

Assignees

@gabriel-peze gabriel-peze

Labels

filigran team use to identify PR from the Filigran team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[SCV] Assign Security Domains to Injectors Contract [SCV] Categorize existing payloads into domains

3 participants

@gabriel-peze @antoinemzs