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

chore: deprecate an option platform-user in favour of whitelist #6209

Merged
merged 1 commit into from
Sep 27, 2024

Conversation

knst
Copy link
Collaborator

@knst knst commented Aug 13, 2024

Issue being fixed or feature implemented

Hard-coded restrictions for platform-user are super-seeded by whitelist feature.

What was done?

Before actually removing feature, let's make it deprecated for now
Split from #6105
6105 - to close and re-open for next major release.

It deprecates old command line argument -platform-user by renaming to -deprecated-platform-user

How Has This Been Tested?

See new 2 functional tests: rpc_deprecated_platform_filter.py and rpc_external_queue.py which are split from rpc_platform_filter.py

Breaking Changes

Command line argument -platform-user is renamed to -deprecated-platform-user

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone

src/init.cpp Show resolved Hide resolved
UdjinM6
UdjinM6 previously approved these changes Sep 3, 2024
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK 668efc4

@knst knst added this to the 22 milestone Sep 4, 2024
Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

LGTM; gonna wait to review until 21.2 is branched off to approve / merge

@PastaPastaPasta PastaPastaPasta marked this pull request as draft September 5, 2024 19:06
@knst knst marked this pull request as ready for review September 26, 2024 07:00
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK e2c66ae

Copy link
Member

@PastaPastaPasta PastaPastaPasta left a comment

Choose a reason for hiding this comment

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

utACK e2c66ae

@PastaPastaPasta PastaPastaPasta merged commit 9272ae2 into dashpay:develop Sep 27, 2024
27 checks passed
thephez added a commit to thephez/dash-user-docs that referenced this pull request Nov 12, 2024
thephez added a commit to dashpay/docs that referenced this pull request Nov 12, 2024
* docs(api): update whitelist description

* docs(rpc): deprecate platform user

Related to dashpay/dash#6209

* docs(rpc): change example formatting

* docs(rpc): add rpc queue info

Describes the new "external user" queue from dashpay/dash#6045

* fix(rpc): correct code block type

* style: filename

* docs(rpc): link to whitelist settings from deprecated platform-user section
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants