-
Notifications
You must be signed in to change notification settings - Fork 30
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
Add client side encryption before Send files to S3 #464
Conversation
gawsoftpl
commented
Dec 11, 2023
- I added client side encryption before send files to S3. Encryption is synchronize, and encrypt and decrypt files with streams.
- Update Readme and nx plugin config files.
Integrate encryption in aws-cache.ts
…and download file - Can set encryption key by NX_CLOUD_ENCRYPTION_KEY or NXCACHE_AWS_ENCRYPTION_KEY
@bojanbass Are you there? |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit b574d16. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ⌛ The following targets are in progress
✅ Successfully ran 1 targetSent with 💌 from NxCloud. |
Give me some time to review this. Thank you for the PR. |
I fix e2e tests and fix format with "yarn format" command |
I think that I have fixed all issues in my pull request. |
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 0c48bb4. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 5 targetsSent with 💌 from NxCloud. |
Fixed all issues, please check. |
packages/nx-aws-cache/src/tasks-runner/encryptor/encrypt-decrypt.spec.ts
Show resolved
Hide resolved
- Move Encrypt and Decrypt to constructor - Fix comments - Fix lint issues
I think this is ready to be merged. Can we cleanup git commit history or we just squash this? |
squash this |
Thank you for contribution and having patience with comments :) |
Ok, but please read my last comment to your test: |
I can add integration tests But how to provide a way to run it with the same command on both environments and be consistent with nx targets? Without docker or docker-compose I cant run minio server in clean and fast way. Best way is create new target "test-integration" with executor nx:run-commands like: packages/nx-aws-cache/project.json "test-integration": {
"executor": "nx:run-commands",
"options": {
"commands": [
"docker-compose up -d",
"npx jest aws-cache.integration.spec.ts"
],
"cwd": "packages/nx-aws-cache"
},
"configurations": {
"ci": {
"ci": true
}
}
} |
So you would like to run |
Yes you are right. In gitlab best way to do it is use service to run minio. Equivalent in github will be prerequisite action. But there will be another issue how to run it with local dev tests? The question is how to run additional service in nx pipeline in dev mode and in ci with one command? All this gives the impression that nx is missing a plugin like @nx/service to run additional service for e2e tests to run service like in gitlab ci pipeline, which will run service and automatically close service. That service should run independent of system and should run in docker or podman. It seems that there is a new idea for a plugin for nx |
Yes, it's tricky. Maybe one idea is to make a custom executor which accepts
|
I have created new discussion in nx nrwl/nx#20903 |
Perfect! |