feat: Generate + store station_id keypair and pass to Zinnia via env var#406
feat: Generate + store station_id keypair and pass to Zinnia via env var#406PatrickNercessian wants to merge 16 commits intoCheckerNetwork:mainfrom
Conversation
|
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
|
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring: Next stepsTake a deeper look at the dependencyTake a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support [AT] socket [DOT] dev. Remove the packageIf you happen to install a dependency that Socket reports as Known Malware you should immediately remove it and select a different dependency. For other alert types, you may may wish to investigate alternative packages or consider if there are other ways to mitigate the specific risk posed by the dependency. Mark a package as acceptable riskTo ignore an alert, reply with a comment starting with |
1c5af33 to
0f42c7c
Compare
juliangruber
left a comment
There was a problem hiding this comment.
Great work so far, @PatrickNercessian! 👏
c024a04 to
f8a9770
Compare
|
@PatrickNercessian would you mind adding Station ID to the Proposed change in - startPingLoop().unref()
+ startPingLoop({ STATION_ID }).unref() |
bajtos
left a comment
There was a problem hiding this comment.
LGTM 👍🏻
@juliangruber: Could you please check the new keytar interaction when both Desktop and Core read from the keychain at startup?
|
Tested and works 👍 |
|
@SocketSecurity ignore npm/keytar@7.9.0 |
This comment was marked as resolved.
This comment was marked as resolved.
Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
|
The builds are failing on Linux now. |
Signed-off-by: Miroslav Bajtoš <oss@bajtos.net>
|
I added 0fe59a9 to install We are getting a different error now: @PatrickNercessian could you please look into this yourself? Maybe we can copy the setup steps from keytar's CI, see the following two blocks:
That code is more than two years old; there may be easier options available by now. Feel free to use a different approach to fix this problem! |
Links:
CheckerNetwork/roadmap#96