`GetOrFetch`: Let `ErrOnlyCachedRecords` wrap underlying `fetchFn` error

[ernstwi]

Is your feature request related to a problem? Please describe.

When fetchFn returns an error during a GetOrFetch, the error is thrown away on this line and ErrOnlyCachedRecords is returned in its place.

As it is now the only way I can find to log the error returned from a fetchFn is to do the logging inside the fetchFn itself. In my case I have a failing request I'd like to know the error code of.

Describe the solution you'd like

Could ErrOnlyCachedRecords wrap the error returned from fetchFn?

[viccon]

Hey @ernstwi I hope you're doing good! I did an attempt to fix the issue here: #33

Could you check if it works for you? E.g go get github.com/viccon/sturdyc@join-errors. If everything works as expected I'll do a new release

[ernstwi]

Awesome, thank you. Hope everything is well with you as well at the new job :)

We'll try this and let you know, looks like it should do the job.

[ernstwi]

Looks good!

Before:

sturdyc: failed to fetch the records that were not in the cache

After:

sturdyc: failed to fetch the records that were not in the cache
ErrTransport: Get "https://data-api.das.a2d.tv/content/metadata/v1/similars?asset_id=c9f2a6d79ea1e5d0476a&company=tv4": context canceled

[viccon]

@ernstwi Great, I merged the PR and did a new release