search

okta / okta-auth-js

The official js wrapper around Okta's auth API
Other
437 stars 255 forks source link

doc: add sri to cdn script tag in readme #1498

Closed shuowu-okta closed 3 months ago

shuowu-okta commented 3 months ago

Compared files from CDN and registry against v7.4.1, both are generated same sri

script used

const fileFromCdn = fs.readFileSync(path.join(ARTIFACTS, 'okta-auth-js-7.4.1-fromCdn.min.js'), { encoding: 'utf8' });
const fileFromPkg = fs.readFileSync(path.join(ARTIFACTS, 'okta-auth-js-7.4.1-fromPkg.min.js'), { encoding: 'utf8' });
const sriFromCdn = generateSri(fileFromCdn);
const sriFromPkg = generateSri(fileFromPkg);
console.info('generated same sri?', sriFromCdn === sriFromPkg);

console output:

shuo.wu@F7HMVX6KDM okta-auth-js % node ./scripts/generate-sri.js
Debugger attached.
generated same sri? true

@lesterchoi-okta @jaredperreault-okta