-
Notifications
You must be signed in to change notification settings - Fork 0
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
refactor!: Open source node #850
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add everything Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove migration tests Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * add ingnore invalid create option for postres mega linter Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Try mega linter 7 Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * edit megalinter.yaml Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Disable some linters Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> --------- Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io>
* Start cleaning up docs Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove more files Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix compilation issue Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Trim docs Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix compilation errors Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove some bitcoin test resources and old docs Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Lint Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix tests Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> --------- Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io>
* ATL-6668: Delete legacy operation related to credentials This commit deletes the IssueCredentialBatchOperation and RevokeCredentialsOperation. It does not delete node API, daos nor repositories related to credentials * ATL-6668: Delete more files related to VCs This commit deletes even more files related to VC operations. It also removes some dependencies on the old SDK * ATL-6668: Delete unused tables This commit deletes tables and indexes from VC legacy operations * ATL-6668: Fix formatting * ATL-6668: Address review comments This commit deletes references to legacy VC code in protobuf and sql definitions
* ATL-6924: Replace users of ECPublicKey This commit replaces the used of ECPublicKey * ATL-6924: Simplify models This commit simplifies models by deleting classes not needed at parsing time * ATL-6924: Simplify types This commit replaces some uses of Array and uses Vector instead to allow more transparent equality checks * ATL-6924: Add support for encoding functions This commit adds support to decode hex strings into Sha256Hash bytes * ATL-6924: Refactor CryptoUtils and remove SHA256Digest This commit improves the organization of CryptoUtils methods It also delete the used of SDK methods related to Sha256 hashing
* ATL-6926: Remove uses of EC classes This commit removes most uses of the legacy crypto SDK * ATL-6926: Fix missing implementations This commit adds missing implementations for key encoding algorithms * ATL-6926: Fix tests * ATL-6926: Clean up code and tests * ATL-6926: Correct typo and add tests This commit adds the final tests to validate the cryptography implementation that replaces the SDK
* clean up source files, replace DID usages Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix some tests Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Linting Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix test Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix fromString on DID Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * formatting Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Address some PR comments Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> --------- Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io>
* Remove uses of the old SDK This commit removes the uses of the old SDK and update tests * Refactor CryptoUtils class This commit re-organizes and renames methods for better use * Remove last dependency on old SDK This commit removes the last dependencies on the old SDK
* ATL-6669: Update README This commits updates the README to focus oly on the PRISM node. * ATL-6669: Add IDE notes in README This commit adds instructions to load the project in different IDEs. The commit also removes incorrect sentences * ATL-6669: Delete old README and complete new one * ATL-6669: Add pre-commit configuration This commit adds a pre-commit hook that runs scalafmt and instructions to configure it * ATL-6669: Refer to identus configuration This commit updates the README and refers users to read identus documentation in order to configure the node.
This commit deletes the legacy DID based authentication
* Change some build files Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Rmove unused files Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove unused files Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove outer node folder Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Revert version name Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove unneeded resolver from the build Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Update fs2 Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Start adding tests Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Add more tests Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Fix 1 failing test Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Minor adjustment in docs Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> * Remove test report aggregation step Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io> --------- Signed-off-by: Shota Jolbordi <shota.jolbordi@iohk.io>
…eys table (#848) * Add http scheme support and removed the contraint from publickeys table Signed-off-by: mineme0110 <shailesh.patil@iohk.io> * ran scalafmt Signed-off-by: mineme0110 <shailesh.patil@iohk.io> --------- Signed-off-by: mineme0110 <shailesh.patil@iohk.io>
Unit Test Results289 tests - 39 288 ✔️ - 39 24s ⏱️ -11s Results for commit 76d49e2. ± Comparison against base commit 15e6865. This pull request removes 80 and adds 41 tests. Note that renamed tests count towards both.
♻️ This comment has been updated with latest results. |
🦙 MegaLinter status:
|
Descriptor | Linter | Files | Fixed | Errors |
---|---|---|---|---|
✅ ACTION | actionlint | 2 | 0 | |
✅ JSON | jsonlint | 9 | 0 | |
✅ JSON | npm-package-json-lint | yes | no | |
prettier | 9 | 1 | ||
✅ JSON | v8r | 9 | 0 | |
markdownlint | 7 | 109 | ||
markdown-link-check | 7 | 12 | ||
markdown-table-formatter | 7 | 1 | ||
protolint | 4 | 4 | ||
✅ REPOSITORY | dustilock | yes | no | |
✅ REPOSITORY | git_diff | yes | no | |
✅ REPOSITORY | grype | yes | no | |
kics | yes | 49 | ||
✅ REPOSITORY | syft | yes | no | |
✅ REPOSITORY | trivy-sbom | yes | no | |
trufflehog | yes | 1 | ||
✅ SQL | sql-lint | 2 | 0 | |
✅ XML | xmllint | 2 | 0 | |
prettier | 3 | 1 | ||
✅ YAML | v8r | 3 | 0 | |
✅ YAML | yamllint | 3 | 0 |
See detailed report in MegaLinter reports
Set VALIDATE_ALL_CODEBASE: true
in mega-linter.yml to validate all sources, not only the diff
yshyn-iohk
approved these changes
May 17, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Screenshots
Checklists
Pre-submit checklist:
Pre-merge checklist: