Refactor SDK initialization commands to fetch templates from GitHub r…#222
Merged
luke-hagar-sp merged 4 commits intomainfrom Feb 17, 2026
Merged
Refactor SDK initialization commands to fetch templates from GitHub r…#222luke-hagar-sp merged 4 commits intomainfrom
luke-hagar-sp merged 4 commits intomainfrom
Conversation
…epositories. Removed embedded file contents and added a new function to handle downloading and initializing projects. Updated command descriptions and example outputs for clarity. Added documentation for template repositories and their usage.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
… test_create.json to change project name and added new test cases for project initialization and extraction in fetch_test.go. Removed outdated SDK_TEMPLATE_REPOS.md file.
…ding test case Enhanced the ExtractAndInitProject function to reject path traversal and absolute paths in archive entries, preventing potential Zip Slip vulnerabilities. Added a new test case in fetch_test.go to verify that the function correctly handles malicious tarball entries, ensuring no files are created outside the intended project directory.
Contributor
|
If we don't do this already, should we notify the user that we are going to download a template from github/sailpoint-oss .... prior to doing the fetch call? |
Added logging statements to the FetchAndInitProject and ExtractAndInitProject functions to provide better visibility into the fetching, downloading, and extraction processes. This includes logging the repository owner, repository name, and project name at various stages, improving traceability during project setup.
philip-ellis-sp
approved these changes
Feb 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…epositories. Removed embedded file contents and added a new function to handle downloading and initializing projects. Updated command descriptions and example outputs for clarity. Added documentation for template repositories and their usage.
Description
What is the intent of this change and why is it being made?
How Has This Been Tested?
What testing have you done to verify this change?