Skip to content
This repository was archived by the owner on Jun 17, 2020. It is now read-only.

Conversation

@rvl
Copy link
Contributor

@rvl rvl commented Mar 20, 2019

Fix problems found when doing input-output-hk/haskell.nix#93.

@rvl rvl self-assigned this Mar 20, 2019
@rvl rvl requested a review from a team March 20, 2019 03:28
rvl added a commit to input-output-hk/haskell.nix that referenced this pull request Mar 20, 2019
rvl added a commit to input-output-hk/haskell.nix that referenced this pull request Mar 20, 2019
Copy link
Contributor

@angerman angerman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Is there anyway one could exfil this key?

@rvl
Copy link
Contributor Author

rvl commented Mar 20, 2019

Is there anyway one could exfil this key?

Yes, a buildkite build could cat the ssh key file and it would appear in the logs.

@angerman
Copy link
Contributor

Is there anyway one could exfil this key?

Yes, a buildkite build could cat the ssh key file and it would appear in the logs.

Should we prevent buildkite to run on untrusted PRs then?

@rvl
Copy link
Contributor Author

rvl commented Mar 20, 2019

Should we prevent buildkite to run on untrusted PRs then?

Yes. Already, Buildkite will not run PR builds from users who don't have repo write access.

@rvl
Copy link
Contributor Author

rvl commented Mar 21, 2019

bors r+

iohk-bors bot added a commit that referenced this pull request Mar 21, 2019
549: Buildkite: fix NIX_PATH and add deployment key for haskell.nix r=rvl a=rvl

Fix problems found when doing input-output-hk/haskell.nix#93.

Co-authored-by: Rodney Lorrimar <rodney.lorrimar@iohk.io>
@iohk-bors
Copy link
Contributor

iohk-bors bot commented Mar 21, 2019

Build succeeded

@iohk-bors iohk-bors bot merged commit 9e37d26 into master Mar 21, 2019
@iohk-bors iohk-bors bot deleted the rvl/buildkite-haskell-nix branch March 21, 2019 03:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants