dapr / go-sdk

Dapr SDK for go
Apache License 2.0
441 stars 170 forks source link

Server-side warnings in the log when using Actors #585

Open thewilli opened 2 weeks ago

thewilli commented 2 weeks ago

Describe the bug

When using Actors, the server-side logs will always contain warnings about methods with invalid signatures, which is fine if it wasn't true for the required ones being part of ServerImplBaseCtx.

The messages are generated here and should imo not warn about required methods, as it just bloats the server logs.

To Reproduce

Expected behavior

The output matches the one listed in the README

Actual behavior

Server-side logs contain the following warnings:

== APP == method GetStateManager is illegal, err = the latest return type actor.StateManagerContext of method "GetStateManager" is not error, just skip it
== APP == method ID is illegal, err = the latest return type string of method "ID" is not error, just skip it
== APP == method SetID is illegal, err = num out invalid, just skip it
== APP == method SetStateManager is illegal, err = num out invalid, just skip it
== APP == method Type is illegal, err = the latest return type string of method "Type" is not error, just skip it