Skip to content
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.

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

Using deprecated googletag.encryptedSignalProviders. Please use googletag.secureSignalProviders instead. #10185

Closed
muuki88 opened this issue Jul 6, 2023 · 1 comment · Fixed by #10172

Comments

@muuki88
Copy link
Collaborator

muuki88 commented Jul 6, 2023

Type of issue

Deprecation of gpt.js API

Description

gpt.js reports an error that googletag.encryptedSignalProviders is deprecated and googletag.secureSignalProviders should be used instead.

window.googletag.encryptedSignalProviders = window.googletag.encryptedSignalProviders || [];
const encryptedSignalSources = config.getConfig('userSync.encryptedSignalSources');
if (encryptedSignalSources) {
const registerDelay = encryptedSignalSources.registerDelay || 0;
setTimeout(() => {
encryptedSignalSources['sources'] && encryptedSignalSources['sources'].forEach(({ source, encrypt, customFunc }) => {
source.forEach((src) => {
window.googletag.encryptedSignalProviders.push({
id: src,
collectorFunction: () => getEncryptedEidsForSource(src, encrypt, customFunc)
});
});
})
}, registerDelay)
} else {
logWarn(`${MODULE_NAME} - ESP : encryptedSignalSources config not defined under userSync Object`);
}
}

Steps to reproduce

  • enabled userId module
  • configured encrypted signal providers

Expected results

No deprecation warning

Actual results

Platform details

  • prebid 7.54.0

Other information

@muuki88
Copy link
Collaborator Author

muuki88 commented Jul 10, 2023

Thanks for the quick fix ❤️ I was unsure if just renaming it would be sufficient 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

1 participant