crossbario / crossbario/autobahn-python

XBR: fix or replace pysha3 build on PyPy

Open
#1,247 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug CI-CD PRIO HIGH wamp XBR
Dominant language
Python
Stars
2.5k
Forks
768
PR merge metrics
No merged PRs in 30d

Description

the build fails, see https://gist.github.com/oberstet/fec756b9489a8b046bfd28813bfe2de1

reason: pysha3 uses CPyExt - I have no clue why they did that, nor why they use CPyExt

background: ehtereum uses keccack256 for hashing, a pre-standard sha3-256 variant

the python 3.6+ standard library contains that variant:

New in version 3.6: SHA3 (Keccak) and SHAKE constructors sha3_224(), sha3_256(), sha3_384(), sha3_512(), shake_128(), shake_256().

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reproducing the PyPy build failure from the linked gist and inspect how pysha3 currently uses CPyExt. Compare that path with the Python 3.6+ standard-library SHA3/Keccak constructors; done means the PyPy build works while Keccak-256 support remains available.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
build-system
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.