Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Invalid
    • Affects Version/s: all
    • Fix Version/s: nonblocking
    • Component/s: unknown
    • Labels:
      None

      Description

      as per Ben Collins-Sussman suggestion...
      
      refer to my two posts on the user list:
      http://subversion.tigris.org/servlets/ReadMsg?list=users&msgNo=3097
      http://subversion.tigris.org/servlets/ReadMsg?list=users&msgNo=3127
      
      send an email directly to me if you need additional information.
      

      Original issue reported by basilwhitehouse

        Activity

        Hide
        brane Branko Čibej added a comment -

        Closing issues marked as "invalid"
        

        Show
        brane Branko Čibej added a comment - Closing issues marked as "invalid"
        Hide
        brane Branko Čibej added a comment -

        Closing issues marked as "invalid"
        

        Show
        brane Branko Čibej added a comment - Closing issues marked as "invalid"
        Hide
        kfogel Karl Fogel added a comment -

        FAQ item committed in r8386.
        

        Show
        kfogel Karl Fogel added a comment - FAQ item committed in r8386.
        Hide
        kfogel Karl Fogel added a comment -

        Note that http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25040 now has the
        "PatchAvailable" keyword.
        
        Closing this issue as INVALID.  Will add a FAQ item about digest auth.
        

        Show
        kfogel Karl Fogel added a comment - Note that http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25040 now has the "PatchAvailable" keyword. Closing this issue as INVALID. Will add a FAQ item about digest auth.
        Hide
        kfogel Karl Fogel added a comment -

        Well, the FIXED resolution is wrong.  Sigh; not going to bother to reopen just
        to close as INVALID or whatever, but I hope someone reads this comment if
        they're reading the issue at all :-).
        

        Show
        kfogel Karl Fogel added a comment - Well, the FIXED resolution is wrong. Sigh; not going to bother to reopen just to close as INVALID or whatever, but I hope someone reads this comment if they're reading the issue at all :-).
        Hide
        kfogel Karl Fogel added a comment -

        Apache HTTPD Server issue filed:
        
           http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25040
        
        I think the best thing to do is close this issue now... Closing.
        

        Show
        kfogel Karl Fogel added a comment - Apache HTTPD Server issue filed: http://nagoya.apache.org/bugzilla/show_bug.cgi?id=25040 I think the best thing to do is close this issue now... Closing.
        Hide
        kfogel Karl Fogel added a comment -

        The right place to fix this is in mod_auth_digest in httpd, as Justin Erenkrantz
        and others have pointed out.  We should close this issue when a corresponding
        issue is opened in the Apache issues database; in the meantime, moving to the
        "no milestone" milestone, as there is no code change to Subversion here.
        

        Show
        kfogel Karl Fogel added a comment - The right place to fix this is in mod_auth_digest in httpd, as Justin Erenkrantz and others have pointed out. We should close this issue when a corresponding issue is opened in the Apache issues database; in the meantime, moving to the "no milestone" milestone, as there is no code change to Subversion here.
        Hide
        kfogel Karl Fogel added a comment -

        Dang it, why didn't the reassignment take?  Assigning to me, again.
        

        Show
        kfogel Karl Fogel added a comment - Dang it, why didn't the reassignment take? Assigning to me, again.
        Hide
        kfogel Karl Fogel added a comment -

        Based on the above, Fitz and I are going to see if it's a quick hack to just
        make our mod_dav_svn authz_read() callback just handle digest auth specially
        right now.  The "right" solution, as Justin says, would be to fix
        mod_auth_digest, but we can't wait for another Apache release anyway, so it
        needs to work with the current  mod_auth_digest.
        
        If this turns out not to be a quick fix, then we'll just have to live without
        digest auth.  I know that sucks, but we can *not* afford to roll back the
        checkout/update performance improvements (which necessitated authz_read and its
        subrequests) just to support digest auth again.
        

        Show
        kfogel Karl Fogel added a comment - Based on the above, Fitz and I are going to see if it's a quick hack to just make our mod_dav_svn authz_read() callback just handle digest auth specially right now. The "right" solution, as Justin says, would be to fix mod_auth_digest, but we can't wait for another Apache release anyway, so it needs to work with the current mod_auth_digest. If this turns out not to be a quick fix, then we'll just have to live without digest auth. I know that sucks, but we can *not* afford to roll back the checkout/update performance improvements (which necessitated authz_read and its subrequests) just to support digest auth again.
        Hide
        kfogel Karl Fogel added a comment -

        See this thread, in which Ben Collins-Sussman describes the diagnosis and asks
        for help, followed by responses from Greg Hudson and Justin Erenkrantz:
        
        http://subversion.tigris.org/servlets/BrowseList?list=dev&by=thread&from=135712
        

        Show
        kfogel Karl Fogel added a comment - See this thread, in which Ben Collins-Sussman describes the diagnosis and asks for help, followed by responses from Greg Hudson and Justin Erenkrantz: http://subversion.tigris.org/servlets/BrowseList?list=dev&by=thread&from=135712
        Hide
        kfogel Karl Fogel added a comment -

        Ben's got this for 0.34.
        

        Show
        kfogel Karl Fogel added a comment - Ben's got this for 0.34.

          People

          • Assignee:
            kfogel Karl Fogel
            Reporter:
            subversion-importer Subversion Importer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development