Why not implement this interface:

go-tpm-tools/client/eventlog.go

Lines 17 to 19 in cf9fd6d

	type EventLogGetter interface {
	EventLog() ([]byte, error)
	}

?

It may add complexity and might make it harder to debug in the future. But sure, I implemented the interface version.

/gcbrun

/gcbrun

/gcbrun

Sorry I should ask this earlier: why do we need to have a flag for the eventlog? Do we expect a user to use an eventlog other than the default one?

And seems like all the existing eventlog fetching logic is built around this idea of using the default eventlog. And it's hard to add a flag and proper test without changing a lot of existing eventlog logic.

/gcbrun

/gcbrun

/gcbrun

/gcbrun