You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2017/07/29 14:17:54 UTC

[Bug 61358] New: Replace deprecated RandomStringUtils by commons-text RandomStringGenerator

https://bz.apache.org/bugzilla/show_bug.cgi?id=61358

            Bug ID: 61358
           Summary: Replace deprecated RandomStringUtils by  commons-text
                    RandomStringGenerator
           Product: JMeter
           Version: 3.2
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: p.mouawad@ubik-ingenierie.com
  Target Milestone: ---

RandomStringUtils is deprecated since commons-lang3-3.6

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 61358] Replace deprecated RandomStringUtils by commons-text RandomStringGenerator

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61358

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Hello,

Not sure it's a good idea for now.
My understanding is that it may incur additional objects creation for builders
particularly for RandomString from a set of chars.

Recoded as of commons-text-1.1, it would be something like:
final String lcharsToUse = charsToUse;
myValue = new RandomStringGenerator.Builder()
                    .filteredBy(c -> lcharsToUse.indexOf(c) >= 0)
                    .build().generate(length);

Version 1.2 has a selectFrom() 

But anyway, AFAIU this consumes more objects than RandomStringUtils.

Am I wrong ?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 61358] Replace deprecated RandomStringUtils by commons-text RandomStringGenerator

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61358

--- Comment #2 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Maybe wait for next versions of commons-text:

- http://markmail.org/message/azxw4nai7fs2laas

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 61358] Replace deprecated RandomStringUtils by commons-text RandomStringGenerator

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=61358

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Hardware|PC                          |All
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om
                 OS|Mac OS X 10.1               |All
           Severity|normal                      |enhancement

-- 
You are receiving this mail because:
You are the assignee for the bug.