Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 0 additions & 39 deletions .github/ISSUE_TEMPLATE.md

This file was deleted.

92 changes: 92 additions & 0 deletions .github/ISSUE_TEMPLATE/bug.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,92 @@
name: Bug report
description: Submit a new bug report.
labels: [bug]
body:
- type: markdown
attributes:
value: |
## This issue tracker is only for technical issues related to Dash Core.

* General Dash questions and/or support requests are best directed to the Dash Forum at https://www.dash.org/forum/.
* For reporting security issues, please read instructions at https://github.com/dashpay/dash/blob/master/SECURITY.md.
* If the node is "stuck" during sync or giving "block checksum mismatch" errors, please ensure your hardware is stable by running `memtest` and observe CPU temperature with a load-test tool such as `linpack` before creating an issue.

----
- type: checkboxes
attributes:
label: Is there an existing issue for this?
description: Please search to see if an issue already exists for the bug you encountered.
options:
- label: I have searched the existing issues
required: true
- type: textarea
id: current-behaviour
attributes:
label: Current behaviour
description: Tell us what went wrong
validations:
required: true
- type: textarea
id: expected-behaviour
attributes:
label: Expected behaviour
description: Tell us what you expected to happen
validations:
required: true
- type: textarea
id: reproduction-steps
attributes:
label: Steps to reproduce
description: |
Tell us how to reproduce your bug. Please attach related screenshots if necessary.
* Run-time or compile-time configuration options
* Actions taken
validations:
required: true
- type: textarea
id: logs
attributes:
label: Relevant log output
description: |
Please copy and paste any relevant log output or attach a debug log file.

