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. ns-xvrn 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. ns-xvrn referenced this in commit e517a37168 on Oct 25, 2023
  6. ns-xvrn referenced this in commit fe3ac3700d on Oct 25, 2023
  7. fanquake closed this on Oct 29, 2023


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-09-29 01:12 UTC

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