Skip to content

Conversation

Alberdi
Copy link
Contributor

@Alberdi Alberdi commented Sep 25, 2025

Some parameters, such as ContentType in httpc:request/4, are wrongly documented as uri_string:uri_string(), but they don't allow binary values. This was introduced when migrating to the new documentation in commit 6108008

For example:

1> httpc:request(post, {<<"http://www.example.com">>, [], <<"application/xhtml+xml">>, <<>>}, [], []).
{error,invalid_request}

2> httpc:request(post, {<<"http://www.example.com">>, [], "application/xhtml+xml", <<>>}, [], []).
{ok,{{"HTTP/1.1",403,"Forbidden"}, [...]

Some parameters, such as ContentType in httpc:request/4, are wrongly
documented as uri_string:uri_string(), but they don't allow binary
values.
Copy link
Contributor

github-actions bot commented Sep 25, 2025

CT Test Results

  2 files   22 suites   9m 8s ⏱️
360 tests 353 ✅  7 💤 0 ❌
647 runs  587 ✅ 60 💤 0 ❌

Results for commit 63ad1ad.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@IngelaAndin IngelaAndin added the team:PS Assigned to OTP team PS label Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team:PS Assigned to OTP team PS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants