Skip to content

Commit b457971

Browse files
mcottontensorgithub-actions[bot]lukehbfabian-rehmleon
authored
Merging 5.7 into master. (#748)
* Fix: Firefox connectivity issues when sdpMLineIndex and sdpMid are stripped. (#709) (#711) (cherry picked from commit 7790838) Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> * Updated NPM changelogs (#712) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update signalling library readme, patch bump * Update readme for ui library, bump patch version * Update frontend docs, patch bump * Update common readme, patch bump * Update RELEASE_VERSION 0.1.1 * Update 5.7 with docs and housekeeping changes (#718) * Bump markdown link checker version for security patch #691 * Update link checker to permit 429 code from Github as valid * Update user-agent in link checker action to try prevent 403 forbidden on some sites * Update link checker to accept HTTP status 403 (forbidden) * Remove broken link about eslint * Modify README for mediasoup-sdp-bridge to remove broken link Updated README to reflect fork status and removed badges. * Update workflow triggers for markdown files in push and PR * Fix: Firefox connectivity issues when sdpMLineIndex and sdpMid are stripped. (#709) * Bring 5.7 changes to master branch (docs + sdpMid fix) (#716) * Fix: Firefox connectivity issues when sdpMLineIndex and sdpMid are stripped. (#709) (#711) (cherry picked from commit 7790838) Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> * Updated NPM changelogs (#712) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update signalling library readme, patch bump * Update readme for ui library, bump patch version * Update frontend docs, patch bump * Update common readme, patch bump --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Update copyright notice * Update copyright notice in CONTRIBUTING.md * Update copyright notice * Update copyright in frontend/readme.md * Delete .changeset/light-games-wonder.md --------- Co-authored-by: mcottontensor <80377552+mcottontensor@users.noreply.github.com> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Cut new GH release for 5.7 - 0.1.2 * [UE5.7] Exposed playerid (#728) (#729) * Exposed playerid (#728) (cherry picked from commit 6733632) * Expose player id and improve signalling library Add ability to access player id on the frontend and remove player id stripping from the signalling library. This change backports the 'Exposed playerid' feature. --------- Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> * Updated NPM changelogs (#730) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * [UE5.7] Make this.peerConnection.getStats() access safe (#719) (#731) * Update PeerConnectionController.ts (#719) Fix for throwing error when peerConnection is not defined (cherry picked from commit 9a5cd8e) * Create polite-knives-hang.md --------- Co-authored-by: fabian-rehm <fabian.rehm@porter.de> Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> * [UE5.7] SecurityError: Failed to execute 'isSessionSupported' on 'XRSystem' when in iframe (#734) (#737) * SecurityError: Failed to execute 'isSessionSupported' on 'XRSystem' when in iframe (#734) * fix: SecurityError: Failed to execute 'isSessionSupported' on 'XRSystem' when in iframe fixes: #722 * Update Frontend/library/src/WebXR/WebXRController.ts * Update Frontend/library/src/WebXR/WebXRController.ts --------- Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> (cherry picked from commit cc8c923) * Fix iframe streaming issue due to SecurityError --------- Co-authored-by: Leon Radley <leon@radley.se> Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> * Updated NPM changelogs (#732) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> * Updating npm publish for new npmjs trusted publisher stuff. (#744) * Trying to figure out OIDC * Trying to debug OIDC issues. * Duplicating id-token * Still debugging OIDC issues. Adding debugging steps. * Debugging OIDC. * Fix OIDC (#745) * Updating npm to allow oidc auth * bumping common patch version to test publish. * Fixing up package info. * Fix link descriptions for UE 5.7 (#743) (#747) (cherry picked from commit ff2b1bf) Co-authored-by: Flo Edelmann <git@flo-edelmann.de> --------- Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Luke Bermingham <1215582+lukehb@users.noreply.github.com> Co-authored-by: fabian-rehm <fabian.rehm@porter.de> Co-authored-by: Leon Radley <leon@radley.se> Co-authored-by: Flo Edelmann <git@flo-edelmann.de>
1 parent ff2b1bf commit b457971

File tree

11 files changed

+62
-12
lines changed

11 files changed

+62
-12
lines changed

‎.github/workflows/changesets-publish-npm-packages.yml‎

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,10 @@ on:
1111
# This makes the matrix of jobs to run one at a time.
1212
concurrency: ${{ github.workflow }}-${{ github.ref }}
1313

14+
permissions:
15+
id-token: write # Required for OIDC
16+
contents: read
17+
1418
jobs:
1519
# gets all publishable npm packages.
1620
fetch-package-info:
@@ -49,6 +53,7 @@ jobs:
4953
runs-on: ubuntu-latest
5054
needs: fetch-package-info
5155
permissions:
56+
id-token: write # Required for OIDC
5257
contents: write
5358
if: ${{ fromJSON(needs.fetch-package-info.outputs.matrix).count > 0 }}
5459
strategy:
@@ -72,10 +77,12 @@ jobs:
7277
node-version: "${{ steps.get_node_version.outputs.node_version }}"
7378
registry-url: 'https://registry.npmjs.org'
7479

80+
# Ensure npm 11.5.1 or later is installed for OIDC
81+
- name: Update npm
82+
run: npm install -g npm@latest
83+
7584
- name: Publish ${{ matrix.package.name }}
7685
working-directory: ${{ matrix.package.path }}
77-
env:
78-
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
7986
run: |
8087
npm install
8188
npm run build

‎.github/workflows/healthcheck-markdown-links.yml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ jobs:
2424
id: lychee
2525
uses: lycheeverse/lychee-action@v2.0.2
2626
with:
27-
args: --user-agent 'GHA/PixelStreamingInfrastructure' --accept '100..=103,200..=299,403,429' --exclude 'localhost' --exclude 'github.com/EpicGames/UnrealEngine' --exclude '.png' --exclude '.jpg' --exclude-path 'SFU/mediasoup-sdp-bridge/README.md' './**/*.md'
27+
args: --user-agent 'GHA/PixelStreamingInfrastructure' --accept '100..=103,200..=299,403,429' --github-token ${{ secrets.WORKFLOW_TOKEN }} --exclude 'localhost' --exclude 'github.com/EpicGames/UnrealEngine' --exclude '.png' --exclude '.jpg' --exclude-path 'SFU/mediasoup-sdp-bridge/README.md' './**/*.md'
2828

2929
- name: Create Issue From File
3030
if: env.lychee_exit_code != 0

‎Common/package.json‎

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@epicgames-ps/lib-pixelstreamingcommon-ue5.7",
3-
"version": "0.1.1",
3+
"version": "0.1.3",
44
"description": "Common utilities library for Unreal Engine 5.7 Pixel Streaming",
55
"main": "dist/cjs/pixelstreamingcommon.js",
66
"module": "dist/esm/pixelstreamingcommon.js",
@@ -44,6 +44,10 @@
4444
},
4545
"author": "Epic Games",
4646
"license": "MIT",
47+
"repository": {
48+
"type": "git",
49+
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure"
50+
},
4751
"publishConfig": {
4852
"access": "public"
4953
}

‎Frontend/implementations/typescript/package.json‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,5 +26,9 @@
2626
"dependencies": {
2727
"@epicgames-ps/lib-pixelstreamingfrontend-ue5.7": "*",
2828
"@epicgames-ps/lib-pixelstreamingfrontend-ui-ue5.7": "*"
29+
},
30+
"repository": {
31+
"type": "git",
32+
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure"
2933
}
3034
}

