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

v0.5.1 #510

Merged
merged 30 commits into from
Feb 20, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
b1caefc
Add override true when sending reset data to devtools.
amovar18 Feb 8, 2024
e178d30
Remove unlimited storage permission.
amovar18 Feb 8, 2024
fe29631
fix:add UA string in CLI
ayushnirwal Feb 8, 2024
e196b1d
Merge pull request #487 from GoogleChromeLabs/fix/UA-string-mocking
mohdsayed Feb 9, 2024
9f97548
Merge pull request #484 from GoogleChromeLabs/fix/slow-loading-cookie…
mohdsayed Feb 9, 2024
b955cea
Merge pull request #491 from GoogleChromeLabs/fix/launcher-dev-tool-d…
gagan0123 Feb 14, 2024
1f93d3a
Temp: Add logs for debugging
mohdsayed Feb 15, 2024
88761db
Fix: add conditional rendering in technology details panel (#499)
ayushnirwal Feb 16, 2024
c0ba3b5
Add one more log
mohdsayed Feb 16, 2024
84d62ac
Merge branch 'develop' of github.com:GoogleChromeLabs/ps-analysis-too…
mohdsayed Feb 16, 2024
f7757ad
Fix: Library detection miscellaneous QA issues (#495)
mohdsayed Feb 19, 2024
f7c4774
Refactor: Restructure `Facilitated Testing` landing page UI (#493)
mayan-000 Feb 19, 2024
4fea486
Two-step process for toggling CDP state (#494)
amovar18 Feb 19, 2024
dd1b286
Enhancement: Update cookie's blocking status enum and UI (#507)
mayan-000 Feb 19, 2024
789a40a
Fix popup ui issue
mohdsayed Feb 20, 2024
7794a22
Chore: Remove logs used for debugging empty cookies
mohdsayed Feb 20, 2024
bd1624b
Remove relative property. (#511)
amovar18 Feb 20, 2024
9fad900
Fix: keep service worker alive (#513)
amovar18 Feb 20, 2024
0ad08a3
Add UTM params to outgoing links (#512)
mohdsayed Feb 20, 2024
ed0823d
style-fix: scale up prefix icons in table (#514)
ayushnirwal Feb 20, 2024
595f1a7
Update changelog
mohdsayed Feb 20, 2024
89378d6
Update cookie database
mohdsayed Feb 20, 2024
7c74c6a
Update rws data
mohdsayed Feb 20, 2024
35af0b4
Bump version
mohdsayed Feb 20, 2024
3847489
Run audit fix
mohdsayed Feb 20, 2024
27242be
fix: add flags to not highlight rows
mayan-000 Feb 20, 2024
809bc38
Add <br/> for uniformity.
amovar18 Feb 20, 2024
59622d0
fix: fix details panel showing unknown reason message
ayushnirwal Feb 20, 2024
4e510b2
Fix the reload button staying on the popup.
amovar18 Feb 20, 2024
bc7ce7d
Merge branch 'develop' of github.com:GoogleChromeLabs/ps-analysis-too…
amovar18 Feb 20, 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
20 changes: 20 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -240,3 +240,23 @@
- Update d.c.c. links https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/417
- Merge `main` into `develop` https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/451
- Update flags and profile color in Chrome launcher https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/467

# v0.5.0

## Extension
* Restructure `Facilitated Testing` landing page UI https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/493
* Two-step process for toggling CDP state https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/494
* Enhancement: Update cookie's blocking status enum and UI https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/507
* Enhancement: Add UTM params to outgoing links https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/512
* Fix: UI and change messaging in toast message https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/511
* Fix: Library detection miscellaneous QA issues https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/495
* Fix: Keep service worker alive https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/513
* Fix: Slow loading cookies on page change https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/484
* Style-Fix: Scale up cookie prefix icon and update message text in cookie details. https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/514

## CLI
* Fix: Add mock UA string https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/487
* Fix: Add conditional rendering in technology details panel https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/499

## Others
* Disable auto opening of Chrome DevTools by default for all tabs https://github.com/GoogleChromeLabs/ps-analysis-tool/pull/491
2 changes: 1 addition & 1 deletion bin/chrome-3pcd-ps.bat
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
:: Chrome 3pcd with PS Extension

:: Download PS Extension
set "ps_analysis_tool_version=v0.5.0"
set "ps_analysis_tool_version=v0.5.1"
cd /d %TEMP%
if not exist %TEMP%\ps-analysis-tool-%ps_analysis_tool_version% (
mkdir %TEMP%\ps-analysis-tool-%ps_analysis_tool_version%
Expand Down
2 changes: 1 addition & 1 deletion bin/chrome-default-ps.bat
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
:: Default Chrome with PS Extension

:: Download PS Extension
set "ps_analysis_tool_version=v0.5.0"
set "ps_analysis_tool_version=v0.5.1"
cd /d %TEMP%
if not exist %TEMP%\ps-analysis-tool-%ps_analysis_tool_version% (
mkdir %TEMP%\ps-analysis-tool-%ps_analysis_tool_version%
Expand Down
2 changes: 1 addition & 1 deletion bin/chrome_launcher.sh
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

# Download Extension
extension_setup() {
ps_analysis_tool_version=v0.5.0
ps_analysis_tool_version=v0.5.1
extension_dir="/var/tmp"
cd $extension_dir
if [ ! -d $extension_dir/ps-analysis-tool-$ps_analysis_tool_version ]; then
Expand Down
30 changes: 24 additions & 6 deletions data/related_website_sets.json
Original file line number Diff line number Diff line change
Expand Up @@ -328,15 +328,21 @@
}
},
{
"contact": "andri.nurdiansah@kgmedia.id",
"primary": "https://kgmedia.id",
"contact": "cs@kgmedia.id",
"primary": "https://kompas.com",
"associatedSites": [
"https://kompas.com",
"https://kompasiana.com"
"https://tribunnews.com",
"https://grid.id",
"https://bolasport.com",
"https://kompasiana.com",
"https://kompas.tv"
],
"rationaleBySite": {
"https://kompas.com": "KG Media - General News provider",
"https://kompasiana.com": "KG Media - Online blog and publication platform"
"https://tribunnews.com": "Indonesian local news by KG Media",
"https://grid.id": "Indonesian entertainment & lifestyle news by KG Media ",
"https://bolasport.com": "Indonesian sport news by KG Media ",
"https://kompasiana.com": "Indonesian largest blog community by KG Media ",
"https://kompas.tv": "Indonesian national news by KG Media "
}
},
{
Expand Down Expand Up @@ -499,6 +505,18 @@
"https://rws2nvtvt.com": "Test 1",
"https://rws3nvtvt.com": "Test 2"
}
},
{
"contact": "admin@stripe.com",
"primary": "https://stripe.com",
"serviceSites": [
"https://stripecdn.com",
"https://stripe.network"
],
"rationaleBySite": {
"https://stripecdn.com": "Serves static assets accelerating page loads and sandboxes select Javascript",
"https://stripe.network": "Receives telemetry preventing fraud across Stripe services"
}
}
]
}
47 changes: 21 additions & 26 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "ps-analysis-tool",
"version": "0.5.0",
"version": "0.5.1",
"description": "Cookie Analysis Tool and CLI for analysis and understanding of cookie usage on web pages.",
"scripts": {
"cli:prebuild": "node ./scripts/delete-build-artifacts.cjs",
Expand Down
2 changes: 1 addition & 1 deletion packages/cli-dashboard/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@ps-analysis-tool/cli-dashboard",
"version": "0.5.0",
"version": "0.5.1",
"description": "Dashboard for visualizing cli analysis output",
"repository": {
"type": "git",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -59,6 +59,7 @@ const AffectedCookies = ({ cookies, selectedSite }: AffectedCookiesProps) => {
>
<CookieTable
data={cookies.map((cookie) => ({ ...cookie, isBlocked: undefined }))} // Hot Fix: To unhighlight cookies in the Affected Cookie table.
useIsBlockedToHighlight={true} // Hot Fix: To use isBlocked to highlight cookies in the Affected Cookie table.
tableColumns={tableColumns}
showTopBar={true}
tableFilters={filters}
Expand All @@ -70,7 +71,10 @@ const AffectedCookies = ({ cookies, selectedSite }: AffectedCookiesProps) => {
setSelectedFrameCookie={setSelectedFrameCookie}
/>
</Resizable>
<CookieDetails selectedFrameCookie={selectedFrameCookie} />
<CookieDetails
selectedFrameCookie={selectedFrameCookie}
isUsingCDP={false}
/>
</div>
);
};
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,7 @@ const CookiesListing = ({
/>
</Resizable>
<CookieDetails
isUsingCDP={true}
isUsingCDP={false}
selectedFrameCookie={selectedFrameCookie}
/>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -120,6 +120,7 @@ const Technologies = ({ selectedSite }: TechnologiesProps) => {
className="h-full flex"
>
<Table
useIsBlockedToHighlight={true} // Hot Fix: To fail the condition intentionally and not highlight the rows.
table={table}
hideFiltering={true}
showTopBar={true}
Expand All @@ -146,16 +147,14 @@ const Technologies = ({ selectedSite }: TechnologiesProps) => {
</p>
</>
)}
{selectedRow.name && (
<>
<p className="font-bold text-granite-gray dark:text-manatee mb-1">
Description
</p>
<p className="text-outer-space-crayola dark:text-bright-gray">
{selectedRow.description}
</p>
</>
)}
<>
<p className="font-bold text-granite-gray dark:text-manatee mb-1">
Description
</p>
<p className="text-outer-space-crayola dark:text-bright-gray">
{selectedRow?.description || 'No description available.'}
</p>
</>
</div>
) : (
<div className="h-full p-8 flex items-center">
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@ps-analysis-tool/cli",
"version": "0.5.0",
"version": "0.5.1",
"description": "CLI tool for analysis",
"main": "index.js",
"scripts": {
Expand Down
2 changes: 1 addition & 1 deletion packages/cli/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@
const program = new Command();

program
.version('0.5.0')
.version('0.5.1')
.description('CLI to test a URL for 3p cookies')
.option('-u, --url <value>', 'URL of a site')
.option('-s, --sitemap-url <value>', 'URL of a sitemap')
Expand Down Expand Up @@ -194,7 +194,7 @@
text: 'Done analyzing cookies.',
});

let technologyAnalysisData: any = null;

Check warning on line 197 in packages/cli/src/index.ts

View workflow job for this annotation

GitHub Actions / Lint

Unexpected any. Specify a different type

if (!shouldSkipTechnologyAnalysis) {
spinnies.add('technology-spinner', {
Expand Down
5 changes: 5 additions & 0 deletions packages/cli/src/utils/browserManagement/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@
this.shouldLogDebug = shouldLogDebug;
}

debugLog(msg: any) {

Check warning on line 57 in packages/cli/src/utils/browserManagement/index.ts

View workflow job for this annotation

GitHub Actions / Lint

Unexpected any. Specify a different type
if (this.shouldLogDebug) {
console.log(msg);
}
Expand All @@ -80,6 +80,11 @@
throw new Error('Browser not intialized');
}
const sitePage = await this.browser.newPage();

await sitePage.setUserAgent(
'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/121.0.0.0 Safari/537.36'
);

sitePage.setViewport({
width: 1440,
height: 790,
Expand Down Expand Up @@ -235,7 +240,7 @@
}
const frameIdMapFromTree = new Map();
const frames = page.frames();
const frameCallback = (frame: any) => {

Check warning on line 243 in packages/cli/src/utils/browserManagement/index.ts

View workflow job for this annotation

GitHub Actions / Lint

Unexpected any. Specify a different type
const frameId = frame._id;
const _url = frame.url();
frameIdMapFromTree.set(frameId, _url);
Expand Down
2 changes: 1 addition & 1 deletion packages/common/package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "@ps-analysis-tool/common",
"version": "0.5.0",
"version": "0.5.1",
"description": "A package for common utilities that are being used in multiple packages",
"main": "./dist/index.js",
"types": "./dist-types/index.d.ts",
Expand Down
Loading
Loading