search

cnumr / GreenIT-Analysis-cli

Wrapper de l'extension GreenIT Analysis
GNU Affero General Public License v3.0
46 stars 19 forks source link

fix(): Erreur 'Target closed' dans une CI #57

Open Lethgir opened 1 year ago

Lethgir commented 1 year ago

J'aimerais proposer une PR pour qu'il soit possible d'ajouter des flags chrome à l'exécution des analyses. Cela me permet d'éviter des erreurs lorsque j'utilise GreenIT-Analysis-cli dans une CI.

Je n'ai pas créé d'issue Github avant mais il m'arrivait de rencontrer cette erreur : 

Error while analyzing URL [REDACTED] :  Error: Protocol error (Target.createTarget): Target closed.
    at /app/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:71:63
    at new Promise (<anonymous>)
    at Connection.send (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/Connection.js:70:16)
    at Browser._createPageInContext (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/Browser.js:221:53)
    at BrowserContext.newPage (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/Browser.js:500:30)
    at Browser.newPage (/app/node_modules/puppeteer/lib/cjs/puppeteer/common/Browser.js:214:37)
    at analyseURL (/app/cli-core/analysis.js:22:36)
    at createJsonReports (/app/cli-core/analysis.js:266:29)
    at analyse_core (/app/commands/analyse.js:77:25)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)

Pour la corriger, j'ajoute le flag --single-process

N'hésitez pas à me dire si la forme ne convient pas ou si vous avez besoin de plus d'information

jpreisner commented 10 months ago

Hello @Lethgir,

Merci pour ta proposition, et excuse-moi de revenir vers toi si tard. En phase pour l'accepter, est-ce que tu pourrais juste avant :