PharmaLedger-IMI / epi-workspace

ePI use case main repository
MIT License
4 stars 0 forks source link

Ensure that /leaflets api calls include the headers from bdns config #1166

Open asaccool opened 1 year ago

asaccool commented 1 year ago

In a test session with MSD we identify that the headers from BDNS are used for the /gtinowner call but miss from the /leaflets

mpmerck commented 1 year ago

There has been also one more small issue that still prevails on the master branch, the empty headers check in the smart url, ./apihub-root/lightweight-pwa/js/utils/LightSmartUrl.js:


function getOptions(options){
    let opts = options || {headers:{}};
    if(!opts.headers){
        opts.headers = {};
    }
    if(url !== bdnsEntry && bdnsEntry.headers){
        Object.assign(opts.headers, bdnsEntry.headers);
    }
    return opts;
}```
salboaie commented 1 year ago

Issue migrated to JIRA: https://pharmaledger.atlassian.net/browse/EPI-192