You can find the debug.log in your [data dir.](https://github.com/dashpay/dash/blob/master/doc/files.md#data-directory-location)

Please be aware that the debug log might contain personally identifying information.
validations:
required: false
- type: dropdown
attributes:
label: How did you obtain Dash Core
multiple: false
options:
- Compiled from source
- Pre-built binaries
- Package manager
- Other
validations:
required: true
- type: input
id: core-version
attributes:
label: What version of Dash Core are you using?
description: Run `dashd --version` or in Dash-Qt use `Help > About Dash Core`
placeholder: e.g. v23.1.9 or develop@f7450ed
validations:
required: true
- type: input
id: os
attributes:
label: Operating system and version
placeholder: e.g. "MacOS Ventura 13.2" or "Ubuntu 22.04 LTS"
validations:
required: true
- type: textarea
id: machine-specs
attributes:
label: Machine specifications
description: |
What are the specifications of the host machine?
e.g. OS/CPU and disk type, network connectivity
validations:
required: false
41 changes: 0 additions & 41 deletions .github/ISSUE_TEMPLATE/bug_report.md

This file was deleted.

8 changes: 8 additions & 0 deletions .github/ISSUE_TEMPLATE/config.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
blank_issues_enabled: true
contact_links:
- name: Dash Core Security Policy
url: https://github.com/dashpay/dash/blob/master/SECURITY.md
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Resolved

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🟡 Suggestion: Security-policy links point to the stale SECURITY.md on master instead of develop

Both this URL and .github/ISSUE_TEMPLATE/bug.yml:11 direct security reporters to master's SECURITY.md, which lists only 23.x as supported. SECURITY.md at the reviewed head lists both 24.x and 23.x, and develop is Dash's default branch. Retaining Bitcoin Core's master branch name in this adaptation therefore sends reporters to an outdated support matrix. Change both links to blob/develop/SECURITY.md so reporters see the current policy.

Suggested change
url: https://github.com/dashpay/dash/blob/master/SECURITY.md
url: https://github.com/dashpay/dash/blob/develop/SECURITY.md

source: glm-5.3-flash (phase1-reviewer: general)

about: View security policy
- name: Dash Forum
url: https://www.dash.org/forum/
about: General Dash questions and support requests
20 changes: 0 additions & 20 deletions .github/ISSUE_TEMPLATE/feature_request.md

This file was deleted.

36 changes: 36 additions & 0 deletions .github/ISSUE_TEMPLATE/feature_request.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
name: Feature Request
description: Suggest an idea for this project.
labels: [enhancement]
body:
- type: textarea
id: feature
attributes:
label: Please describe the feature you'd like to see added.
description: Attach screenshots or logs if applicable.
validations:
required: true
- type: textarea
id: related-problem
attributes:
label: Is your feature related to a problem, if so please describe it.
description: Attach screenshots or logs if applicable.
validations:
required: false
- type: textarea
id: solution
attributes:
label: Describe the solution you'd like
validations:
required: false
- type: textarea
id: alternatives
attributes:
label: Describe any alternatives you've considered
validations:
required: false
- type: textarea
id: additional-context
attributes:
label: Please leave any additional context
validations:
required: false
22 changes: 0 additions & 22 deletions .github/ISSUE_TEMPLATE/good_first_issue.md

This file was deleted.

41 changes: 41 additions & 0 deletions .github/ISSUE_TEMPLATE/good_first_issue.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,41 @@
name: Good First Issue
description: (Regular devs only) Suggest a new good first issue
labels: [Good First Issue]
body:
- type: markdown
attributes:
value: |
Please add the label "Good First Issue" manually before or after opening

A good first issue is an uncontroversial issue, that has a relatively unique and obvious solution

Motivate the issue and explain the solution briefly
- type: textarea
id: motivation
attributes:
label: Motivation
description: Motivate the issue
validations:
required: true
- type: textarea
id: solution
attributes:
label: Possible solution
description: Describe a possible solution
validations:
required: false
- type: textarea
id: useful-skills
attributes:
label: Useful Skills
description: For example, “`std::thread`”, “Qt5 GUI and async GUI design” or “basic understanding of Dash masternodes and the Dash Core RPC interface”.
validations:
required: false
- type: textarea
attributes:
label: Guidance for new contributors
description: Please leave this to automatically add the footer for new contributors
value: |
Want to work on this issue?

For guidance on contributing, please read [CONTRIBUTING.md](https://github.com/dashpay/dash/blob/master/CONTRIBUTING.md) before opening your pull request.
4 changes: 2 additions & 2 deletions contrib/seeds/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ Utility to generate the seeds.txt list that is compiled into the client
(see [src/chainparamsseeds.h](/src/chainparamsseeds.h) and other utilities in [contrib/seeds](/contrib/seeds)).

The seeds compiled into the release are created from the current protx list and AS map
data, like this:
data. Run the following commands from the `/contrib/seeds` directory:
Comment thread
coderabbitai[bot] marked this conversation as resolved.
Resolved

```bash
dash-cli protx list valid true 2563000 > protx_list.json
Expand All @@ -18,7 +18,7 @@ do
nc -v -x 127.0.0.1:9050 -z $address $port
done < "onion_seeds.txt"

python3 makeseeds.py -a asmap-filled.dat protx_list.json onion_seeds.txt > nodes_main.txt
python3 makeseeds.py -a asmap-filled.dat -p protx_list.json -o onion_seeds.txt > nodes_main.txt
python3 generate-seeds.py . > ../../src/chainparamsseeds.h
```

Expand Down
2 changes: 1 addition & 1 deletion contrib/seeds/asmap.py
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ class _Instruction(Enum):
# Not an actual instruction, but a way to encode the empty program that fails. In the
# encoder, it is used more generally to represent the failure case inside MATCH instructions,
# which may (if used inside the context of a DEFAULT instruction) actually correspond to
# a succesful return. In this usage, they're always converted to an actual MATCH or RETURN
# a successful return. In this usage, they're always converted to an actual MATCH or RETURN
# before the top level is reached (see make_default below).
END = 4

Expand Down
18 changes: 8 additions & 10 deletions contrib/seeds/makeseeds.py
Original file line number Diff line number Diff line change
Expand Up @@ -154,8 +154,8 @@ def ip_stats(ips: List[Dict]) -> str:
def parse_args():
argparser = argparse.ArgumentParser(description='Generate a list of dash node seed ip addresses.')
argparser.add_argument("-a","--asmap", help='the location of the asmap asn database file (required)', required=True)
argparser.add_argument("protx_list", nargs='?', help='json as outputted by "protx list valid 1" (default: stdin)')
argparser.add_argument("onion_seeds", nargs='?', help='file with onion seeds, one per line')
argparser.add_argument("-p","--protx", help='the location of the protx list file, json as outputted by "protx list valid 1" (required)', required=True)
argparser.add_argument("-o","--onion", help='the location of the onion seeds file, one per line (optional)')
return argparser.parse_args()

def main():
Expand All @@ -166,16 +166,14 @@ def main():
asmap = ASMap.from_binary(f.read())
print('Done.', file=sys.stderr)

# This expects a json as outputted by "protx list valid 1"
if args.protx_list is not None:
with open(args.protx_list, 'r', encoding="utf8") as f:
mns = json.load(f)
else:
mns = json.load(sys.stdin)
print(f'Loading and parsing protx list "{args.protx}"…', end='', file=sys.stderr, flush=True)
with open(args.protx, 'r', encoding="utf8") as f:
mns = json.load(f)
print('Done.', file=sys.stderr)

onions = []
if args.onion_seeds is not None:
with open(args.onion_seeds, 'r', encoding="utf8") as f:
if args.onion is not None:
with open(args.onion, 'r', encoding="utf8") as f:
onions = f.read().split('\n')

print(f'Total mns: {len(mns)}', file=sys.stderr)
Expand Down
Loading