Throw on upload limit reached

[pierreminiggio]

I don't think reaching the upload limit should be a simple log, we should probably throw a documented exception after closing the browser

youtube-uploader/src/upload.ts

Lines 171 to 174 in db8c5b1

	await page.waitForXPath('//*[contains(text(),"Daily upload limit reached")]', { timeout: 500 }).then(() => {
	messageTransport.log("Daily upload limit reached.");
	browser.close();
	}).catch(() => {});

[github-actions]

Paid support is available for quick resolution, new features etc. For paid support, please write mail to youtube-uploader@datascraping.pp.ua

[PeterStegarus]

I've also noticed the 0.5 seconds timeout is too short on some occasions. 2 seconds did the trick for me but we should probably handle this some other way. Unsure if waiting indefinitely for 'Upload complete' is what we want

Maybe we could Promise.race the selectors for both the 'Daily upload limit' and 'Upload complete' elements. Will have to look further into this

youtube-uploader/src/upload.ts

Line 176 in c34d632

await page.waitForXPath('//*[contains(text(),"Upload complete")]', { timeout: 0 })

[fawazahmed0]

I guess this should be closed with this PR