Age | Commit message (Collapse) | Author | |
---|---|---|---|
2008-07-08 | attachment: Support perl 5.8's buggy version of CGI.pm. | Joey Hess | |
This is truely horribly disgusting. CGI::tmpFileName, in current perls, is an undocumented function (which should be a clue..) that takes the original filename of an uploaded attachment, and returns the name of the tempfile that CGI has stored it in. In old perls, though, CGI::tmpFileName does not take a filename. It takes a key from the object's {'.tmpfiles'} hash. This key is something crazy like '*Fh::fh00001group' -- apparently the stringification of a filehandle object. Just to add to the fun, tmpFileName doesn't take the key, it expects a refernce to the key. Argh?! But the fun doesn't stop there, because in perl 5.8, CGI.pm is also broken in two other ways. The upload() method is supposed to return a filehandle to the temp file. It doesn't. The param() method is supposed to return a filehandle to the temp file, that stringifies to the original filename. It returns just the original filename, no filehandle. Combine all these bugs, and you end up with this disgusting commit. Since I have no way to get the filehandle, I *need* to get the tempfile name. If I had the filehandle, I could probably pass it into tmpFileName, and it might strigify to the right key name. But I don't, so the only way to determine the key is to grub through the .tmpfiles hash ourselves. And finally, one the temp file name is discovered, a filehandle can finally be obtained by (re)opening it. I recommend that this commit be reverted when perl 5.8 is a mercifully faded memory. I'm really, really, really glad I'm actually being paid for working on this right now! | |||
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Thanks! | Joey Hess | |
2008-07-08 | web commit by HenrikBrixAndersen: This happens on FreeBSD as well | Joey Hess | |
2008-07-08 | reproduced | Joey Hess | |
2008-07-08 | response | Joey Hess | |
2008-07-08 | rm | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * The bug still occurs | Joey Hess | |
2008-07-08 | add dist-bugs | Joey Hess | |
2008-07-08 | merge | Joey Hess | |
2008-07-08 | response | Joey Hess | |
2008-07-08 | web commit by mrled | Joey Hess | |
2008-07-08 | web commit by mrled | Joey Hess | |
2008-07-08 | web commit by mrled | Joey Hess | |
2008-07-08 | web commit by mrled | Joey Hess | |
2008-07-08 | web commit by mrled | Joey Hess | |
2008-07-08 | undef $fh before opening | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Still no success | Joey Hess | |
2008-07-08 | response | Joey Hess | |
2008-07-08 | another try at dealing with CGI.pm problem | Joey Hess | |
open the temp file if all attempts to get a fd fail | |||
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Debugging again | Joey Hess | |
2008-07-08 | possible fix | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Next debbuging | Joey Hess | |
2008-07-08 | new thing to try | Joey Hess | |
2008-07-08 | fix fix | Joey Hess | |
2008-07-08 | close | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Missing Apache2 version | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Response | Joey Hess | |
2008-07-08 | split out separate bug | Joey Hess | |
2008-07-08 | response | Joey Hess | |
2008-07-08 | attachment: Fix an uninitialised value warning when editing a page that ↵ | Joey Hess | |
currently has no attachments. | |||
2008-07-08 | Merge branch 'master' of ssh://git.ikiwiki.info/srv/git/ikiwiki.info | Joey Hess | |
2008-07-08 | web commit by http://jcflack.myopenid.com/ | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Problem with colon in report ↵ | Joey Hess | |
page name | |||
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Problem with uploading attachments | Joey Hess | |
2008-07-08 | web commit by http://ptecza.myopenid.com/: * Thanks! | Joey Hess | |
2008-07-08 | add an interesting ikiwiki site I stumbled across | Joey Hess | |
2008-07-07 | test | Joey Hess | |
2008-07-07 | response | Joey Hess | |
2008-07-07 | move to todo | Joey Hess | |
2008-07-07 | clarify how an item is wishlisted | Joey Hess | |
2008-07-07 | undo wishlist changes | Joey Hess | |
You don't want to post new wishlist items, really, as they're lower priority than just todo items. :-) Anyway, I want to keep wishlist and todos both under todo/ so that they can easily be changed from one state to another. And the inline caused items to be created under wishlist/ , but didn't actually inline them. | |||
2008-07-07 | web commit by http://dmarti.myopenid.com/ | Joey Hess | |
2008-07-07 | web commit by http://www.openidfrance.fr/beaufils | Joey Hess | |
2008-07-07 | web commit by http://dmarti.myopenid.com/ | Joey Hess | |
2008-07-07 | resp | Joey Hess | |
2008-07-07 | web commit by http://jcflack.myopenid.com/ | Joey Hess | |
2008-07-07 | testing | Joey Hess | |
2008-07-07 | example of how to limit to specific directory | Joey Hess | |
2008-07-07 | new idea | Joey Hess | |
2008-07-07 | typo the second | Joey Hess | |