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

doc-enの差分を反映(4) #142

Merged
merged 13 commits into from
Nov 10, 2024
Merged

Conversation

zonuexe
Copy link
Contributor

@zonuexe zonuexe commented Oct 31, 2024

</para>
<caution>
<simpara>
パスフレーズは、その名前から連想されるような安全な鍵生成機能は内包していません。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

若干の意訳。

https://github.com/php/doc-en/blob/e5ab2937efd9b1d7184993e0fdfa957893f7f827/reference/openssl/functions/openssl-decrypt.xml#L58-L60

直訳すると「名前が示唆するように、パスフレーズには鍵導出関数は使用されません。」

Copy link
Contributor Author

Choose a reason for hiding this comment

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

読者が暗号の専門家とは限らないということを意識して、「鍵導出関数」をそのまま使っていない (括弧書きで残してもいいが)

</para>
</listitem>
</varlistentry>
<varlistentry>
<term><parameter>output</parameter></term>
<listitem>
<para>
成功した場合、オープンしたデータをここに返します
成功した場合、オープンしたデータをこのパラメータのリファレンス変数として返します
Copy link
Contributor Author

Choose a reason for hiding this comment

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

原文

If the call is successful the opened data is returned in this parameter.

https://github.com/php/doc-en/blob/e5ab2937efd9b1d7184993e0fdfa957893f7f827/reference/openssl/functions/openssl-open.xml#L47-L48

「のリファレンス変数として」は原文にはない表現を補っている。

@@ -56,8 +56,11 @@
<term><parameter>iterations</parameter></term>
<listitem>
<para>
イテレーションの回数。
<link xlink:href="https://pages.nist.gov/800-63-3/sp800-63b.html#sec5">NIST は少なくとも10000を推奨しています</link>.
イテレーションの回数。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

「反復回数」と呼ぶべきかもしれないが、一旦は元の訳を踏襲。

@@ -56,8 +56,11 @@
<term><parameter>iterations</parameter></term>
<listitem>
<para>
イテレーションの回数。
<link xlink:href="https://pages.nist.gov/800-63-3/sp800-63b.html#sec5">NIST は少なくとも10000を推奨しています</link>.
反復回数。
Copy link
Contributor Author

Choose a reason for hiding this comment

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

#142 (review) で「イテレーションの回数」を維持しようか悩んだが、 #131 で変更されている中では「反復回数」と訳されていたので、この期に統一。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ほかに「イテレーション」がこの意味で使われている箇所は残っていなさそう

それぞれ、libzip ≥ 1.11.1 以降でビルドした場合に利用できます。
</simpara>
</listitem>
</varlistentry>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Girgias ビルドが失敗する理由がわかりますか?

https://github.com/php/doc-ja/actions/runs/11666252553/job/32480765523?pr=142

The document didn't validate
/home/runner/work/doc-ja/doc-ja/doc-base/.manual.xml:220111:82: error: element "constant" not allowed here; expected the element end-tag, text or element "abbrev", "acronym", "alt", "anchor", "annotation", "biblioref", "coref", "date", "emphasis", "firstterm", "footnote", "footnoteref", "foreignphrase", "glossterm", "indexterm", "inlinemediaobject", "link", "olink", "phrase", "quote", "remark", "replaceable", "subscript", "superscript", "trademark", "wordasword" or "xref"

php/doc-en@8afee82 と同じ構造が保たれているようです。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

rebaseしたらビルド通過していたので大丈夫そう

chapters/intro.xml Outdated Show resolved Hide resolved
chapters/intro.xml Outdated Show resolved Hide resolved
reference/openssl/functions/openssl-open.xml Outdated Show resolved Hide resolved
<caution>
<simpara>
デフォルト値 (<literal>'RC4'</literal>) はセキュアでない値です
PHP 8.0 以前のデフォルト値 (<literal>'RC4'</literal>) は安全ではありません
Copy link
Collaborator

Choose a reason for hiding this comment

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

IMO:

PHP 8.0 以前

以前 はそれ自体を含むニュアンスが強いため より前未満 と言った表現の方が正確かもしれません。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

「より前」にしましょう

@@ -76,7 +75,7 @@
暗号化方式
<caution>
<simpara>
デフォルト値 (<literal>'RC4'</literal>) はセキュアでない値です
PHP 8.0 以前のデフォルト値 (<literal>'RC4'</literal>) は安全ではありません
Copy link
Collaborator

Choose a reason for hiding this comment

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

IMO:

PHP 8.0 以前

以前 はそれ自体を含むニュアンスが強いため より前未満 と言った表現の方が正確かもしれません。

reference/openssl/functions/openssl-seal.xml Outdated Show resolved Hide resolved
reference/openssl/functions/openssl-seal.xml Outdated Show resolved Hide resolved
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 89b09cd into php:master Nov 10, 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