[gatsby-source-contentful] retry fails due to undefined reporter

[njbmartin]

In the following function, if reporter is undefined, any retries will result in an error being thrown and the file failing to be downloaded, which isn't reported back to the console, but results in this error behing thrown instead: [gatsby-source-contentful] Please install gatsby-plugin-sharp

gatsby/packages/gatsby-source-contentful/src/download-with-retry.js

Lines 16 to 33 in 54d4721

	function getAxios(reporter) {
	if (!RetryAxios) {
	RetryAxios = axios.create()
	RetryAxios.defaults.raxConfig = {
	instance: RetryAxios,
	onRetryAttempt: err => {
	const cfg = rax.getConfig(err)
	reporter.verbose(
	`Retry attempt #${cfg.currentRetryAttempt} for ${err.config.url}`
	)
	},
	}
	rax.attach(RetryAxios)
	}
	return RetryAxios
	}

The downloadWithRetry function expects reporter to be passed in...

gatsby/packages/gatsby-source-contentful/src/download-with-retry.js

Line 35 in 54d4721

export default async function downloadWithRetry(requestConfig, reporter) {

But, in the two places it's actually used, reporter isn't passed in:

gatsby/packages/gatsby-source-contentful/src/cache-image.js

Lines 62 to 65 in e1a74b5

	downloadWithRetry({
	url: previewUrl,
	responseType: `stream`,
	})

gatsby/packages/gatsby-source-contentful/src/extend-node-type.js

Lines 114 to 117 in e1a74b5

	const imageResponse = await downloadWithRetry({
	url: requestUrl,
	responseType: `arraybuffer`,
	})

[github-actions]

Hiya!

This issue has gone quiet. Spooky quiet. 👻

We get a lot of issues, so we currently close issues after 60 days of inactivity. It’s been at least 20 days since the last update here.
If we missed this issue or if you want to keep it open, please reply here.
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks for being a part of the Gatsby community! 💪💜

[github-actions]

Hey again!

It’s been 60 days since anything happened on this issue, so our friendly neighborhood robot (that’s me!) is going to close it.
Please keep in mind that I’m only a robot, so if I’ve closed this issue in error, I’m HUMAN_EMOTION_SORRY. Please feel free to comment on this issue or create a new one if you need anything else.
As a friendly reminder: the best way to see this issue, or any other, fixed is to open a Pull Request. Check out gatsby.dev/contribute for more information about opening PRs, triaging issues, and contributing!

Thanks again for being part of the Gatsby community! 💪💜