Skip to content

Conversation

@parfeon
Copy link
Contributor

@parfeon parfeon commented Jan 15, 2025

feat(file-share): add file upload and download timeout configuration

Add fileRequestTimeout client configuration option which is specific only for requests which upload and download files.

fix: fix instanceId query parameter

Fix issue with instanceId set to undefined for requests with useInstanceId configuration flag set to true.

Add `fileRequestTimeout` client configuration option which is specific only for requests which
upload and download files.

fix: fix `instanceId` query parameter

Fix issue with `instanceId` set to `undefined` for requests with `useInstanceId` configuration
flag set to `true`.
@parfeon parfeon added status: done This issue is considered resolved. priority: high This PR should be reviewed ASAP. type: feature This PR contains new feature. labels Jan 15, 2025
@parfeon parfeon self-assigned this Jan 15, 2025
request.operation() === RequestOperation.PNDownloadFileOperation
) {
// Set file upload / download request delay.
transportRequest.timeout = this._configuration.getFileTimeout();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@parfeon
Copy link
Contributor Author

parfeon commented Jan 15, 2025

@pubnub-release-bot release

@parfeon parfeon merged commit 875e44b into master Jan 15, 2025
6 checks passed
@parfeon parfeon deleted the CLEN-2459 branch January 15, 2025 23:17
@pubnub-release-bot
Copy link
Contributor

🚀 Release successfully completed 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority: high This PR should be reviewed ASAP. status: done This issue is considered resolved. type: feature This PR contains new feature.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants