Skip to content
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

feat(adex-cli): provide activation request types to prevent malicious… #1912

Merged
merged 8 commits into from
Jul 26, 2023

Conversation

rozhkovdmitrii
Copy link

@rozhkovdmitrii rozhkovdmitrii commented Jul 12, 2023

Initial comment: #1871 (comment)

Activation types have been introduced to be sure if the malicious person substituted them in the activation scheme file it would not lead to an unexpected action.

@rozhkovdmitrii rozhkovdmitrii added bug Something isn't working adex-cli labels Jul 12, 2023
@rozhkovdmitrii rozhkovdmitrii self-assigned this Jul 12, 2023
@rozhkovdmitrii rozhkovdmitrii added 1.0.6-beta and removed bug Something isn't working labels Jul 12, 2023
… intents when activation scheme is being used
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

Thank you for the fast fix! First review iteration!

mm2src/mm2_rpc/src/data/legacy.rs Outdated Show resolved Hide resolved
mm2src/mm2_rpc/src/data/legacy.rs Outdated Show resolved Hide resolved
mm2src/mm2_rpc/src/data/legacy.rs Outdated Show resolved Hide resolved
mm2src/mm2_rpc/src/data/legacy.rs Outdated Show resolved Hide resolved
shamardy
shamardy previously approved these changes Jul 19, 2023
Copy link
Collaborator

@shamardy shamardy left a comment

Choose a reason for hiding this comment

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

Only a few non-blockers! Please merge with latest dev to fix conflicts and pass the adex-cli failing tests.
I will also not merge this PR until @Alrighttt approval, it can wait to v1.0.7-beta too.

mm2src/adex_cli/src/rpc_data.rs Show resolved Hide resolved
mm2src/adex_cli/src/rpc_data.rs Show resolved Hide resolved
mm2src/adex_cli/src/rpc_data.rs Show resolved Hide resolved
mm2src/adex_cli/src/rpc_data.rs Show resolved Hide resolved
rozhkovdmitrii added 2 commits July 19, 2023 22:18
@shamardy shamardy merged commit 80f7e6f into dev Jul 26, 2023
26 of 30 checks passed
@shamardy shamardy deleted the feature-1682-interoperating branch July 26, 2023 00:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants