Skip to content

[v16] web player: make query parameters optional#57626

Merged
zmb3 merged 1 commit intobranch/v16from
bot/backport-56100-branch/v16
Aug 8, 2025
Merged

[v16] web player: make query parameters optional#57626
zmb3 merged 1 commit intobranch/v16from
bot/backport-56100-branch/v16

Conversation

@zmb3
Copy link
Copy Markdown
Collaborator

@zmb3 zmb3 commented Aug 7, 2025

Backport #56100 to branch/v16

In #50262, we made it so that the "durationMs" query paramater in
the session player URL is optional (at the expense of an extra API
call to determine the recording length prior to playback).

We did not, however, do the same for the "recordingType" parameter.
This commit adds that information to the existing API call.

As a result, users  who want to build playback URLs directly instead
of clicking the play button in the UI have a stable URL format that
will play the session.

Closes #55780
Closes gravitational/customer-sensitive-requests#472
@zmb3 zmb3 added the no-changelog Indicates that a PR does not require a changelog entry label Aug 7, 2025
@github-actions github-actions Bot requested review from ravicious and rudream August 7, 2025 01:53
@zmb3 zmb3 added this pull request to the merge queue Aug 7, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2025
@zmb3 zmb3 added this pull request to the merge queue Aug 7, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 7, 2025
@zmb3 zmb3 added this pull request to the merge queue Aug 8, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 8, 2025
@zmb3 zmb3 added this pull request to the merge queue Aug 8, 2025
Merged via the queue into branch/v16 with commit ec64b95 Aug 8, 2025
45 checks passed
@zmb3 zmb3 deleted the bot/backport-56100-branch/v16 branch August 8, 2025 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport no-changelog Indicates that a PR does not require a changelog entry size/sm ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants