Skip to content

Feature/splunk integration datapipe - #53

Open
michaelsss1 wants to merge 2 commits into
mainfrom
Feature/Splunk_integration_datapipe
Open

michaelsss1 wants to merge 2 commits into
mainfrom
Feature/Splunk_integration_datapipe

Conversation

@michaelsss1

Copy link
Copy Markdown
Contributor

No description provided.

… Wecom

fix: Type error: Argument of type 'Partial<Workflow>' is not assignable to parameter of type 'WorkflowWritePayload'.
…esting

- Updated Integration model to support additional integration types: 'splunk_search' and 'splunk_rule_publisher'.
- Modified IntegrationSerializer to validate new integration types.
- Added new endpoint for testing connection to integrations, including Splunk.
- Implemented helper functions for Splunk authentication and default paths.
- Enhanced IntegrationViewSet to handle fetching data from Splunk integrations.
- Updated frontend API service to include new connection testing and data fetching functions.
- Expanded integration forms in the frontend to accommodate Splunk-specific fields and validation.
Comment thread backend/alerts/tasks.py
payload = json.loads(text)
if isinstance(payload, dict):
return payload
except Exception:
Comment thread backend/alerts/tasks.py
payload = json.loads(text[start : end + 1])
if isinstance(payload, dict):
return payload
except Exception:

@alexchen16 alexchen16 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

This branch has not been deployed

No deployments
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