Re: [PR] [CXF-9105] add synchronized to the principal iteration [cxf]

2025-02-01 Thread via GitHub
reta merged PR #2245: URL: https://github.com/apache/cxf/pull/2245 -- 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: dev-unsubscr...@cxf.apache.org For q

Re: [PR] [CXF-9105] add synchronized to the principal iteration [cxf]

2025-01-31 Thread via GitHub
sstremler commented on PR #2245: URL: https://github.com/apache/cxf/pull/2245#issuecomment-2628556571 > Thank you @sstremler , a few comments but LGTM otherwise! Thank you, I added the changes you suggested. -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] [CXF-9105] add synchronized to the principal iteration [cxf]

2025-01-31 Thread via GitHub
reta commented on PR #2245: URL: https://github.com/apache/cxf/pull/2245#issuecomment-2628539485 Thank you @sstremler , a few comments but LGTM otherwise! -- 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 t

Re: [PR] [CXF-9105] add synchronized to the principal iteration [cxf]

2025-01-31 Thread via GitHub
reta commented on code in PR #2245: URL: https://github.com/apache/cxf/pull/2245#discussion_r1938070843 ## rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java: ## @@ -125,31 +126,36 @@ private String getPrincipal(Message message) {

Re: [PR] [CXF-9105] add synchronized to the principal iteration [cxf]

2025-01-31 Thread via GitHub
reta commented on code in PR #2245: URL: https://github.com/apache/cxf/pull/2245#discussion_r1938068328 ## rt/features/logging/src/main/java/org/apache/cxf/ext/logging/event/DefaultLogEventMapper.java: ## @@ -125,31 +126,36 @@ private String getPrincipal(Message message) {