]> zoso.dev Git - buffer.git/commitdiff
Fix base 64 character exclusion regex
authordaverayment <dave.rayment@gmail.com>
Tue, 4 Aug 2015 09:53:47 +0000 (10:53 +0100)
committerdaverayment <dave.rayment@gmail.com>
Tue, 4 Aug 2015 09:53:47 +0000 (10:53 +0100)
Using [A-z] in the regex brings in non-alphanumeric characters between the uppercase and lowercase alphabets, meaning the following chars were not correctly stripped from base 64 strings: left and right square bracket, backslash, caret, underline, and backtick. Explicitly separating the uppercase and lowercase letter ranges fixes this.

index.js

index 7b7b02c2d541d21e76a6c8129b1c7ab2d1b628a5..cf8a14c89e839fa972989423f19926eca8b9b2bc 100644 (file)
--- a/index.js
+++ b/index.js
@@ -1290,7 +1290,7 @@ Buffer._augment = function _augment (arr) {
   return arr
 }
 
-var INVALID_BASE64_RE = /[^+\/0-9A-z\-]/g
+var INVALID_BASE64_RE = /[^+\/0-9A-Za-z\-]/g
 
 function base64clean (str) {
   // Node strips out invalid characters like \n and \t from the string, base64-js does not