user_pin and so_pin in Token.open() are documented as bytes, instead they are str

オープン 初心者向け
#225 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1時間未満
初心者へのやさしさ
65/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
停滞
技術スタック
python
領域
documentation

調査の方向性

Token.open() の docstring から始め、user_pin と so_pin について記載されている型を、issue に示されている encode 呼び出しと比較します。パラメーターのドキュメントを、実装が受け付ける string 値に一致するよう更新し、その後、関連する既存のテストまたはドキュメントチェックを実行します。type-checking ツールでこの不一致が報告されなくなれば完了です。

索引モデルが issue の本文から書いたものです。

説明

In the docstring of the Token open function user_pin and so_pin are documented as bytes, but the code assumes they are strings:

      elif user_pin is not None:
            pin = user_pin.encode('utf-8')
            c_user_type = user_type if user_type is not None else CKU_USER
        elif so_pin is not None:
            pin = so_pin.encode('utf-8')
            c_user_type = CKU_SO

This causes pycharm (and other type checking tools) to complain

主要言語
Python
スター
170
フォーク
79
30日以内にマージされた PR はありません

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

pyauth/python-pkcs11 のほかの issue

pyauth/python-pkcs11 の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。