‎Frontend/library/CHANGELOG.md‎

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,23 @@
11
# @epicgames-ps/lib-pixelstreamingfrontend-ue5.6
22

3+
## 0.2.1
4+
5+
### Patch Changes
6+
7+
- c37e0f3: [UE5.7] QoL: Change `this.peerConnection.getStats()` access to the safer `this.peerConnection?.getStats()` (#719)
8+
- 81031a8: [UE5.7] Fix: Streaming in iframe broken due to SecurityError checking if XR is supported (#734)
9+
10+
## 0.2.0
11+
12+
### Minor Changes
13+
14+
- 05bebea: Add: Ability to access player id on the frontend.
15+
QoL: Remove player id stripping from the signalling library.
16+
17+
It is useful to be able to use the player id as a unique identifier that is common between UE side stats and frontend side stats; however, the player id is not actually exposed to TS/JS because the SS strips it out of signalling messages.
18+
19+
This change is a backport of "Exposed playerid" (#728)
20+
321
## 0.1.1
422

523
### Patch Changes

‎Frontend/library/package.json‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@epicgames-ps/lib-pixelstreamingfrontend-ue5.7",
3-
"version": "0.1.2",
3+
"version": "0.2.1",
44
"description": "Frontend library for Unreal Engine 5.7 Pixel Streaming",
55
"main": "dist/cjs/pixelstreamingfrontend.js",
66
"module": "dist/esm/pixelstreamingfrontend.js",
@@ -38,7 +38,7 @@
3838
},
3939
"repository": {
4040
"type": "git",
41-
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure.git"
41+
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure"
4242
},
4343
"author": "Epic Games",
4444
"license": "MIT",

‎Frontend/ui-library/CHANGELOG.md‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,12 @@
11
# @epicgames-ps/lib-pixelstreamingfrontend-ui-ue5.6
22

3+
## 0.1.2
4+
5+
### Patch Changes
6+
7+
- Updated dependencies [05bebea]
8+
- @epicgames-ps/lib-pixelstreamingfrontend-ue5.7@0.2.0
9+
310
## 0.2.0
411

512
### Minor Changes

‎Frontend/ui-library/package.json‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "@epicgames-ps/lib-pixelstreamingfrontend-ui-ue5.7",
3-
"version": "0.1.1",
3+
"version": "0.1.2",
44
"description": "Reference frontend UI library for Unreal Engine 5.7 Pixel Streaming - gives the stock look and feel.",
55
"main": "dist/cjs/pixelstreamingfrontend-ui.js",
66
"module": "dist/esm/pixelstreamingfrontend-ui.js",
@@ -26,15 +26,15 @@
2626
"typescript-eslint": "^8.24.0"
2727
},
2828
"dependencies": {
29-
"@epicgames-ps/lib-pixelstreamingfrontend-ue5.7": "^0.1.0",
29+
"@epicgames-ps/lib-pixelstreamingfrontend-ue5.7": "^0.2.0",
3030
"@babel/runtime": "^7.26.10",
3131
"jss": "^10.10.0",
3232
"jss-plugin-camel-case": "^10.10.0",
3333
"jss-plugin-global": "^10.10.0"
3434
},
3535
"repository": {
3636
"type": "git",
37-
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure.git"
37+
"url": "https://github.com/EpicGamesExt/PixelStreamingInfrastructure"
3838
},
3939
"author": "Epic Games",
4040
"license": "MIT",

‎RELEASE_VERSION‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
0.1.0
1+
0.1.2

‎Signalling/CHANGELOG.md‎

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,12 @@
11
# @epicgames-ps/lib-pixelstreamingsignalling-ue5.6
22

3+
## 0.2.0
4+
5+
### Minor Changes
6+
7+
- 05bebea: Add: Ability to access player id on the frontend.
8+
QoL: Remove player id stripping from the signalling library.
9+
10+
It is useful to be able to use the player id as a unique identifier that is common between UE side stats and frontend side stats; however, the player id is not actually exposed to TS/JS because the SS strips it out of signalling messages.
11+
12+
This change is a backport of "Exposed playerid" (#728)

0 commit comments

Comments
 (0)