Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[PHP 8.4] Sodium関数追加分の翻訳 #178

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

Ippey
Copy link
Contributor

@Ippey Ippey commented Nov 19, 2024

php/doc-en#3878 の変更を反映しました。

Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

多く目のファイル数ありがとうございます!
ほとんど問題ないと思うのですが、2点ほど確認お願いします!

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

EN-Revision を更新頂けると助かります!

これを書いている時点の最新 = 元となるプルリクエスト = 01cb7d495c454cea3828d75e181241b6ac3f15ae でした。

<simpara>
<function>sodium_crypto_aead_aegis256_encrypt</function>
が生成したフォーマット
(暗号化されたテキスト、タグを連結したもの)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

質問です。暗号はあまり詳しくないので、おかしな質問だとしたらすみません。

既存の他の sodium_crypto_aead_*_decrypt は:

  • 全ての日本語訳に、次の文章

    (暗号化されたテキスト、タグを連結したもの)

  • 対応する原文に、次の文章

    (ciphertext and tag, concatenated)

となっていました。一方、今回追加された関数は、原文の時点でこの文章はありませんでした。

追加分の原文も同じ仕様だが原文で漏れている、ということであれば、意訳としての補足の範囲内で、日本語版に追加しておくのが親切だと思います。仮にそうでない場合、または確証が持てない場合は、原文と同じように (暗号化...) の箇所は外しておいたほうが良いかもしれません。

ご確認頂けると助かります!

@Ippey
Copy link
Contributor Author

Ippey commented Nov 20, 2024

@KentarouTakeda ご確認ありがとうございます。修正しました!

Copy link
Collaborator

@KentarouTakeda KentarouTakeda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます!お手数おかけしました!

@KentarouTakeda KentarouTakeda merged commit 1bb4baf into php:master Nov 20, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants