API cleanup
まだ誰も着手していません。
評価
調査の方向性
まず、src/_frida.c と frida/core.py の API 構造を比較し、Core/init の分割と同期・非同期インターフェースを含めて確認します。どの Python クラスが C の機能をラップしているかを対応付け、それらを変更した場合の互換性への影響を特定します。実装前に、合意済みの整理範囲と移行計画が定まっていれば完了とします。
索引モデルが issue の本文から書いたものです。
説明
The current API is a bit messy, especially compared to comparative wrappers like frida-node. I think we have three main areas where we should focus our cleanup one:
- Core and init merge - if there is a reason for the split I would love to know but as it seems, the split only make the API confusing and verbose.
- Uniform interface - we have the C wrapper of Frida in
src/_frida.cand the Python wrapper of the C wrapper atfrida/core.py. I think we should try to merge them when we can, sometimes we have Python classes that are just dumb wrappers. When we can't merge them, we should only expose the Python wrapper no matter what, otherwise it become confusing really fast. - Async everything - We only support Python 3.7+ and soon will drop support for 3.7 (EOF in June), so we can use the "new"
asynciopackage. We would keep the sync methods with a_syncprefix, at least I think we should.
The big question of course is how to handle this cleanup, it will break all usages of the library but it's probably for the best.
- 主要言語
- Python
- スター
- 873
- フォーク
- 173
- PR マージ指標
- 30日以内にマージされた PR はありません
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
frida/frida-python のほかの issue
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
frida/frida-python#265 · コメント 2 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
frida/frida-python#262 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 35/100
frida/frida-python#247 · コメント 2 件 · リアクション 2 件 ·
-
After a long period of hooking applications on Ubuntu, Python and JavaScript are unable to interact. オープン
難易度 4/5 3〜5日 初心者へのやさしさ 25/100
frida/frida-python#246 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 20/100
frida/frida-python#243 ·
frida/frida-python の issue をすべて見る
似ている issue
-
area: harness bug status: needs-triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
Human-Agent-Society/reef#625 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 80/100
learningequality/kolibri#15351 · コメント 2 件 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
Name consistency オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
eellak/triplestore#65 · コメント 1 件 ·