Document Org.Apache.Http.Client.Protocol

[jonathanpeppers]

Summary

• Replaced all 55 existing To be added. entries under Org.Apache.Http.Client.Protocol with API-specific documentation.
• Documented HTTP context keys, client context configuration, authentication interceptors, and JNI peer metadata.

Closes#265

Sources

• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/ClientContext
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/ClientContextConfigurer
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/RequestAddCookies
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/RequestDefaultHeaders
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/RequestProxyAuthentication
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/RequestTargetAuthentication
• Android reference: https://developer.android.com/reference/org/apache/http/client/protocol/ResponseProcessCookies
• AOSP source: https://android.googlesource.com/platform/external/apache-http/+/android-6.0.1_r1/src/org/apache/http/client/protocol/

Validation

• Parsed every docs/xml/Org.Apache.Http.Client.Protocol/*.xml file as XML.
• Confirmed zero To be added. placeholders in the scoped XML files.
• Confirmed CRLF line endings and ran git diff --check.

[Copilot]

Pull request overview

This PR replaces placeholder XML documentation (“To be added.”) across Org.Apache.Http.Client.Protocol with API-specific docs covering HTTP context keys, context configuration helpers, request/response interceptors, and JNI peer metadata entries.

Changes:

• Added namespace/type/member summaries and remarks for the Org.Apache.Http.Client.Protocol XML docs (removing placeholders).
• Documented HTTP context attribute keys (IClientContext and ClientContextConfigurer.InterfaceConsts) and context configuration (ClientContextConfigurer).
• Documented interceptor types and their purpose (e.g., target/proxy authentication interceptors) and filled in JNI-related member docs.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
docs/xml/Org.Apache.Http.Client.Protocol/ResponseProcessCookies.xml	Replaces placeholder docs; updates JNI peer member documentation.
docs/xml/Org.Apache.Http.Client.Protocol/RequestTargetAuthentication.xml	Adds interceptor summary; updates JNI peer member documentation.
docs/xml/Org.Apache.Http.Client.Protocol/RequestProxyAuthentication.xml	Adds interceptor summary; updates JNI peer member documentation.
docs/xml/Org.Apache.Http.Client.Protocol/RequestDefaultHeaders.xml	Updates JNI peer member documentation.
docs/xml/Org.Apache.Http.Client.Protocol/RequestAddCookies.xml	Updates JNI peer member documentation.
docs/xml/Org.Apache.Http.Client.Protocol/IClientContext.xml	Documents context attribute constants and adds obsolescence/remar ks content.
docs/xml/Org.Apache.Http.Client.Protocol/ClientContextConfigurer+InterfaceConsts.xml	Documents inherited context attribute names (consts).
docs/xml/Org.Apache.Http.Client.Protocol/ClientContextConfigurer.xml	Documents context configurer type and members; updates JNI peer member documentation.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[jonathanpeppers]

@dalexsoto review

[jonathanpeppers]

@dalexsoto review

[dalexsoto]

Two protocol documentation gaps remain:

• ClientContextConfigurer has a nullable managed constructor parameter, but AOSP rejects null with Java.Lang.IllegalArgumentException; document the non-null requirement and exception.
• The public compatibility ClientContext type still has blank summaries for all ten constants while equivalent IClientContext and InterfaceConsts members are documented.

Please add the constructor contract and keep the compatibility projection in parity.

[jonathanpeppers]

Two protocol documentation gaps remain:

Addressed in 9ea4350: the configurer constructor now documents its non-null contract and Java.Lang.IllegalArgumentException, and all ten ClientContext compatibility constants now match the documented projections.

[dalexsoto]

The configurer null contract and all ClientContext compatibility constants now match the Apache implementation and managed projections.