#twisted

/

      • glyph
        oops sorry I mean, "gotLength"
      • _moshez
        ok. so I'm assuming that in hCC, I'm doing some sort of incremental MIME parsing and putting the file into the target file?
      • glyph
        sorry this interface is not super well documented, I always have to read HTTPChannel to remember it, and then I forget parts of it are actually in HTTPChannel
      • _moshez: right so that's the second problem, Request.requestReceived calls 'cgi.parse_multipart'
      • _moshez
        glyph: sounds like that's the part where I steal/borrow bits from django to stream-parse mime?
      • lazybear joined the channel
      • ok I'm not sure we got into real regrets territory yet, so I'm guessing that's coming up?
      • glyph
        _moshez: right.
      • if you think feed-parsing MIME is easy and fun then great, you're done
      • so I doubt you're going to regret this
      • moshez joined the channel
      • also worth mentioning, however, is https://github.com/twisted/klein/pull/181
      • oh hrm
      • maybe you should just work with wsanchez to make that One Hundred Per Cent For Real
      • it's not far
      • review https://github.com/twisted/klein/pull/199 and it's like halfway there
      • wsanchez
      • But 199 is a subset, me trying to get someone to actually approve something
      • runciter
        does #181 require twisted depend on tubes?
      • is that going to be a blocker?
      • wsanchez
        181 requires tubes
      • glyph
        runciter: it requires klein depend on tubes :)
      • runciter: I doubt it's going to be much of a blocker
      • wsanchez
        But it's in Klein right now, not Twisted
      • glyph
        runciter: there's a point where we need to do a switcheroo
      • runciter
        glyph: the streaming part is a lie without giving t.w.http.Request a streaming body :(
      • wsanchez
        It's starting by defining the API
      • The implementation underneath can then catch up
      • glyph
        once the API exists, it will be pretty easy to adjust the implementation in the way I just said
      • runciter
        wsanchez: yeah, you've done the hardest part :)
      • Wulf has quit
      • glyph
        you just need to create an explicit opt-in API
      • wsanchez
        runciter: No the hardest part is getting it reviewed ;-)
      • runciter
        i'm still interested in making a streaming version of t.w.http.Request - that's why i ask
      • wsanchez
        (To be fair it's been reviewed by a few folks.)
      • I want to kill t.w.http.Request (and t.w.server.Request) with a knife while it sleeps
      • Instantiating one of those requires a Ph.D. in jank
      • runciter
        no complaints here
      • _moshez
        ok cool, I'll try to find some time to look at these things and see if I can help push them forward
      • on a personal note, it's so weird for me to have a conversation about http on #twisted :)
      • I still remember the days where it was all "#twisted-web is that way"
      • I'm an old I guess.
      • hopefully that will be useful for WSGI too, and then we can put warehouse behind twisted WSGI :)
      • simpson
        I remember those days.
      • glyph
        we just need everyone involved in this conversation (HI SIMPSON) to review one ticket a week :)
      • moshez has quit
      • simpson
        glyph: See, this is why I never remember things. It just causes me to remember how much stuff is on my plate.
      • _moshez
        ah thursday.
      • glyph
        _moshez: yeah, twisted.web is gradually getting deprecated :)
      • sorry
      • #twisted.web
      • I mean
      • and twisted-web, the mailing list
      • foom
        You realize that a _literal_decade_ago_, we were all talking about this same damn thing.
      • _moshez
        foom: twisted prevails
      • foom
        Or was it a decade and a half.
      • Does it? It still can't stream-parse file uploads. :)
      • simpson
        foom: Further evidence that it's a hard problem, I guess.
      • foom
        Well, there's tubes now, so that's progress.
      • glyph
        foom: wsanchez is surprisingly close to done on this PR, even
      • foom
        "New stream stuff. *THIS* one is good for *sure*."
      • (spoilers: nope)
      • simpson
        foom: Today I realized that I can *reuse* a stream library for a thing. My elation was nearly palpable.
      • kenaan
        Tickets pending review: https://tm.tl/#8159, #761, #8966 (the0id), #9100, #9101, #9118 (the0id), #4964 (jameshilliard), #9131, #9138, #9143
      • worrelsik1 joined the channel
      • worrelsik has quit
      • rthille-ciena1 joined the channel
      • clokep joined the channel
      • EACFreddy has quit