Nested attributes with dynamic_jsonp_keys about puffing-billy HOT 7 CLOSED

That config option manipulates the GET query string for the cache.

I think the fix you're looking for would be in the cache key generation here: https://github.com/oesmith/puffing-billy/blob/master/lib/billy/cache.rb#L80.

Seems like you want to remove keys from the request body before using the body to generate the cache key.

Want to take a stab at adding a new config option for that?

from puffing-billy.

Actually, all of these parameters are in the URL. The gem uses Faraday for the requests, so I'm guessing putting the parameters in the URL is part of Faraday's implementation. Is this unusual? If so, is it too unusual to justify adding support for removing nested keys?

from puffing-billy.

@michaelrkn, whoa, Balanced Payments sends credit card details in the GET query string? That violates nearly every rule about when to use a GET. I haven't seen nested GET params in a very long time and have reservations about helping enable Balanced Payments' poor architecture if that's indeed the case.

from puffing-billy.

Because the balanced.js library uses JSONP, it creates an async <script> tag to send the information and execute the callback:

function jsonp(path, callback) {
    var funct = "balanced_jsonp_" + Math.random().toString().substr(2);
    var tag = document.createElement("script");
    tag.type = "text/javascript";
    tag.async = true;
    tag.src = path.replace("{callback}", funct);
    var where = document.getElementsByTagName("script")[0];
    where.parentNode.insertBefore(tag, where);
    window[funct] = function(result) {
        try {
            callback(result);
        } catch (e) {
            if (typeof console !== "undefined" && console.error) {
                console.error(e);
            }
        }
        tag.parentNode.removeChild(tag);
    };
}

So, yes, it is performing a GET request, but that's because it's using JSONP. Honestly, I didn't actually understand how JSONP until doing a bit of research just now, but this seems to be the standard way to make requests with JSONP.

from puffing-billy.

@michaelrkn, is this still an issue? I see that the related issue in your repo was closed in June.

from puffing-billy.

We switched from Balanced to Stripe and no longer need to ignore nested keys. I have no idea if Puffing Billy supports this now, but my feelings won't be hurt if you decide this isn't important and close the issue :)

from puffing-billy.

I'll close it for now until another Balanced user runs into the same issue.

from puffing-billy.