Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upAdd ads api support [WIP] #687
Conversation
sferik
and others
added some commits
Nov 6, 2014
This comment has been minimized.
This comment has been minimized.
ianks
commented
Jul 24, 2015
|
@thomas-holmes do you want help with this? |
thomas-holmes
added some commits
Jul 30, 2015
This comment has been minimized.
This comment has been minimized.
thomas-holmes
commented
Aug 12, 2015
|
So I just added all of the stats endpoints though I have a few pending issues.
|
This comment has been minimized.
This comment has been minimized.
thomas-holmes
commented
Aug 13, 2015
|
So I've somewhat tracked down the issue: It seems that httprb/http versions >= 8.0 change something in the way query parameters are encoded which causes a mismatch during oauth signing and validation. The specific case I've tracked down is colons in date strings. If you supply a string with colons the signature gets generated (with colons being encoded as According to twitter's oauth documentation colons do not need to be encoded so perhaps the oauth library is incorrect? @sferik Any intuition as to who which library is "responsible" for this problem? |
This comment has been minimized.
This comment has been minimized.
thomas-holmes
commented
Aug 13, 2015
|
A simple interim fix seems be to just roll the http dependency back to |
This comment has been minimized.
This comment has been minimized.
I would open an issue with the It looks like |
thomas-holmes
referenced this pull request
Aug 13, 2015
Closed
Not percent encoding colons in query parameters #246
This comment has been minimized.
This comment has been minimized.
ixti
commented
Aug 14, 2015
|
Issue with escaping colon fixed in |
thomas-holmes
added some commits
Aug 18, 2015
sferik
force-pushed the
sferik:master
branch
11 times, most recently
from
0c18bf5
to
925baa2
Sep 2, 2015
This comment has been minimized.
This comment has been minimized.
|
@thomas-holmes Based on the discussion in #671, we've decided it makes more sense to point to Twitter's SDK. Undoubtedly your work here reflects a considerable amount of effort and while we aren't going to merge this in right now, I'd be remiss if I didn't thank you for opening this pull request and bringing this very close to the finish line. Thanks for contributing! |
thomas-holmes commentedApr 28, 2015
Accounts
Campaigns
Line Items
Promoted Accounts
Promotable Users
Promoted Tweets
Funding Instruments
Targeting Suggestions
Reach Estimate
Targeting Criteria
Tailored Audiences
Tweet!
Mobile App Cards
Mobile App (Image) Cards
Lead Gen Cards
Website Cards
App Events
Web Events
Scoped Timeline
Bidding Rules
Card Stats
Campaign Stats
Funding Instrument Stats
Line Item Stats
Promoted Account Stats
Promoted Tweets Stats
Account Stats