Skip to content

Add golang based hook-sdk #3786

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

Merged
J12934 merged 10 commits into from
Sep 9, 2026
Merged

Add golang based hook-sdk #3786

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
500d337
Add hook-sdk for golang hooks
J12934 Aug 25, 2026
3fcc9da
Resolve review comments
J12934 Sep 7, 2026
e07ff46
Properly set defaults and ensure that node.js hook taskfile builds are
J12934 Sep 8, 2026
484e15f
Rework sdk
J12934 Sep 8, 2026
ead2fbf
Add some docs for the golang hook sdk
J12934 Sep 8, 2026
5a8d81f
Have the build context be set more cleanly for new golang based hooks
J12934 Sep 8, 2026
e509b8a
Ensure that severities are properly uppercased
J12934 Sep 8, 2026
3d52d87
Resolve minor inconsistency with url and arg handling
J12934 Sep 8, 2026
3dca955
Add a simple test to the sdk
J12934 Sep 8, 2026
b050de3
add license header stuff
J12934 Sep 9, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Resolve minor inconsistency with url and arg handling 
Signed-off-by: Jannik Hollenbach <jannik.hollenbach@iteratec.com>
  • Loading branch information
@J12934
J12934 committed Sep 9, 2026
commit 3d52d87e7c0f391fe9c943da9e0571b1baa60c71
11 changes: 7 additions & 4 deletions hook-sdk/golang/sdk.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,8 +116,11 @@ type Option func(*Client)

func WithK8sClient(client K8sClient) Option { return func(c *Client) { c.k8sClient = client } }
func WithFileClient(client FileClient) Option { return func(c *Client) { c.fileClient = client } }
func WithArgs(args []string) Option { return func(c *Client) { c.args = args } }
func WithLogger(logger *slog.Logger) Option { return func(c *Client) { c.logger = logger } }

// WithArgs sets the operation URLs (rawResults, findings, rawResultsUpload,
// findingsUpload) used by Run, in that order.
func WithArgs(urls []string) Option { return func(c *Client) { c.urls = urls } }
func WithLogger(logger *slog.Logger) Option { return func(c *Client) { c.logger = logger } }

func NewClient(opts ...Option) (*Client, error) {
scanName, namespace := os.Getenv("SCAN_NAME"), os.Getenv("NAMESPACE")
Expand All @@ -127,7 +130,7 @@ func NewClient(opts ...Option) (*Client, error) {
if namespace == "" {
return nil, fmt.Errorf("NAMESPACE environment variable is required")
}
client := &Client{scanName: scanName, namespace: namespace, args: os.Args}
client := &Client{scanName: scanName, namespace: namespace, urls: os.Args[1:]}
for _, option := range opts {
option(client)
}
Expand All @@ -148,7 +151,7 @@ func NewClient(opts ...Option) (*Client, error) {
}

func (c *Client) Run(ctx context.Context, handler HookHandler) error {
Comment thread
J12934 marked this conversation as resolved.
request := &hookRequest{k8sClient: c.k8sClient, fileClient: c.fileClient, scanName: c.scanName, namespace: c.namespace, urls: c.args[1:]}
request := &hookRequest{k8sClient: c.k8sClient, fileClient: c.fileClient, scanName: c.scanName, namespace: c.namespace, urls: c.urls}
if err := handler.Handle(ctx, request); err != nil {
return fmt.Errorf("run hook handler: %w", err)
}
Expand Down