Goal: Add check for signer passed in the case of logic sig rekeyed account

[barnjamin]

Summary

When trying to sign a transaction file with goal, if the sender is a normal address but the signer is a logic program, the call fails with an error message like:

send.txn: txn[0] error LogicNot signed and not a Logic-only account

The error message comes from

go-algorand/data/transactions/verify/txn.go

Line 314 in 33b87c4

// if the txn.Authorizer() == hash(Logic) then this is a (potentially) valid operation on a contract-only account

It notes that a valid case is when Auth is set to the hash of the program but we aren't setting the auth on the transaction even if the -S flag is passed

Test Plan

Made a txn where sender is a regular account and signed it with

goal clerk sign -i tmp.txn -o tmp.txn -S BJATCHES5YJZJ7JITYMVLSSIQAVAWBQRVGPQUDT5AZ2QSLDSXWWM46THOY -p tmp.teal

I borked the git history on the last one #3459 so just nuked it and starting over here.

Left it at @tsachiherman wanting better tests for this #3459 (comment)

[tsachiherman]

For me, this PR was abit confusing since it was completely omitting the fact that we're taking about a logicsig account that was rekeyed.

[tsachiherman]

Could you please take a look on the test failures ?

+ goal -w E1BC5E2446A40EDEA92E12A9F331A0E0 clerk sign -i /opt/cibuild/project/tmp/out/e2e/139021-1647380295636/E1BC5E2446A40EDEA92E12A9F331A0E0/gx-0.tx -p /opt/cibuild/project/tmp/out/e2e/139021-1647380295636/E1BC5E2446A40EDEA92E12A9F331A0E0/split.teal -o /opt/cibuild/project/tmp/out/e2e/139021-1647380295636/E1BC5E2446A40EDEA92E12A9F331A0E0/gx-0.stx
Signer invalid (): decoded bad addr: