Skip to content

ci: merge staging to master#596

Merged
maxwell-aiden merged 22 commits intomasterfrom
staging
Oct 24, 2023
Merged

ci: merge staging to master#596
maxwell-aiden merged 22 commits intomasterfrom
staging

Conversation

@maxwell-aiden
Copy link
Copy Markdown
Member

This is an automatic PR generated by the pipeline CI/CD. This will be automatically fast-forward merged if successful.

@maxwell-aiden maxwell-aiden self-assigned this Oct 19, 2023
@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1042105916 for 56a6de5

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1042105916

@ghost
Copy link
Copy Markdown

ghost commented Oct 19, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

amydevs and others added 2 commits October 21, 2023 12:15
* only `PolykeyAgentOptions` and `PolykeyClientOptions` should be used,
  all other subdomains should use flattened options
* updated all uses of subdomains to stop using `options`
* `PolykeyClient` encapsulates certificate verification behaviour
* `PolykeyClient` creation is now timed cancellable
* new utilities to deal with `NodeId` - `isNodeId`, `assertNodeId`, `generateNodeId`
…ulation

Encapsulate WebSocketClient in PolykeyClient
@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1044477323 for 426b5dd

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1044477323

addievo and others added 3 commits October 21, 2023 12:28
Fixing identities claim to be in-line with identities discover and identities trust, by introducing a new parser parseGestaltIdentityId
@CMCDragonkai
Copy link
Copy Markdown
Member

@amydevs I think you forgot to lintfix the GitHubProvider.ts, I've done this now.

@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1044480172 for 5b01c9e

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1044480172

@CMCDragonkai
Copy link
Copy Markdown
Member

@amydevs there's a test failure in identities: https://gitlab.com/MatrixAI/open-source/Polykey/-/jobs/5342777910 the property test failed, can you check, might have been a regression from the GitHubProvider.

@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1046992342 for 9f52ca0

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1046992342

* general refactor of the signalling protocol.
* Added signatures and verification to the signalling requests and relay messages. #148
NAT Signalling (Hole Punching) should be Fire & Forget, Coalesced, and Secured with Signatures
@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1047081110 for 479f8b6

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1047081110

@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1047101227 for ec8b63f

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1047101227

Fix `recoveryCode`, `privateKey`, `privateKeyPath` options usage in `PolykeyAgentOptions`
@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Attempt on 1047179674 for 4cba6af

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1047179674

@maxwell-aiden
Copy link
Copy Markdown
Member Author

Pipeline Succeeded on 1047179674 for 4cba6af

https://gitlab.com/MatrixAI/open-source/Polykey/-/pipelines/1047179674

@maxwell-aiden maxwell-aiden merged commit 4cba6af into master Oct 24, 2023
@CMCDragonkai
Copy link
Copy Markdown
Member

If this wasn't caught MatrixAI/Polykey-CLI#42 (comment) then we need more tests here.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants