Re: About SSL properties in JMeter and their use in Java / HC Impl and Recorder

2021-11-14 Thread Vladimir Sitnikov
I see, +1 for unifying the properties. I don't quite like the naming of the fields: String CIPHER_SUITE_LIST becomes String[] SUPPORTED_CIPHER_LIST after split(" "). The names are not that important as the fields are private (which is good), however, it is really hard to follow which property

[GitHub] [jmeter] codecov-commenter edited a comment on pull request #660: bz65108: escape/unescape jmeter variable references

2021-11-14 Thread GitBox
codecov-commenter edited a comment on pull request #660: URL: https://github.com/apache/jmeter/pull/660#issuecomment-968444522 #

[GitHub] [jmeter] codecov-commenter edited a comment on pull request #660: bz65108: escape/unescape jmeter variable references

2021-11-14 Thread GitBox
codecov-commenter edited a comment on pull request #660: URL: https://github.com/apache/jmeter/pull/660#issuecomment-968444522 #

[GitHub] [jmeter] vlsi commented on a change in pull request #677: Align ssl props / Enable setting cipher suite (WIP)

2021-11-14 Thread GitBox
vlsi commented on a change in pull request #677: URL: https://github.com/apache/jmeter/pull/677#discussion_r748907085 ## File path: src/protocol/http/src/main/java/org/apache/jmeter/protocol/http/proxy/Proxy.java ## @@ -89,6 +89,19 @@ private static final String

[GitHub] [jmeter] vlsi commented on a change in pull request #677: Align ssl props / Enable setting cipher suite (WIP)

2021-11-14 Thread GitBox
vlsi commented on a change in pull request #677: URL: https://github.com/apache/jmeter/pull/677#discussion_r748906691 ## File path: src/protocol/http/src/main/java/org/apache/jmeter/protocol/http/proxy/Proxy.java ## @@ -89,6 +89,19 @@ private static final String

Re: About SSL properties in JMeter and their use in Java / HC Impl and Recorder

2021-11-14 Thread Philippe Mouawad
Hello, https://github.com/apache/jmeter/pull/677 Regards On Sun, Nov 14, 2021 at 11:13 AM Vladimir Sitnikov < sitnikov.vladi...@gmail.com> wrote: > Hi, > > I do not completely understand the use case, however, I support adding the > common property names > so different implementations could be

[GitHub] [jmeter] pmouawad opened a new pull request #677: Align ssl props / Enable setting cipher suite (WIP)

2021-11-14 Thread GitBox
pmouawad opened a new pull request #677: URL: https://github.com/apache/jmeter/pull/677 ## Description - Use https.cipherSuites property for HC4 - Use https.cipherSuites and https.socket.protocols in Proxy ## Motivation and Context Clarification of use. Allow fine

[GitHub] [jmeter] codecov-commenter edited a comment on pull request #674: WIP: Open Model Thread Group

2021-11-14 Thread GitBox
codecov-commenter edited a comment on pull request #674: URL: https://github.com/apache/jmeter/pull/674#issuecomment-968265204 #

Re: Questions following switch from Eclipse to IntelliJ

2021-11-14 Thread Vladimir Sitnikov
>What do you mean ? https://www.jetbrains.com/help/idea/project-security.html#open_first_time >Maybe you can share this configuration in the document. I think we could even share the configuration via .idea/... file (see

Re: Questions following switch from Eclipse to IntelliJ

2021-11-14 Thread Philippe Mouawad
Thanks for rapid answer. My notes inline below. On Sun, Nov 14, 2021 at 7:09 PM Vladimir Sitnikov < sitnikov.vladi...@gmail.com> wrote: > Philippe>1) Using IntelliJ IDEA 2021.2.3, I didn't get a popup to be able > to select: > Philippe> - Make sure Create separate module per source set is

Re: Questions following switch from Eclipse to IntelliJ

2021-11-14 Thread Vladimir Sitnikov
Philippe>1) Using IntelliJ IDEA 2021.2.3, I didn't get a popup to be able to select: Philippe> - Make sure Create separate module per source set is selected Philippe> - Make sure Use default gradle wrapper is selected Those steps indeed seem obsolete. I think nowadays the only question it

Questions following switch from Eclipse to IntelliJ

2021-11-14 Thread Philippe Mouawad
Hello, I have switched today to using IntelliJ as my future hopefully preferred IDE for JMeter. I followed below tutorial which is not up to date anymore I think: - https://github.com/apache/jmeter/blob/c7279348335b820c35ee570462cb2e0b4eb1c370/CONTRIBUTING.md 1) Using IntelliJ IDEA

Re: Should we drop TestSaveService.testFILEVERSION test and SaveService.FILEVERSION field?

2021-11-14 Thread Vladimir Sitnikov
Felix>So, in a sense, it is being used. (No idea, if it is useful enough. Has Felix>it done its purpose and reminded you to edit both files at once?) I find the nagging useless since FILEVERSION is the **only** thing that everybody changes when adding a new sampler :-/ If other modifications are

Re: Should we drop TestSaveService.testFILEVERSION test and SaveService.FILEVERSION field?

2021-11-14 Thread Felix Schumacher
Am 14.11.21 um 09:28 schrieb Vladimir Sitnikov: > Hi, > > I see that TestSaveService > testFILEVERSION fails on every update of > saveservice.properties. > What is the purpose of testFILEVERSION? I believe it is intended to nag the developers to keep the too files SaveService.java and

[GitHub] [jmeter] codecov-commenter edited a comment on pull request #674: WIP: Open Model Thread Group

2021-11-14 Thread GitBox
codecov-commenter edited a comment on pull request #674: URL: https://github.com/apache/jmeter/pull/674#issuecomment-968265204 #

[GitHub] [jmeter] codecov-commenter commented on pull request #674: WIP: Open Model Thread Group

2021-11-14 Thread GitBox
codecov-commenter commented on pull request #674: URL: https://github.com/apache/jmeter/pull/674#issuecomment-968265204 # [Codecov](https://codecov.io/gh/apache/jmeter/pull/674?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)

Re: Move "precise throughput computation" to thread group

2021-11-14 Thread Vladimir Sitnikov
Upd: I've added the relevant tests and CI now passes. I've updated the build configuration, so the build now passes without warnings with Gradle 7.3 and Java 17. SecurityManager has been deprecated for removal, so we might need to drop its use. Vladimir

Re: About SSL properties in JMeter and their use in Java / HC Impl and Recorder

2021-11-14 Thread Vladimir Sitnikov
Hi, I do not completely understand the use case, however, I support adding the common property names so different implementations could be configured in the same way. So please file the PR with the property names and the supported values. I don't understand what do you mean by

About SSL properties in JMeter and their use in Java / HC Impl and Recorder

2021-11-14 Thread Philippe Mouawad
Hello, I see few problems in this field in JMeter. 1) Cipher Suite is configurable through 2 properties: Documented one called "https.cipherSuites" in HttpSSLProtocolSocketFactory Undocumented one called "https.socket.ciphers" in LazyLayeredConnectionSocketFactory For HC4, the latter is the

Should we drop TestSaveService.testFILEVERSION test and SaveService.FILEVERSION field?

2021-11-14 Thread Vladimir Sitnikov
Hi, I see that TestSaveService > testFILEVERSION fails on every update of saveservice.properties. What is the purpose of testFILEVERSION? I suggest we just remove SaveService#FILEVERSION field as it is not really used. The failure looks as follows: FAILURE 0.0sec,