Skip to content

Commit

Permalink
UI: Bugfix. Move to a different TextEncoder/Decoder (#4613)
Browse files Browse the repository at this point in the history
1. The previously used TextEncoder/Decoder (used as a polyfill for
browsers that don't have a native version) didn't expose an encoder via
CommonJS. Use a different polyfill that exposes both a decoder and an
encoder.
2. The feature detection itself was flawed. This does a less error prone
detection that ensures native encoding/decoding where available and polyfilled
encoding/decoding where not available.
  • Loading branch information
johncowen authored Sep 12, 2018
1 parent f916962 commit e5f300d
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 11 deletions.
9 changes: 5 additions & 4 deletions ui-v2/app/utils/atob.js
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
import TextEncoderLite from 'npm:text-encoder-lite';
import TextEncoding from 'npm:text-encoding';
import base64js from 'npm:base64-js';
export default function(str, encoding = 'utf-8') {
// str = String(str).trim();
//decode
// decode
const bytes = base64js.toByteArray(str);
return new (TextDecoder || TextEncoderLite)(encoding).decode(bytes);
return new ('TextDecoder' in window ? TextDecoder : TextEncoding.TextDecoder)(encoding).decode(
bytes
);
}
4 changes: 2 additions & 2 deletions ui-v2/app/utils/btoa.js
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
import TextEncoderLite from 'npm:text-encoder-lite';
import TextEncoding from 'npm:text-encoding';
import base64js from 'npm:base64-js';
export default function(str, encoding = 'utf-8') {
// encode
const bytes = new (TextEncoder || TextEncoderLite)(encoding).encode(str);
const bytes = new ('TextEncoder' in window ? TextEncoder : TextEncoding.TextEncoder)(encoding).encode(str);
return base64js.fromByteArray(bytes);
}
2 changes: 1 addition & 1 deletion ui-v2/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -86,7 +86,7 @@
"loader.js": "^4.2.3",
"prettier": "^1.10.2",
"svgo": "^1.0.5",
"text-encoder-lite": "^1.0.1"
"text-encoding": "^0.6.4"
},
"engines": {
"node": "^4.5 || 6.* || >= 7.*"
Expand Down
4 changes: 0 additions & 4 deletions ui-v2/yarn.lock
Original file line number Diff line number Diff line change
Expand Up @@ -8933,10 +8933,6 @@ testem@^2.0.0:
tap-parser "^5.1.0"
xmldom "^0.1.19"

text-encoder-lite@^1.0.1:
version "1.0.1"
resolved "https://registry.yarnpkg.com/text-encoder-lite/-/text-encoder-lite-1.0.1.tgz#121c5ee74dfeb307037770ec75748c6824ac0518"

text-encoding@0.6.4, text-encoding@^0.6.4:
version "0.6.4"
resolved "https://registry.yarnpkg.com/text-encoding/-/text-encoding-0.6.4.tgz#e399a982257a276dae428bb92845cb71bdc26d19"
Expand Down

0 comments on commit e5f300d

Please sign in to comment.