Skip to content

Conversation

@nsfisis
Copy link
Contributor

@nsfisis nsfisis commented May 2, 2025

refs #150

概要

php/doc-en#3818 他で追加・編集されている reference/curl/constants_curl_setopt.xml を新規に翻訳しました。

@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch 2 times, most recently from 5fcb828 to 49441a2 Compare May 17, 2025 16:02
@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch from 1631275 to 0853e3a Compare June 24, 2025 11:46
@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch from 601cf16 to 9e83525 Compare July 16, 2025 17:15
mumumu added a commit that referenced this pull request Sep 13, 2025
- #295 以外は curl の定数は完了。
@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch from 1621056 to 31f2f25 Compare October 9, 2025 17:01
@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch from 31f2f25 to 252d0cb Compare October 12, 2025 06:39
@nsfisis nsfisis marked this pull request as ready for review October 12, 2025 06:46
@nsfisis nsfisis changed the title WIP reference/curl/constants_curl_setopt.xml を新規翻訳 [PHP 8.4] reference/curl/constants_curl_setopt.xml を新規翻訳 Oct 12, 2025
@nsfisis
Copy link
Contributor Author

nsfisis commented Oct 12, 2025

終わりました!!!Ready for review です!

<para>
接続するまでの最大待機ミリ秒数を指定します。
<literal>0</literal> を指定するといつまでも待機します。
cURL がシステム標準の名前解決器を使うようにビルドされている場合、
Copy link
Member

Choose a reason for hiding this comment

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

「名前解決器」は、DNS の文脈では リゾルバ と訳すべきだと思います。

https://ja.wikipedia.org/wiki/%E3%83%AA%E3%82%BE%E3%83%AB%E3%83%90

Copy link
Contributor Author

Choose a reason for hiding this comment

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

修正しました。

</term>
<listitem>
<para>
&true; を指定すると、必要なプロキシの認証をおこなって接続を確立するだけで、
Copy link
Member

Choose a reason for hiding this comment

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

&true; を指定すると、必要なプロキシの認証をおこなって接続を確立するだけで、

本件PRで、「行って」と「おこなって」がブレている箇所が複数見られます。

Copy link
Contributor Author

Choose a reason for hiding this comment

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

修正しました。

<member>
<literal>RELOAD</literal>
は <constant>CURLOPT_COOKIEFILE</constant> で指定されたファイルからすべてのクッキーを読み込みます
</member>
Copy link
Member

Choose a reason for hiding this comment

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

<member></member> 内の文末は合わせたほうが良いと思います。

Suggested change
</member>
<simplelist type="inline">
<member>
<literal>ALL</literal>
はメモリ上に保持されているすべてのクッキーを削除します
</member>
<member>
<literal>SESS</literal>
はメモリ上に保持されているすべてのセッションクッキーを削除します
</member>
<member>
<literal>FLUSH</literal>
は <constant>CURLOPT_COOKIEJAR</constant> で指定されたファイルにすべてのクッキーを書き出します
</member>
<member>
<literal>RELOAD</literal>
は <constant>CURLOPT_COOKIEFILE</constant> で指定されたファイルからすべてのクッキーを読み込みます
</member>

Copy link
Contributor Author

@nsfisis nsfisis Oct 13, 2025

Choose a reason for hiding this comment

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

原文の表記に合わせた意図的なものです。原文は一文で記載されており、訳文も一文で記載しています。

ALL erases all cookies held in memory , SESS erases all session cookies held in memory , FLUSH writes all known cookies to the file specified by CURLOPT_COOKIEJAR , RELOAD loads all cookies from the files specified by CURLOPT_COOKIEFILE .

読点は原文から意図せず脱落していたため追加しました。

</term>
<listitem>
<para>
HTTPS プロキシ用のピン留めされた (ピンニングされた) 公開鍵を指定します。
Copy link
Member

Choose a reason for hiding this comment

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

Set the pinned public key for HTTPS proxy.

[imo] 原文は上記です。括弧の注意書きは不要だと感じました。

Suggested change
HTTPS プロキシ用のピン留めされた (ピンニングされた) 公開鍵を指定します
HTTPS プロキシ用のピン留めされた公開鍵を指定します

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"Pinning certificate" 等における "pin" の訳は 「ピンニング」と「ピン留め」で揺れており、ネット上の多くの文書でもこのような括弧書きの形で併記されています。どちらの方が一般的とも言い難い状況であると判断し併記しました。

@nsfisis nsfisis force-pushed the feature/constants_curl_setopt branch from e3f1ba1 to bf47e32 Compare October 13, 2025 12:22
Co-authored-by: Yoshinari Takaoka <mumumu@mumumu.org>
@nsfisis
Copy link
Contributor Author

nsfisis commented Oct 13, 2025

https://github.com/php/doc-ja/compare/e3f1ba124277da86efd0d168a937ce23e804d412..bf47e32b8d3fad177cfcb0d49dd91eb616b2b26a のときの force-push で一度適用した suggestion を上書きしてしまい、続くコミットで適用した suggestion を手で再現し直しました。ただ、対応する suggestion コメントが失われてしまったようです。申し訳ありません。

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