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

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Lethgir
Copy link

@Lethgir Lethgir commented Jun 7, 2023

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
Copy link
Collaborator

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 :

  • Faire un rebase et corriger les conflits
  • Mettre à jour le README.md pour documenter la nouvelle option

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

Successfully merging this pull request may close these issues.

2 participants