test: Replace random_bytes() with random.randbytes() #28720

issue maflcko openend this issue on October 24, 2023
  1. maflcko commented at 5:10 pm on October 24, 2023: member

    Motivation

    Now that Python3.9 is available, the random_bytes helper function can be removed.

    Possible solution

    No response

    Useful Skills

    • Compiling Bitcoin Core from source
    • Running the C++ unit tests and the Python functional tests
    • Python 3.9

    Guidance for new contributors

    Want to work on this issue?

    For guidance on contributing, please read CONTRIBUTING.md before opening your pull request.

  2. maflcko added the label good first issue on Oct 24, 2023
  3. maflcko added the label Tests on Oct 24, 2023
  4. nsvrn commented at 3:40 am on October 25, 2023: contributor
    I can take care of this, if I understand correctly remove the random_bytes function in the util and instead utilize the randbytes from Python 3.9
  5. nsvrn referenced this in commit e517a37168 on Oct 25, 2023
  6. nsvrn referenced this in commit fe3ac3700d on Oct 25, 2023
  7. fanquake closed this on Oct 29, 2023

  8. Julio-Rats referenced this in commit f028470654 on Jan 26, 2024
  9. bitcoin locked this on Oct 29, 2024

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-12-22 12:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me