Skip to content

[DRAFT] Add ProjectDiscovery Cloud integration with changelog datastream #15760

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[DRAFT] Add ProjectDiscovery Cloud integration with changelog datastream #15760

wants to merge 1 commit into from

Conversation

@yahyaghani
Copy link
Contributor

Description

This is a draft PR implementing the initial ProjectDiscovery Cloud integration with changelog datastream.

Current Status:

  • ✅ HTTPJSON input with offset-based pagination
  • ✅ ECS-compliant field mapping
  • ✅ Mock server for testing
  • ✅ Pipeline and system tests passing

Relates to #15061

TODO:

  • Review datastream naming based on latest proposal
  • Add additional datastreams as needed
  • Final documentation review

@yahyaghani
[DRAFT] Add ProjectDiscovery Cloud integration with one datastream
e5f313d 
This is a draft PR implementing the initial datastream for the
ProjectDiscovery Cloud integration.

Implemented:
- HTTPJSON input with offset-based pagination
- ECS-compliant field mapping
- Mock server for testing
- Pipeline and system tests

Work in progress for issue elastic#15061
@yahyaghani yahyaghani requested a review from a team as a code owner October 27, 2025 00:19
@yahyaghani yahyaghani marked this pull request as draft October 27, 2025 00:20
@elasticmachine
Copy link

elasticmachine commented Oct 27, 2025 •
edited
Loading

💔 Build Failed

Failed CI Steps

History

efd6
efd6 reviewed Oct 27, 2025
View reviewed changes
packages/projectdiscovery_cloud/data_stream/vulnerability/agent/stream/stream.yml.hbs
Copy link
Contributor

Choose a reason for hiding this comment

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

Strongly recommend to not write this to use the HTTP JSON input.

Copy link
Member

Choose a reason for hiding this comment

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

Do you mean to use CEL instead?

@andrewkroh andrewkroh added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. New Integration Issue or pull request for creating a new integration package. labels Oct 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jsoriano jsoriano jsoriano left review comments

@efd6 efd6 efd6 left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. New Integration Issue or pull request for creating a new integration package.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@yahyaghani @elasticmachine @jsoriano @efd6 @andrewkroh