#musicbrainz-devel

/

      • reosarevok
        (in music)
      • 2011-11-25 32938, 2011

      • reosarevok
        (and to my plans to have a centralised spot to get CC-licensed Spanish hip hop, too :P )
      • 2011-11-25 32928, 2011

      • warp
        reosarevok: they did show interest in archiving and making available more music (all music, not just cc licensed).
      • 2011-11-25 32949, 2011

      • reosarevok
        Making available not cc-licensed music might be an issue though?
      • 2011-11-25 32953, 2011

      • reosarevok
        (archiving it is fine I guess)
      • 2011-11-25 32937, 2011

      • warp
        there are many issues :)
      • 2011-11-25 32954, 2011

      • reosarevok
        Sure, but I meant more like "an invitation for everyone to sue at the same time"
      • 2011-11-25 32958, 2011

      • reosarevok
        =
      • 2011-11-25 32902, 2011

      • reosarevok
        *)
      • 2011-11-25 32942, 2011

      • reosarevok would love to use MB as a tool for that though, like we will do for the cover art
      • 2011-11-25 32951, 2011

      • reosarevok
        I think it makes a lot of sense too
      • 2011-11-25 32917, 2011

      • ocharles
        ijabz: got a moment to talk about paypal ipn?
      • 2011-11-25 32941, 2011

      • ijabz
        oh yeah okay
      • 2011-11-25 32908, 2011

      • ocharles
        ijabz: ok, one mo, let me find the code I was working on
      • 2011-11-25 32926, 2011

      • ocharles
        ijabz: are you doing it manually or using a library for it?
      • 2011-11-25 32951, 2011

      • ijabz
        manually
      • 2011-11-25 32959, 2011

      • ocharles
        https://gist.github.com/aa85d843f558f52d642a from what I understand is what you're meant to do - keep the parameters in order, append cmd=_notify_validate, and resend the request
      • 2011-11-25 32912, 2011

      • ocharles
        it may well just be that it doesn't work against the sandbox or something
      • 2011-11-25 32940, 2011

      • ijabz
        So this is the asynchronous IPN stuff
      • 2011-11-25 32907, 2011

      • ijabz
        Did you get the PDT stuff working, or are you only doing the IPN
      • 2011-11-25 32921, 2011

      • ocharles
        Only IPN I guess, I'm just trying to use the existing scripts
      • 2011-11-25 32943, 2011

      • ocharles
        I haven't heard of PDT
      • 2011-11-25 32941, 2011

      • ijabz
        to summarise, if making a payment from web send something to paypal, paypal sends something straight back to your request saying if payment appears ok, which you can verify
      • 2011-11-25 32912, 2011

      • ijabz
        Then if you have IPN setup payola will asychnrously send something to your server when payment has actually gone through
      • 2011-11-25 32944, 2011

      • ijabz
        So sometimes people only do the easier PDT , and not the IPN which seems the opposite to what you have
      • 2011-11-25 32955, 2011

      • ocharles
        hmm
      • 2011-11-25 32902, 2011

      • ocharles
        well, it's ipn for now, so I think I should probably stick with it
      • 2011-11-25 32906, 2011

      • ocharles
        rob pretty much just wants a direct port
      • 2011-11-25 32920, 2011

      • ijabz
        In my IPN code I use GET not POST , could that be the problem
      • 2011-11-25 32930, 2011

      • ocharles
        ah, interesting
      • 2011-11-25 32931, 2011

      • ocharles
        maybe that's it
      • 2011-11-25 32917, 2011

      • ijabz
        ah, actually my comment maybe wrong, I think Im using Post
      • 2011-11-25 32930, 2011

      • ijabz
        Ill paste a code extract
      • 2011-11-25 32953, 2011

      • ocharles
        that'd be great
      • 2011-11-25 32950, 2011

      • ijabz
        take me a ferw minutes to clean it up
      • 2011-11-25 32951, 2011

      • ijabz
      • 2011-11-25 32916, 2011

      • ijabz
        Java Im afraid, this was done quite sometime ago, so not aufait with it
      • 2011-11-25 32949, 2011

      • ijabz
        Note the problem I had with charsets, maybe that is your issue
      • 2011-11-25 32908, 2011

      • ocharles
        np, I can read Java fine
      • 2011-11-25 32922, 2011

      • ocharles
        certainly seems to match what I'm doing
      • 2011-11-25 32955, 2011

      • ijabz
        so what happens when you send back to Paypal
      • 2011-11-25 32918, 2011

      • ocharles
        I was just getting a HTML document back
      • 2011-11-25 32925, 2011

      • ocharles
        but I've just made a proper paypal account now for myself
      • 2011-11-25 32933, 2011

      • ocharles
        and I'm resending an IPN, so we'll see what happens
      • 2011-11-25 32921, 2011

      • ijabz
        You are sending an IPN in response to receiving a request from Paypal aren't you ?
      • 2011-11-25 32929, 2011

      • ocharles
        I make the payment through a form that submits to paypal, and I get confirmation and all that. Then I see an IPN message gets sent from the paypal control panel, which I'm meant to reply to
      • 2011-11-25 32941, 2011

      • ocharles
        but that ipn isn't even getting to me, think that might be because the server is on port 3000, not 80
      • 2011-11-25 32944, 2011

      • ocharles
        just changing that now and trying again
      • 2011-11-25 32922, 2011

      • ocharles
        I just get a 301 response back when I try and verify
      • 2011-11-25 32923, 2011

      • ocharles
        hmm
      • 2011-11-25 32938, 2011

      • ocharles
        It's really annoying that Catalyst doesn't let you get to the raw request body
      • 2011-11-25 32905, 2011

      • ijabz
        So you're receiving it now
      • 2011-11-25 32928, 2011

      • ocharles
        I get something back, but verification fails
      • 2011-11-25 32947, 2011

      • ijabz
        can u print out what parameters you get from the paypal request
      • 2011-11-25 32936, 2011

      • ocharles
        response*
      • 2011-11-25 32919, 2011

      • ocharles
        _content and _uri are the bits that are interesting in the request. the response is just a 301 :(
      • 2011-11-25 32944, 2011

      • ocharles
        I guess I could try and follow that redirect
      • 2011-11-25 32916, 2011

      • ocharles
        mmm, if I go to that I just end up at a page saying "You have requested an outdated version of PayPal. This error often results from the use of bookmarks."
      • 2011-11-25 32937, 2011

      • warp
        fun
      • 2011-11-25 32958, 2011

      • ocharles
        I've been putting this stuff off for 6 days now, figured I'd get it done :P
      • 2011-11-25 32919, 2011

      • reosarevok
        :D
      • 2011-11-25 32934, 2011

      • ijabz
        Sorry, don't really follow the output, not very web literate
      • 2011-11-25 32955, 2011

      • reosarevok answers to a BBC editor who wrote a polite and useful mail to support
      • 2011-11-25 32909, 2011

      • ijabz
        Is the 1st bit the request you get fro Paypal, or the request you send
      • 2011-11-25 32902, 2011

      • ocharles
        that's the request I send
      • 2011-11-25 32909, 2011

      • ocharles
        in response to an IPN request
      • 2011-11-25 32931, 2011

      • ijabz
        I want to see the IPN request
      • 2011-11-25 32921, 2011

      • ocharles
        https://gist.github.com/f8a8380b32bce4674a0e that's a copy and paste of what's in the IPN console on the paypal site
      • 2011-11-25 32932, 2011

      • ocharles
        bleh, meant to [off] that, oh well :P
      • 2011-11-25 32951, 2011

      • ijabz
        are you actually responding to the IPN request as you receive it, or just creating a new request based on that information
      • 2011-11-25 32909, 2011

      • ocharles
        not sure I follow
      • 2011-11-25 32917, 2011

      • ocharles
        I create a new request and copy the body into it, and add cmd=_notify_validate
      • 2011-11-25 32933, 2011

      • ijabz
        I may have this wrong but it doesn't sound like you actually have a server running which paypal is connecting to, am I right ?
      • 2011-11-25 32959, 2011

      • ocharles
        I do, paypal are sending me the request, I respond to it, and then they respond with a 301
      • 2011-11-25 32911, 2011

      • ocharles
        and they should be responding with "success" or whatever
      • 2011-11-25 32942, 2011

      • ijabz
        Right and you are constructing your response based on the parameters you actually receive, not just hardcoding the values in that last paste
      • 2011-11-25 32958, 2011

      • ocharles
        right
      • 2011-11-25 32906, 2011

      • ocharles
        i'll just keep tweaking it until something seems to work
      • 2011-11-25 32916, 2011

      • ocharles
        I seem to understand what I'm meant to be doing, obviously just missing something simple
      • 2011-11-25 32940, 2011

      • ocharles
        I think the problem is that catalyst doesn't give me the raw unprocessed request body, so I have to transform it back into just a string, and that's going wrong
      • 2011-11-25 32943, 2011

      • ijabz
        it could be the iso8859, utf problem I encountered
      • 2011-11-25 32955, 2011

      • ocharles
        think I could have something that's working now
      • 2011-11-25 32907, 2011

      • ijabz
        ooh
      • 2011-11-25 32932, 2011

      • ocharles
        I rewrote how I construct the body string (again), and now sending the ipn request manually to myself with curl seems to get further
      • 2011-11-25 32940, 2011

      • ocharles
        (but crashes later, but that's my fault)
      • 2011-11-25 32951, 2011

      • ocharles
        so once the crashes are fixed, I'll donate another $0.50 to myself :P
      • 2011-11-25 32906, 2011

      • ocharles
        ok, cool. The only thing is I have "payment pending" atm
      • 2011-11-25 32913, 2011

      • ocharles
        when you receive payments, do you have to manually accept them?
      • 2011-11-25 32922, 2011

      • ocharles
        yep, ipn is working
      • 2011-11-25 32925, 2011

      • ocharles
        thank fuck for that. what a mess
      • 2011-11-25 32927, 2011

      • ijabz
        no i don't, maybe a setting
      • 2011-11-25 32951, 2011

      • ijabz
        heh, yeah Google Checkout is much nicer
      • 2011-11-25 32919, 2011

      • ijabz
        (now Google Wallet)
      • 2011-11-25 32944, 2011

      • ocharles
        yea, I think we might support that soon
      • 2011-11-25 32936, 2011

      • djce joined the channel
      • 2011-11-25 32925, 2011

      • voiceinsideyou joined the channel
      • 2011-11-25 32904, 2011

      • kepstin-netbook joined the channel
      • 2011-11-25 32945, 2011

      • warp
        ocharles: just fixed another important breakage (regression) in the release editor. (MBS-3813)
      • 2011-11-25 32958, 2011

      • ocharles
        yay
      • 2011-11-25 32925, 2011

      • warp
        so I'm almost ready for a hotfix release. until what time are you available for that?
      • 2011-11-25 32943, 2011

      • ocharles
        another 2 hours at least
      • 2011-11-25 32903, 2011

      • warp
        ah, then we should get started on it soon :)
      • 2011-11-25 32907, 2011

      • warp
        I'm still fixing the selenium tests. (i.e. I fixed them all in the IDE, but some are still broken when I run them with prove).
      • 2011-11-25 32929, 2011

      • warp
        but I suggest we shouldn't wait for that.
      • 2011-11-25 32927, 2011

      • kepstin-laptop joined the channel
      • 2011-11-25 32933, 2011

      • ocharles
        i'd rather wait for that :)
      • 2011-11-25 32941, 2011

      • ocharles
        give yourself another hour, see how it looks then
      • 2011-11-25 32950, 2011

      • warp
        ok
      • 2011-11-25 32959, 2011

      • ijabz joined the channel
      • 2011-11-25 32936, 2011

      • ijabz joined the channel
      • 2011-11-25 32919, 2011

      • ocharles
      • 2011-11-25 32944, 2011

      • warp
        FFFUUUUUUUU
      • 2011-11-25 32955, 2011

      • ocharles
        umad?
      • 2011-11-25 32959, 2011

      • warp
        these tests really are more trouble than they are worth sometimes.
      • 2011-11-25 32904, 2011

      • warp
        13:04 warp [at 18-volt]:~/code/mb$ cat selenium.log | grep ^not
      • 2011-11-25 32905, 2011

      • warp
        not ok 5 - t/selenium/bugfixes/mbs-1204-edit-associations.html
      • 2011-11-25 32912, 2011

      • warp
        ocharles: I do not wish to work on this any further today.
      • 2011-11-25 32921, 2011

      • ocharles
        warp: ok
      • 2011-11-25 32938, 2011

      • warp logs into carl.
      • 2011-11-25 32900, 2011

      • ocharles
        wait, have we got production setup?
      • 2011-11-25 32906, 2011

      • ocharles
        have you hotfixed into that?
      • 2011-11-25 32919, 2011

      • warp
        no, I just merged into master.
      • 2011-11-25 32946, 2011

      • ocharles
        so what are we hotfixing? if you give me the commit shas of the merges, I will cherry pick them into production
      • 2011-11-25 32958, 2011

      • warp
        ok, let's see
      • 2011-11-25 32914, 2011

      • warp
        8da8cdbe58d7ea4e86c409695892f6f48491ab6a remove placeholder next
      • 2011-11-25 32946, 2011

      • warp
        f8779a6cf3cf61d89a44aa312ea1f1c2daee7483 broken guess case
      • 2011-11-25 32917, 2011

      • warp
        bf159f5709b10f840eb1526f8f2eee37d8dfd26d broken seeded param
      • 2011-11-25 32926, 2011

      • warp
        that's it I think.
      • 2011-11-25 32930, 2011

      • warp
        from me anyway.
      • 2011-11-25 32947, 2011

      • reosarevok
        Is the seeded param for the include work-level-rels thing?
      • 2011-11-25 32950, 2011

      • reosarevok
        Or something else?
      • 2011-11-25 32957, 2011

      • warp
        no, it's MBS-3813
      • 2011-11-25 32905, 2011

      • reosarevok
        (also, didn't you finish the hide/show credits while you're at it? 8) )
      • 2011-11-25 32916, 2011

      • warp
        reosarevok: no, didn't finish.
      • 2011-11-25 32925, 2011

      • ocharles
        warp: oh, just commits, no branches?
      • 2011-11-25 32929, 2011

      • ocharles
        easy peasy
      • 2011-11-25 32938, 2011

      • reosarevok
        Aww
      • 2011-11-25 32944, 2011

      • reosarevok
        Well, still, hotfix!
      • 2011-11-25 32913, 2011

      • warp
        ocharles: f8779a6cf3cf61d89a44aa312ea1f1c2daee7483 was in a branch, I gave you the hash of the original commit, no the merge commit.
      • 2011-11-25 32937, 2011

      • ocharles
        ok
      • 2011-11-25 32943, 2011

      • ocharles
        you can cherry-pick merges
      • 2011-11-25 32950, 2011

      • ocharles
        but if it's just a single commit that needs picking, I'll do that
      • 2011-11-25 32956, 2011

      • ocharles
        ready to update
      • 2011-11-25 32902, 2011

      • warp
        yeah, a single commit in that branch.
      • 2011-11-25 32906, 2011

      • warp
        alright
      • 2011-11-25 32955, 2011

      • warp
        ocharles: I have taken astro out.
      • 2011-11-25 32909, 2011

      • ocharles pokes his vpn
      • 2011-11-25 32932, 2011

      • ocharles
        warp: bring astro back in, my vpn is playingup
      • 2011-11-25 32925, 2011

      • warp
        astro back.
      • 2011-11-25 32942, 2011

      • ocharles
        typical, now it's all working