Skip to content

Conversation

@christohill
Copy link
Contributor

Changed the examples in the README to use "api_" instead of "consumer_". Using consumer_ returns an error from Checkfront about a malformed auth header.

Changed the examples in the README to use "api_*" instead of "consumer_*". Using consumer_ returns an error from Checkfront about a malformed auth header.
@k-jackson
Copy link
Contributor

Hi @christohill, can you give us more information about the error you're seeing?

Your proposed change might also require the addition of an 'auth_type' => 'token', parameter, as not supplying one will use oauth2 (requiring the use of the consumer_ prefix).

Specifically, I'm referring to the code following line 105 of the constructor in https://github.com/Checkfront/PHP-SDK/blob/master/lib/CheckfrontAPI.php

@christohill
Copy link
Contributor Author

@k-jackson That's fair, I didn't try the oauth2 authentication and when I read through the constructor I stopped at the token auth type. Maybe just adding a second example for the token auth method and labeling each?

Added a second example for users using the token auth type to distinguish the difference between api_* and consumer_* config values
@k-jackson k-jackson merged commit 514ae4b into Checkfront:master Oct 28, 2017
@christohill christohill deleted the patch-1 branch October 28, 2017 17:58
gegere added a commit to htmlgraphic/Checkfront-PHP-SDK that referenced this pull request Nov 27, 2017
gegere added a commit to htmlgraphic/Checkfront-PHP-SDK that referenced this pull request Oct 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants