-
Notifications
You must be signed in to change notification settings - Fork 73
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 Xray Source Control Service #451
Conversation
utils/scandetails.go
Outdated
// In some VCS providers, there are no projects, fallback to the repository owner. | ||
if gitProject == "" { | ||
gitProject = sc.RepoOwner | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// In some VCS providers, there are no projects, fallback to the repository owner. | |
if gitProject == "" { | |
gitProject = sc.RepoOwner | |
} |
utils/scandetails.go
Outdated
if err != nil { | ||
log.Warn(fmt.Sprintf("failed getting latest commit, repository: %s, branch: %s. error: %s ", sc.RepoName, scannedBranch, err.Error())) | ||
return | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return error
utils/scandetails.go
Outdated
if err != nil { | ||
log.Warn(fmt.Sprintf("failed getting repository information, for repository: %s, branch: %s. error: %s ", sc.RepoName, scannedBranch, err.Error())) | ||
return |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
return error
Co-authored-by: Michael Sverdlov <sverdlov93@gmail.com>
Generating
GitInfoContext
before scanning a project, and passes it to cli-core.In the case the user have XSC server enabled, scan results will be visible in the UI.
In case not, regular Xray flow will be executed.
Only
Scan-Repository
command sends theGitInfoContext
asScan-Pull-Request
will need extra fields.jfrog/jfrog-client-go#823
jfrog/jfrog-cli-core#940