-
Notifications
You must be signed in to change notification settings - Fork 29
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
Private key security updates/fixes #2108
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2108 +/- ##
=======================================
Coverage 94.13% 94.13%
=======================================
Files 255 255
Lines 15810 15810
=======================================
Hits 14883 14883
Misses 927 927
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
if [[ "$AEA_PASSWORD" != "" ]]; then | ||
aea add-key cosmos --connection --password $AEA_PASSWORD | ||
aea issue-certificates --password $AEA_PASSWORD | ||
else | ||
aea add-key cosmos --connection | ||
aea issue-certificates | ||
fi |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #2088
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trader-quickstart related PR valory-xyz/trader-quickstart#55 has been updated to latest branch (now on develop branch):
- Issue Password stored in clear in deployment files #2089 seems to be resolved in this PR.
- Issue Framework fails when running with password-protected keys #2088 has not been resolved yet. Please, see the detailed comments on that issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Steps to test this PR:
Build the agent image according to this PR:
cd open-autonomy/deployments/Dockerfiles/autonomy
docker build . -t valory/open-autonomy:rc0
Edit file open-autonomy/autonomy/data/Dockerfiles/agent/Dockerfile
Change
FROM ${AUTONOMY_IMAGE_NAME}:${AUTONOMY_IMAGE_VERSION}
by
FROM ${AUTONOMY_IMAGE_NAME}:rc0
Reinstall framework from local directory:
pip3 install -e ../../open-autonomy
@angrybayblade you can now mark issues #2108 and #2088 as completed.
Proposed changes
This PR
Fixes
fixes #2088
fixes #2089
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an
x
in the box that appliesChecklist
Put an
x
in the boxes that apply.main
branch (left side). Also you should start your branch off ourmain
.