Skip to content

Debug logs #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
krcummings1 merged 3 commits into from
Apr 23, 2024
Merged

Debug logs #3

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3bee655
pointed user-agent to initialstate forked repo
krcummings1 Apr 22, 2024
2bf05e6
added logs for axios request
krcummings1 Apr 22, 2024
3a2c7fb
removed unicode characters
krcummings1 Apr 23, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 22 additions & 6 deletions index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ var formatPath = function (path, path_params) {


Mailchimp.prototype.post = function (options, body, done) {
options = _.clone(options) || {};
options = _.clone(options) || {};

if (_.isString(options)) {
options = {
Expand All @@ -83,7 +83,7 @@ Mailchimp.prototype.post = function (options, body, done) {
}

Mailchimp.prototype.patch = function (options, body, done) {
options = _.clone(options) || {};
options = _.clone(options) || {};

if (_.isString(options)) {
options = {
Expand All @@ -109,6 +109,8 @@ Mailchimp.prototype.patch = function (options, body, done) {
}

Mailchimp.prototype.request = function (options, done) {
console.log(JSON.stringify({ request_options: options }));

var mailchimp = this;
var promise = new Promise(function(resolve, reject) {
if (!options) {
Expand All @@ -118,17 +120,17 @@ Mailchimp.prototype.request = function (options, done) {

var path = formatPath(options.path, options.path_params);
var method = options.method || 'get';
var body = options.body || {};
var body = options.body || {};
var params = options.params;
var query = options.query;

var headers = {
'User-Agent' : 'mailchimp-api-v3 : https://github.com/thorning/node-mailchimp'
'User-Agent' : 'mailchimp-api-v3 : https://github.com/initialstate/node-mailchimp'
};

// Mailchimp does not respect on the language set in requests bodies for confirmation emails on new subscribers (and maybe other)
// A workaround is to make sure the language header matches
var language = options.language || body.language || null;
var language = options.language || body.language || null;
if (language) {
headers['Accept-Language'] = language;
}
Expand All @@ -145,6 +147,18 @@ Mailchimp.prototype.request = function (options, done) {
return;
}

let axios_req = {
method : method,
url : mailchimp.__base_url + path,
data : body,
params : query,
headers : headers,
responseType: 'json',
}

// omitted auth in log
console.log({ axios_req });

axios({
method : method,
url : mailchimp.__base_url + path,
Expand All @@ -160,13 +174,15 @@ Mailchimp.prototype.request = function (options, done) {

if (err) {
var error = new Error(err);
console.log('Error', error.response);
console.log('Mailchimp Error', error.response);
error.response = response;
error.status = response ? response.status : undefined;
reject(error)
return;
}

console.log('Mailchimp Response: ', response)

if (response.status < 200 || response.status > 299) {
var error = Object.assign(new Error(response.data ? response.data : response.status), response.data || response)
error.response = response;
Expand Down