Menu ▾ ▴

#1291 test 906 in testsuite fails for curl 7.33.0

closed-fixed
Steve Holme
None
5
2014-08-16
2013-10-15
David Walser
No

"make check" fails for curl 7.33.0 on test 906 with error 500 Authentication Failed, as can be seen in this build log:
http://pkgsubmit.mageia.org/uploads/failure/cauldron/core/release/20131015011351.luigiwalser.valstar.7293/log/curl-7.33.0-1.mga4/build.0.20131015011403.log

Discussion

  • Steve Holme

    Steve Holme - 2013-10-15

    Test 906 is known to fail on some platforms (such with Windows SSPI) as it requires NTLM and both the Native routines and the Windows SSPI API will generate slightly different authentication strings.

    However, you appear to be compiling under Linux and from what I know it should generate the same strings. Do you know if the other NTLM tests are failing or are configure to not run due to being disabled in some way?

    Their are elements of the Test Suite that I'm not fully aware so I may have missed something in adding this test.

     

  • Daniel Stenberg

    Daniel Stenberg - 2013-10-15

    It doesn't fail for me on (64bit) Linux...

     

  • Steve Holme

    Steve Holme - 2013-10-15

    It appears that the end of the string is slightly different:

    "cmFiYml0" is being generated rather than "Y3VybGhvc3Q="

    If I base64 decode the two strings then host name appears to be "curlhost" in the successful test but seems to be "rabbit" in the unsuccessful one.

    That would seem to indicate that the CURL_GETHOSTNAME override isn't working properly if I have understood that correctly?

     

  • David Walser

    David Walser - 2013-10-15

    I just did a grep and it looked like there were a couple other tests with NTLM in them that do run and pass, but most of them (there were several) are not run.

     

  • Steve Holme

    Steve Holme - 2013-10-15

    I think the tests that run fall back to other authentication mechanisms like digest and such like.

    In order for the host name override to work curl needs to be a debug build and I don't think your build is:

    curl lacks debug support" 37 times (1210, 1310, 1311, 1312, 1313, 1334, 1335, 1336, 1337 and 28 more)

    And other output - I guess we need to disable this test if it isn't a debug build.

     

  • Steve Holme

    Steve Holme - 2013-10-15

    and the LD_PRELOAD hack fails so it's then picking up your box's real hostname.

     

  • Steve Holme

    Steve Holme - 2013-10-15

    I think I need the following in test 906:

    <precheck>
    chkhostname curlhost
    </precheck>

    Daniel - Do you know if this is correct? If so I'll push a fix.

    Note: Sourceforge removed the precheck tags from around that :(

     

    Last edit: Steve Holme 2013-10-15

  • Dan Fandrich

    Dan Fandrich - 2013-10-15

    I just came to the same conclusion--the precheck section is missing. Go ahead and add it!

     

  • Steve Holme

    Steve Holme - 2013-10-15

    Thanks for the confirmation Dan, and thanks Daniel and David in helping with this.

    I have pushed commit 4d49ffe165a51d and would appreciate confirmation that it fixes the problem.

     

  • David Walser

    David Walser - 2013-10-15

    Can I get a link to the commit so I can try it?

     

  • Steve Holme

    Steve Holme - 2013-10-15

    Thank you for reporting the problem again David and also confirming the fix.

    Sorry I missed this in the first place!

     

  • Steve Holme

    Steve Holme - 2013-10-15
    • status: open --> open-confirmed
    • assigned_to: Steve Holme
     

  • Steve Holme

    Steve Holme - 2013-10-15
    • status: open-confirmed --> closed-fixed
     
MongoDB Logo MongoDB