OldTruckDriver opened a new pull request, #436:
URL: https://github.com/apache/commons-codec/pull/436

   **Merge after [CODEC-343]**
   
   ## Summary
   
   Fix `Base32.Builder#setEncodeTable(byte...)` so custom Base32 alphabets 
derive a matching decode table and can decode their own output.
   
   ## Details
   
   Previously, custom encode tables still used the standard Base32 decode table 
unless they exactly matched the built-in Base32-Hex table. This caused codecs 
with custom alphabets to encode correctly but decode incorrectly.
   
   This change derives a decode table from custom 32-byte alphabets, rejects 
invalid or duplicate alphabet entries, and keeps the built-in Base32/Base32-Hex 
tables using their case-insensitive decoders.
   
   **Depends on CODEC-343, please merge  CODEC-343 first.**
   
   ## Tests
   
   - `mvn -q -Dtest=org.apache.commons.codec.binary.Base32Test test`
   - `mvn -q javadoc:javadoc`
   - `mvn -q`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to