Skip to content

Test curl_basic_024 fails with curl 8.8.0 #14307

Closed

Description

@rbgarga

Description

After building PHP 8.3.7 on Chimera Linux and run test suite there is one test failing:

ext/curl/tests/curl_basic_024.phpt

Here is the log:

  ---- EXPECTED OUTPUT
  bool(true)
  bool(true)
  string(4) "HTTP"
  ---- ACTUAL OUTPUT
  bool(true)
  bool(true)
  string(4) "http"
  ---- FAILED

This test was passing without problems with older curl versions.

PHP Version

PHP 8.3.7

Operating System

Chimera linux

Activity

  1. ndossche commented on May 23, 2024

    @ndossche
    Member

    I can reproduce this. The data comes straight out of curl so I think we should simply update the test to tolerate both forms.

  2. added a commit that references this issue on May 23, 2024
    ab2c698
  3. lucasnetau commented on May 24, 2024

    @lucasnetau
    Contributor

    There was a change done to Curl two weeks ago to lower case scheme

    lib: make protocol handlers store scheme name lowercase curl/curl@c294f9c

    CURLINFO_SCHEME documentation now notes

    The returned scheme might be upper or lowercase. Do comparisons case insensitively.

  4. ndossche commented on May 24, 2024

    @ndossche
    Member

    Thanks for the link @lucasnetau !
    I'll include it in the commit's description.

  5. added 2 commits that reference this issue on May 24, 2024
    a2c5b4e
    ae26dc1
  6. added a commit that references this issue on May 24, 2024
    9ad1779
  7. added 2 commits that reference this issue on Oct 23, 2024
    4f2eb92
    25c0993
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions