[PR] Fixes fate ranged lock encoding [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner opened a new pull request, #6350:
URL: https://github.com/apache/accumulo/pull/6350

   Encoding the fate ranged lock to store in zookeeper was using standard Base64 which could include a '/'. When this happened the code would go into an infinite loop.  Changed to using base64 URL encoding and added a few checks to make the fate lock encoding/decoding more strict.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fixes fate ranged lock encoding [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner merged PR #6350:
URL: https://github.com/apache/accumulo/pull/6350


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Fixes fate ranged lock encoding [accumulo]

["keith-turner (via GitHub)" <gi...@apache.org>]

keith-turner commented on PR #6350:
URL: https://github.com/apache/accumulo/pull/6350#issuecomment-4349234744

   Discovered this bug while testing #6351 and making splits with random bytes in them.  


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org