Skip to content
This repository has been archived by the owner on Oct 16, 2018. It is now read-only.

Proper main setting #24

Closed
wants to merge 1 commit into from
Closed

Proper main setting #24

wants to merge 1 commit into from

Conversation

viktorbezdek
Copy link

Hey, I just added proper main linking so people don't have to shim the module.

"bundledDependencies": [],
"license": "MIT"
}
"main": "dist/gibberish-aes-1.0.0.js",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will break as soon as that version number goes up. May as well do the same thing as the component.json file and point to src/gibberish-aes.js instead. All the grunt task does is copy the source file and applies the version suffix to the file name anyway.

@mdp
Copy link
Owner

mdp commented Oct 16, 2018

Closing this without merging. This library was deprecated in 2014, and I'm not doing any updates to it anymore. In encryption terms, it's ancient and there are much better alternatives available out there (listed on the README). Of course, you're free to fork and update as you see fit, but please be aware that's it's using a very old AES mode. Thanks

@mdp mdp closed this Oct 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants