Scott Gibson

Thank you for your quick reply @jaredbeck ! I believe your recollection is exactly correct. And, in fact, OAuth generally uses a Header for the `Authorization: Bearer token`. From https://en.wikipedia.org/wiki/HTTPS:...

Overall, I realize I totally missed DRY the first time -- not a novice programmer, but new to submitting PRs to open source. Since this was security, I was a...

@tiegz @jaredbeck I agree that this should not be enabled by default. This makes perfect sense to me because my intent was not to make a change that would become...

> Perhaps we should only accept `single_access_token` over TLS connections. Now _that_ would be a breaking change 😁 💯

@tiegz I appreciate how thoroughly you're considering the security implications, since of course most security holes (aside from intentional trojan horse) are due to holes that are simply overlooked. Having...

@jaredbeck I had to fix specs, and also decided to explicitly assert the defaults for both `params_key` and `headers_key` (since they are different now).

> Looks really close to me, but I can see 2 blockers: > > * we need to access `controller.request.headers` instead of `controllers.headers` (which is for response headers) > *...

Hey @nicolaskruchten , I don't think it's possible to pass in custom ```aggregators``` -- [here is where the renderers can be passed in](https://github.com/plotly/react-pivottable/blob/master/src/PivotTable.jsx#L22)

It's possible that adding ```aggregators``` [here in this prop list](https://github.com/plotly/react-pivottable/blob/master/src/Utilities.js#L785) will all the defaults to be overridden. I may submit a PR if I can get it to work (fairly...

You are not alone, @mbhnyc -- and the workaround precludes showing the pagination in the return data: ``` expose @people.paginate(:page => params[:page]) ... "count": 2, "pagination": { "previous": null, "next":...