Skip to content
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.

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

Add appinspect rest support #64

Merged
merged 12 commits into from
Oct 23, 2023
Merged

Add appinspect rest support #64

merged 12 commits into from
Oct 23, 2023

Conversation

pyth0n1c
Copy link
Contributor

Allow contentctl build and contentctl test to directly run AppInspect via the REST API.
This eliminates the need for users to write their own shell script (or otherwise) to accomplish this vetting before deploying to their own Splunk Cloud Environment.

support.  Initial stub support
for appinspect api
api usgae in python.  Just need
to check for final results and
do final error handling
to disk, but we sstill need to fully
parse them and fail on any warnings.
There are also some contentctl
issues we need to account for so
that it does not throw failures.
and app.conf files to resolve
appinspect issues.
sure directory has correct name
and that build is always
updated with current utc timestamp.
Check for non-ascii characters
and extra spacing in string fields.
for ascii or newline encoding
issue
usage in contentctl test. Re add
notable/message parsing and
validation
@pyth0n1c pyth0n1c added WIP and removed WIP labels Oct 19, 2023
@pyth0n1c pyth0n1c requested a review from patel-bhavin October 20, 2023 18:33
@patel-bhavin
Copy link
Contributor

Tested locally and the report look fine :)

image

@patel-bhavin patel-bhavin merged commit e63ed1f into main Oct 23, 2023
6 checks passed
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