Different class names are but Same obfuscated name

[hoangnhan2ka3]

Version: v2.2.2 released

As the title:

I tried it twice so I think this is not a 1% random error case (cuz I don't know if this plugin supports making sure the class names won't overlap)

The .xjun0e class (aka .pr-[1.41dvw]) should be in <main> tag ✅:

Below, right where .side-box-line should be, there should be another obfuscated class name. But it has the same name as .pr-[1.41dvw] ⛔:

Note: .side-box-line is just a className, no direct styles were applied to it, it only gets styles through class like this [&_.side-box-line]:left-0 and this class is also not in <main> (I explain like this to avoid misunderstanding)

That causes .side-box-line, which should look like this ✅:

to look like this ⛔:

The green color is the padding-right effect:

[hoangnhan2ka3]

recording-2024-02-06-125200_WaX9GAdY.mp4

This feature doesn't seem really stable yet, although I see this as a feature that would be really great 🤩.

This is the seed I used in the video:

module.exports = {
   ...
    generatorSeed: 848178188989999,
   ...
}

[soranoo]

A new issue was created.
soranoo/recoverable-random#1

[hoangnhan2ka3]

Hi @soranoo, just because I'm too lazy to create a new issue, I'm commenting on the old bug here.

I think it's very rare that this happens, but is there any way to be sure it doesn't overlap anymore?

Package version 2.2.6

I just hope this patch won't change the class name again, because it seems like you'll be editing the recovery-random package.

My next-css-obfuscator.config.cjs:

module.exports = {
    enable: true, // Enable or disable the plugin.
    mode: "random", // Obfuscate mode, "random" or "simplify".
    buildFolderPath: ".next", // Build folder of your project.
    classConversionJsonFolderPath: "./css-obfuscator", // The folder path to store the before obfuscate and after obfuscated classes conversion table.
    refreshClassConversionJson: false, // Refresh the class conversion JSON file.

    classLength: 6, // Length of the obfuscated class name.
    classPrefix: "n", // Prefix of the obfuscated class name.
    classSuffix: "", // Suffix of the obfuscated class name.

    classIgnore: [
        "dark",
        "light",
        "no-transition",
        "opa-hidden",
        "big",
        "small",
        /__.*/,
        /os-scrollbar*/,
    ], // The class names to be ignored during obfuscation.

    allowExtensions: [".jsx", ".tsx", ".js", ".ts", ".html", ".rsc"], // The file extensions to be processed.

    contentIgnoreRegexes: [/\.jsxs\)\("\w+"/g], // The regexes to match the file content to be ignored during obfuscation.

    whiteListedFolderPaths: [], // Only obfuscate files in these folders

    blackListedFolderPaths: ["./.next/cache"], // Don't obfuscate files in these folders

    enableMarkers: false, // Enable or disable the obfuscate marker classes.
    markers: ["obf"], // Classes that indicate component(s) need to obfuscate.
    removeMarkersAfterObfuscated: true, // Remove the obfuscation markers from HTML elements after obfuscation.

    removeOriginalCss: true, // Delete original CSS from CSS files if it has a obfuscated version.

    generatorSeed: 84817818898, // The seed for the random generator.

    //! Experimental feature
    enableJsAst: false, // Whether to obfuscate JS files using abstract syntax tree parser (Experimental feature)

    logLevel: "info", // Log level
};

[soranoo]

It is not necessary to comment on both issues. It is redundant.

Any issue with random generation may related to the underlay random generation algo if a new algo is introduced the rng behaviour will be changed and a new class name will be generated.

If you want to keep the old class name for some reason, you can simply modify the JSON manually.

[hoangnhan2ka3]

Added ClassName Duplication Check & New Random

Sounds cool!

Have you had time to push it to npm yet, or is there still work to be done?

I still think it's a good idea that you can have it check if there is a duplicate class in conversion.json and replace it with another random string.

However, I still wonder if I'm that unlucky, in

$36^6 = 2,176,782,336$

possible cases, I have 2 overlapping classes... or is it called luck?

[soranoo]

I found I forgot to pull it Sorry about that.

Do you mean the collision before the update?

[hoangnhan2ka3]

Yes, in v2.2.6

[hoangnhan2ka3]

🤔

[soranoo]

You have to use string seed.

[hoangnhan2ka3]

omg when did I remove the " from generatorSeed😂, sorry for the brainless question.

[soranoo]

If it is no dulipcation report after 24hrs I will close this issue.

But plz feel free to comment here if it happened again.

[hoangnhan2ka3]

It seems like this bug needs to wait until someone "lucky" encounters the same situation as I mentioned above to be sure it works haha

[hoangnhan2ka3]

Bruhh, I understand how you feel when you comment on the postcss-obfuscator project and get no response, so much so that you had to create this project.

No need to wait anymore, I'm the "lucky" one 🤣

[soranoo]

Yes, exactly. So I try my best to give response in any format to let the issue raiser know "someone" is care about it.