search

security-anthem / MAID

MAID -MAIl Detective-
MIT License
3 stars 6 forks source link

data_registerの実装 #23

Closed MMikipenguin closed 3 years ago

MMikipenguin commented 3 years ago

data_register.pyを作成しました．

Pythonの関数の引数やSQLのinsert文について正確にコーディングできているかわかりません． それぞれのテーブルのIDについても実装が正しいかわかりません．

ご確認お願いします

MMikipenguin commented 3 years ago

現在Linux仮想マシン環境でテストをしています． 文法的に誤った箇所は修正しています．

init.pyの実装で，真偽値をとるspfなどの要素をbitとして実装していたようで，booleanに修正したほうが良いかもしれません． また，serial型だと，キーを各テーブルに格納するたび値がインクリメントされるので，どう直したらよいでしょうか．

よろしくお願いします．

shu1oka28 commented 3 years ago

init.pyの実装で，真偽値をとるspfなどの要素をbitとして実装していたようで，booleanに修正したほうが良いかもしれません． boolean型に修正でよいと思います。

また，serial型だと，キーを各テーブルに格納するたび値がインクリメントされるので，どう直したらよいでしょうか． 主キーは、一意に定まればいいのでインクリメントされてよいと思います。 外部キーになっている属性はinit.pyで整数の型で宣言した方がよいかもしれないです。

shu1oka28 commented 3 years ago

https://blog.ohgaki.net/are-you-using-prepared-query-only

プリペアドステートメントを使用しないとSQLインジェクションの脆弱性を作りこんでしまいます。そこだけ注意して修正お願いします。

shu1oka28 commented 3 years ago

関数の引数について、json形式の文字列が入ると想定してください。

MMikipenguin commented 3 years ago

修正しました． ご指摘いただきありがとうございます．

ご確認お願いします．