Opened 14 years ago
Closed 14 years ago
#7400 closed defect (wontfix)
[patch][no-cla] Fix base64 trailing padding removal
Reported by: | rgl | Owned by: | Tom Trenka |
---|---|---|---|
Priority: | high | Milestone: | tbd |
Component: | Dojox | Version: | |
Keywords: | Cc: | ||
Blocked By: | Blocking: |
Description
dojox trunk base64 code is wrongly removing all NUL characters from the end of decoded data. this ticket included the unit tests that demonstrate the problem and a respective patch.
Attachments (2)
Change History (4)
Changed 14 years ago by
Attachment: | base64-tests.patch added |
---|
comment:1 Changed 14 years ago by
Owner: | changed from Adam Peller to Tom Trenka |
---|---|
Summary: | Fix base64 trailing padding removal → [patch][no-cla] Fix base64 trailing padding removal |
comment:2 Changed 14 years ago by
Resolution: | → wontfix |
---|---|
Status: | new → closed |
The null byte removal was deliberate and from my understanding, both part of the spec as well as needed for all of the crypto code...so I'm not even going to look at this patch (since it's no CLA) and close this as "won't fix".
Appreciate the theoretical exactness but unless I see a real use case (and after 3 years, I haven't)...
Note: See
TracTickets for help on using
tickets.
unit tests