Add logging by callback

.github/workflows/go.yml

@@ -52,13 +52,13 @@ jobs:
       - name: Build
         run: |
           export LD_LIBRARY_PATH="/opt/pdfium/lib"
-          go build -o ./bin/pdfium.exe ./main.go
-          go build -o ./bin/subprocess.exe ./subprocess/main.go
+          go build -o ./bin/pdfium ./main.go
+          go build -o ./bin/subprocess ./subprocess/main.go
 
       - name: Archive production artifacts
         uses: actions/upload-artifact@v2
         with:
           name: binary
           path: |
-            ./bin/pdfium.exe
-            ./bin/subprocess.exe
+            ./bin/pdfium
+            ./bin/subprocess

go.mod

@@ -3,16 +3,14 @@ module github.com/klippa-app/go-pdfium
 go 1.15
 
 require (
-	github.com/getsentry/sentry-go v0.5.1
 	github.com/golang/protobuf v1.4.2 // indirect
 	github.com/google/go-cmp v0.5.2 // indirect
 	github.com/hashicorp/go-hclog v0.14.1
 	github.com/hashicorp/go-plugin v1.4.0
 	github.com/jolestar/go-commons-pool/v2 v2.1.1
+	github.com/kr/pretty v0.1.0 // indirect
 	github.com/mattn/go-colorable v0.1.6 // indirect
 	github.com/mitchellh/go-testing-interface v1.0.0 // indirect
-	github.com/pkg/errors v0.9.1 // indirect
-	github.com/sirupsen/logrus v1.6.0
 	github.com/stretchr/testify v1.7.0
 	golang.org/x/net v0.0.0-20201006153459-a7d1128ccaa0 // indirect
 	google.golang.org/genproto v0.0.0-20200521103424-e9a78aa275b7 // indirect

main.go

@@ -1,11 +1,18 @@
 package main
 
-import "github.com/klippa-app/go-pdfium/pdfium"
+import (
+	"fmt"
+
+	"github.com/klippa-app/go-pdfium/pdfium"
+)
 
 func main() {
 	pdfium.InitLibrary(pdfium.Config{
 		MinIdle:  1,
 		MaxIdle:  1,
 		MaxTotal: 1,
+		LogCallback: func(s string) {
+			fmt.Println("[PDFIUM ERROR]: " + s)
+		},
 	})
 }

pdfium/pdfium.go

@@ -12,10 +12,8 @@ import (
 	pool "github.com/jolestar/go-commons-pool/v2"
 	"github.com/klippa-app/go-pdfium/pdfium/internal/commons"
 
-	"github.com/getsentry/sentry-go"
 	"github.com/hashicorp/go-hclog"
 	"github.com/hashicorp/go-plugin"
-	log "github.com/sirupsen/logrus"
 )
 
 type worker struct {
@@ -27,9 +25,10 @@ type worker struct {
 var workerPool *pool.ObjectPool
 
 type Config struct {
-	MinIdle  int
-	MaxIdle  int
-	MaxTotal int
+	MinIdle     int
+	MaxIdle     int
+	MaxTotal    int
+	LogCallback func(string)
 }
 
 func InitLibrary(config Config) { // serve one thread that is "native" through cgo
@@ -95,29 +94,25 @@ func InitLibrary(config Config) { // serve one thread that is "native" through c
 		}, nil, func(ctx goctx.Context, object *pool.PooledObject) bool {
 			worker := object.Object.(*worker)
 			if worker.pluginClient.Exited() {
-				log.Println("[pdfium error] Worker exited")
-				sentry.CaptureException(fmt.Errorf("[pdfium error] Worker exited"))
+				config.LogCallback("Worker exited")
 				return false
 			}
 
 			err := worker.rpcClient.Ping()
 			if err != nil {
-				log.Printf("[pdfium error] Error on RPC ping: %s", err.Error())
-				sentry.CaptureException(fmt.Errorf("[pdfium error] Error on RPC ping: %s", err.Error()))
+				config.LogCallback(fmt.Sprintf("Error on RPC ping: %s", err.Error()))
 				return false
 			}
 
 			pong, err := worker.plugin.Ping()
 			if err != nil {
-				log.Printf("[pdfium error] Error on plugin ping: %s", err.Error())
-				sentry.CaptureException(fmt.Errorf("[pdfium error] Error on plugin ping: %s", err.Error()))
+				config.LogCallback(fmt.Sprintf("Error on plugin ping:: %s", err.Error()))
 				return false
 			}
 
 			if pong != "Pong" {
 				err = errors.New("Wrong ping/pong result")
-				log.Printf("[pdfium error] Error on plugin ping: %s", err.Error())
-				sentry.CaptureException(fmt.Errorf("[pdfium error] Error on plugin ping: %s", err.Error()))
+				config.LogCallback(fmt.Sprintf("Error on plugin ping:: %s", err.Error()))
 				return false
 			}