-
Notifications
You must be signed in to change notification settings - Fork 252
Add TokenSource token fetching abstraction
#1645
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from 41 commits
Commits
Show all changes
54 commits
Select commit
Hold shift + click to select a range
7a2f59f
feat: port ConnectionCredentials implementation from agent-starter-re…
1egoman 8f6fb41
refactor: move roomName / participantName docs to ConnectionDetails
1egoman 70416fe
refactor: reorder literal and custom
1egoman 7f8cce4
fix: rename SandboxTokenServer -> SandboxTokenServerOptions
1egoman 6929247
feat: optimistically regenerate new credentials on disconnect
1egoman 981f80c
fix: add changeset
1egoman 1059a82
fix: run prettier
1egoman f2c07fe
fix: add BigInt
1egoman 4e54fec
fix: remove @types/node to try to fix ci build issue
1egoman 4173823
fix: migrate back to known good pnpm lock
1egoman a3ff2d9
feat: use logger for warnings instead of console
1egoman 70ed1a2
feat: add ConnectionCredentials.Response / ConnectionCredentials.Resp…
1egoman 8edc1ae
feat: add mechanism to get room config from token rather than returni…
1egoman c48c7c1
feat: add ability to explicitly pass ConnectionCredentials.Request in…
1egoman f8c92f2
fix: run npm run format
1egoman 30c825f
fix: remove dead code
1egoman 8a8a549
feat: re-add accidentally removed roomConfig from SandboxTokenServerO…
1egoman e85f4b2
fix: address lint issue
1egoman 3e3b300
feat: change ConnectionCredentials so Request gets set ahead of time …
1egoman 1b346df
feat: get rid of `roomName` / `participantName` from response, these …
1egoman b8301be
feat: get rid of connectionCredentialsRequest parameter when calling …
1egoman 379eda6
fix: run npm run format
1egoman 43bb782
fix: run eslint --fix
1egoman c2848bc
fix: address issue where token expiry error would always show on the …
1egoman a51dff8
feat: store currently active credentials fetch and re-use it if multi…
1egoman ab012bc
feat: remove ability to inject custom logger name
1egoman a9e038d
feat: add new ConnectionCredentials.Refreshable abstract base class
1egoman 352f89f
fix: update class name in docs comment
1egoman 4d83430
feat: add participantIdentity / participantMetadata / participantAttr…
1egoman f6150cf
feat: updated deprecation warnings to include mention of ConnectionCr…
1egoman 60a5e16
feat: remove SandboxTokenServer security warning, it sounds like some…
1egoman 15255fe
feat: rename from ConnectionCredentials -> TokenSource
1egoman 1f8089d
fix: update old token source name
1egoman 9791955
feat: remove individual getters and return the whole jwt payload
1egoman fa47c48
feat: add -Payload suffix to TokenSource.Request / TokenSource.Respon…
1egoman cfbc9e2
feat: add new TokenSource.Endpoint and update TokenSource.SandboxToke…
1egoman 9baa260
feat: add old sandbox token server back temporarily as SandboxTokenSe…
1egoman de1f6f6
fix: update to TokenSource name in changeset
1egoman 8c8fba0
fix: update to tokenSource name in Room class
1egoman 5a767cb
feat: migrate to using static method constructors for TokenSource types
1egoman 058f10b
feat: add TokenSourceRequest/TokenSourceResponse protobuf messages in…
1egoman cf26880
feat: expose protobufs in TokenSource.Custom implementation over raw …
1egoman ba1dd74
feat: get rid of tokensource integration logic in Room temporarily
1egoman bcfa010
feat: more over TokenSource modifications from components-js
1egoman dd811e6
refactor: split up TokenSource into multiple files and add lots of docs
1egoman 81ab62d
fix: update missing instance of RoomConfigurationPayload
1egoman e1bdbfc
fix: address linting errors
1egoman 44509bd
fix: run npm run format
1egoman 6145659
fix: rename TokenSource to TokenSourceBase to fix name clash
1egoman e3a9034
feat: export more tokensource values
1egoman 9a17386
feat: rename TokenSourceOptions => TokenSourceFetchOptions
1egoman 438c5dc
fix: re-add missing Promise<void>
1egoman 98eac04
feat: add agentMetadata param alongside agentName
1egoman c8b46ae
fix: allow second parameter to TokenSource.sandboxTokenServer to be o…
1egoman File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| 'livekit-client': patch | ||
| --- | ||
|
|
||
| add TokenSource token fetching abstraction |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.