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.

Sign up for GitHub

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

Jump to bottom

Maximum call stack size exceeded with some settings #493

Closed
thib3113 opened this issue Mar 4, 2022 · 4 comments · May be fixed by snyk-omar/snyk-juice-shop#106, antz-snyk1/juice-shop-alexei#5, JonChiAppRisk/juice-shop-goof#7, slatersnyk/juice-shop#676 or fanibahar/juice-shop1#69
Closed

Maximum call stack size exceeded with some settings #493

thib3113 opened this issue Mar 4, 2022 · 4 comments · May be fixed by snyk-omar/snyk-juice-shop#106, antz-snyk1/juice-shop-alexei#5, JonChiAppRisk/juice-shop-goof#7, slatersnyk/juice-shop#676 or fanibahar/juice-shop1#69
Assignees
@mashpie
Labels
bug

Comments

@thib3113
Copy link

thib3113 commented Mar 4, 2022

I think I found a bug,
I'm using version 0.14.1 .

I'm using the options :

retryInDefaultLocale = true;
syncFiles: true,

So when I translate :

  • i18nTranslate => call translate
  • translate see (!skipSyncToAllFiles && syncFiles) === true => call syncToAllFiles
  • syncToAllFiles => call translate with skipSyncToAllFiles = true
  • translate go to here (same here) => call translate file without skipSyncToAllFiles
  • translate see (!skipSyncToAllFiles && syncFiles) === true => call syncToAllFiles (and then, restart the loop) .

Maybe passing syncToAllFiles here and here, can be a quick fix
@mashpie
Copy link
Owner

mashpie commented Mar 4, 2022

would you be able to setup a minimal reproduction repository?

@thib3113
Copy link
Author

thib3113 commented Mar 4, 2022 •
edited
Loading

@mashpie here is it : https://github.com/thib3113/i18n-493-issue .

But, it's mostly :

const { I18n } = require("i18n");
const path = require("path");

const i18n = new I18n({
    locales: ["en", "de"],
    directory: path.join(__dirname, "locales"),
    defaultLocale: "en",
    retryInDefaultLocale: true,
    syncFiles: true,
});

i18n.__("test");

Setting syncFiles and retryInDefaultLocale in the same times that produce the bug .

@mashpie mashpie self-assigned this Mar 4, 2022
@mashpie
Copy link
Owner

mashpie commented Mar 4, 2022

Thanks, will look into it over weekend probably

@mashpie mashpie added the bug label Mar 5, 2022
@mashpie mashpie closed this as completed in 1d956f3 Mar 5, 2022
@mashpie
Copy link
Owner

mashpie commented Mar 5, 2022

Thank you @thib3113 for reporting and analysing. Your guess for a fix was also perfectly right. I've just added tests to cover your issue.

A new version 0.14.2 should be released within minutes

@snyk-omar snyk-omar mentioned this issue Oct 22, 2022
Open
@antz-snyk antz-snyk mentioned this issue Sep 30, 2023
Open
@jonathanchisnyk jonathanchisnyk mentioned this issue Mar 29, 2024
Open
@slatersnyk slatersnyk mentioned this issue May 25, 2024
Open
@fanibahar fanibahar mentioned this issue May 25, 2024
Open
@marcos-test-dev marcos-test-dev mentioned this issue May 26, 2024
Open
@PRAVEENKUMAR6130 PRAVEENKUMAR6130 mentioned this issue May 26, 2024
Open
@worstyrite worstyrite mentioned this issue May 26, 2024
Open
@denk247 denk247 mentioned this issue May 31, 2024
Open
@yangpulse-snyk yangpulse-snyk mentioned this issue Jun 2, 2024
Open
@ZolotarenkoDan ZolotarenkoDan mentioned this issue Jun 7, 2024
Open
@kostatuno kostatuno mentioned this issue Jun 8, 2024
Closed
@cicdpipeline1 cicdpipeline1 mentioned this issue Jun 9, 2024
Open
@valentina-creator valentina-creator mentioned this issue Jun 9, 2024
Open
@harinee harinee mentioned this issue Jun 13, 2024
Closed
@arunkp arunkp mentioned this issue Jun 15, 2024
Open
@ShishiraKashyap ShishiraKashyap mentioned this issue Jun 15, 2024
Open
@bkiranraj bkiranraj mentioned this issue Jun 15, 2024
Open
@Suas1901 Suas1901 mentioned this issue Jun 15, 2024
Open
@AmoghGowda AmoghGowda mentioned this issue Jun 16, 2024
Open
@KrissWorks KrissWorks mentioned this issue Jun 16, 2024
Open
@vinaychandranac vinaychandranac mentioned this issue Jun 16, 2024
Open
@grimmgit grimmgit mentioned this issue Jun 21, 2024
Open
@boduk000 boduk000 mentioned this issue Aug 19, 2024
Closed
@Natalie-A Natalie-A mentioned this issue Aug 19, 2024
Open
@MVenell MVenell mentioned this issue Aug 22, 2024
Open
@srithreepo srithreepo mentioned this issue Aug 24, 2024
Open
@benjma benjma mentioned this issue Aug 24, 2024
Open
@PowrSlave PowrSlave mentioned this issue Aug 24, 2024
Open
@captainyarr captainyarr mentioned this issue Aug 25, 2024
Open
@felipemaragao felipemaragao mentioned this issue Aug 25, 2024
Open
@itsarraj itsarraj mentioned this issue Aug 26, 2024
Open
@boosef-snyk boosef-snyk mentioned this issue Aug 28, 2024
Open
@kiranvodnala kiranvodnala mentioned this issue Aug 28, 2024
Open
@elanonc elanonc mentioned this issue Aug 28, 2024
Open
@cybersec111 cybersec111 mentioned this issue Aug 30, 2024
Open
@ukmaker ukmaker mentioned this issue Aug 30, 2024
Open
@aleksander-acc aleksander-acc mentioned this issue Aug 30, 2024
Open
@naweedjaulim naweedjaulim mentioned this issue Aug 30, 2024
Open
@chuanl-accenture chuanl-accenture mentioned this issue Aug 30, 2024
Open
@mateuszsalata-acc mateuszsalata-acc mentioned this issue Aug 30, 2024
Open
@czhu314 czhu314 mentioned this issue Aug 30, 2024
Open
@prestonbateman prestonbateman mentioned this issue Aug 30, 2024
Open
@SindhuThammineni SindhuThammineni mentioned this issue Aug 31, 2024
Open
@anvitaakr anvitaakr mentioned this issue Aug 31, 2024
Open
@IT-MASTERMINDS IT-MASTERMINDS mentioned this issue Sep 4, 2024
Open
@M6xbom1 M6xbom1 mentioned this issue Sep 6, 2024
Open
@aravind-optiv aravind-optiv mentioned this issue Sep 7, 2024
Open
@ashly1111111 ashly1111111 mentioned this issue Sep 7, 2024
Open
@snykfrog snykfrog mentioned this issue Sep 9, 2024
Open
@kapilrk-04 kapilrk-04 mentioned this issue Sep 9, 2024
Open
@caishenwong caishenwong mentioned this issue Sep 13, 2024
Open
@rickhull67 rickhull67 mentioned this issue Sep 13, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mashpie mashpie

Labels
bug
Projects
None yet
Milestone
No milestone
2 participants
@mashpie @thib3113