import baseRequest from './baseRequest';
import sanitize from './sanitize';


const DEFAULT_REQUEST_CONFIG = {
    credentials: 'include',
    headers: {
        'Accept': 'application/json'
    }
};

/**
 * Small wrapper around js-utility-belt's request that provides url resolving, default settings, and
 * response handling.
 */
export default function request(url, config = {}, onlyJsonResponse=true) {
    // Load default fetch configuration and remove any falsy query parameters
    const requestConfig = Object.assign({}, DEFAULT_REQUEST_CONFIG, config, {
        query: config.query && sanitize(config.query)
    });
    let apiUrl = url;

    if (requestConfig.jsonBody) {
        requestConfig.headers = Object.assign({}, requestConfig.headers, {
            'Content-Type': 'application/json'
        });
    }
    if (!url) {
        return Promise.reject(new Error('Request was not given a url.'));
    }

    return baseRequest(apiUrl, requestConfig)
        .then((res) => {
            return onlyJsonResponse ? res.json() :
                {
                    json: res.json(),
                    url: res.url
                };
        })
        .catch((err) => {
            console.error(err);
            throw err;
        });
}