Skip to content

Support s3-compatible sinks#659

Closed
eatonphil wants to merge 1 commit into
crashappsec:mainfrom
eatonphil:dev/pe/support-generic-s3-sinks
Closed

Support s3-compatible sinks#659
eatonphil wants to merge 1 commit into
crashappsec:mainfrom
eatonphil:dev/pe/support-generic-s3-sinks

Conversation

@eatonphil

@eatonphil eatonphil commented Apr 28, 2026

Copy link
Copy Markdown
Contributor

Companion PR in nimutils crashappsec/nimutils#98

sink_config chalk_s3_sink {
  enabled:  true
  sink:     "s3"
  endpoint: "http://localhost:8080"
  uri:      "s3://myorg/chalks3sink/report.json"
  uid:      "mykey"
  secret:   "mysecret"
  region:   "us-east-1"
}
subscribe("report", "my_rclone")

and to test

$ mkdir -p data/myorg
$ rclone serve s3 --auth-key "mykey,mysecret" data
$ ./chalk load s3sink.c4m
$ ./chalk insert ./chalk
$ ls data/myorg/chalks3sink
1777919291137-D00YFFGN2XG857EA5YG9Z1KJ5G-report.json

@eatonphil eatonphil requested a review from viega as a code owner April 28, 2026 20:26

@miki725 miki725 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

this pr needs to bump con4m with:

diff --git chalk.nimble chalk.nimble
index ba87089..20b69db 100644
--- chalk.nimble
+++ chalk.nimble
@@ -15,7 +15,7 @@ bin           = @["chalk"]

 # Dependencies
 requires "nim >= 2.0.8"
-requires "https://github.com/crashappsec/con4m#33743aada3a2db054804d92992d7df2536840012"
+requires "https://github.com/crashappsec/con4m#e8020aa2768041ee760739df96494d26d84ed8d2"
 requires "https://github.com/viega/zippy == 0.10.7" # MIT
 requires "https://github.com/NimParsers/parsetoml == 0.7.1" # MIT

Comment thread src/sinks.nim
@eatonphil eatonphil force-pushed the dev/pe/support-generic-s3-sinks branch 2 times, most recently from b22c0a7 to bdc8389 Compare May 5, 2026 13:11
@eatonphil eatonphil force-pushed the dev/pe/support-generic-s3-sinks branch from bdc8389 to c48707d Compare May 5, 2026 13:14
@eatonphil

Copy link
Copy Markdown
Contributor Author

Closing in favor of #663.

@eatonphil eatonphil closed this May 5, 2026
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