[GitHub] [druid] himanshug commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib

himanshug commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib URL: https://github.com/apache/druid/pull/8992#issuecomment-609592760 @soumyajose0784 that sounds like that the ssl cert returned from server configured in `druid.auth.pac4j.oidc.discoveryURI`

[GitHub] [druid] himanshug commented on a change in pull request #9573: Fix some flaws of KafkaEmitter

himanshug commented on a change in pull request #9573: Fix some flaws of KafkaEmitter URL: https://github.com/apache/druid/pull/9573#discussion_r403854106 ## File path: extensions-contrib/kafka-emitter/src/main/java/org/apache/druid/emitter/kafka/KafkaEmitter.java ## @@ -

[GitHub] [druid] soumyajose0784 commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib

soumyajose0784 commented on issue #8992: druid extension for OpenID Connect auth using pac4j lib URL: https://github.com/apache/druid/pull/8992#issuecomment-609586100 We need the ssl property for common.runtime.properties, to enable connection to ssl enabled discovery URI. We get following

[druid] branch master updated: Eliminate common subfilters when converting it to a CNF (#9608)

This is an automated email from the ASF dual-hosted git repository. himanshug pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/master by this push: new 40e84a1 Eliminate common subfilters when conv

[GitHub] [druid] himanshug merged pull request #9608: Eliminate common subfilters when converting it to a CNF

himanshug merged pull request #9608: Eliminate common subfilters when converting it to a CNF URL: https://github.com/apache/druid/pull/9608 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [druid] himanshug commented on a change in pull request #9518: AWS RDS token based password provider

himanshug commented on a change in pull request #9518: AWS RDS token based password provider URL: https://github.com/apache/druid/pull/9518#discussion_r403831641 ## File path: extensions-core/druid-aws-rds-extensions/pom.xml ## @@ -0,0 +1,80 @@ + + + +http://maven.apache.

[GitHub] [druid] abhishekrb19 opened a new pull request #9622: Compaction numeric nullhandle

abhishekrb19 opened a new pull request #9622: Compaction numeric nullhandle URL: https://github.com/apache/druid/pull/9622 With `druid.generic.useDefaultValueForNull=false ` set and post compaction, we notice the dimensions with numeric types the original `null` value gets replaced with th

[GitHub] [druid] als-sdin closed issue #9386: Kafka Indexing Service failed - could not allocate segment for row with timestamp[*]

als-sdin closed issue #9386: Kafka Indexing Service failed - could not allocate segment for row with timestamp[*] URL: https://github.com/apache/druid/issues/9386 This is an automated message from the Apache Git Service. To

[GitHub] [druid] als-sdin commented on issue #9386: Kafka Indexing Service failed - could not allocate segment for row with timestamp[*]

als-sdin commented on issue #9386: Kafka Indexing Service failed - could not allocate segment for row with timestamp[*] URL: https://github.com/apache/druid/issues/9386#issuecomment-609506646 There were issues with the segments and/or segment metadata for a particular period. Nothing to do

[GitHub] [druid] himanshug merged pull request #9615: [Backport] Fix double count ssl connection metrics

himanshug merged pull request #9615: [Backport] Fix double count ssl connection metrics URL: https://github.com/apache/druid/pull/9615 This is an automated message from the Apache Git Service. To respond to the message, plea

[druid] branch 0.18.0 updated: Fix double count ssl connection metrics (#9594) (#9615)

This is an automated email from the ASF dual-hosted git repository. himanshug pushed a commit to branch 0.18.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.18.0 by this push: new 404a558 Fix double count ssl connection metri

[GitHub] [druid] lgtm-com[bot] commented on issue #9621: Add context to AuthenticationResult [WIP]

lgtm-com[bot] commented on issue #9621: Add context to AuthenticationResult [WIP] URL: https://github.com/apache/druid/pull/9621#issuecomment-609426018 This pull request **introduces 3 alerts** when merging 66ac801e348fe30fbfd703f7c1be9ae7bb8e99e8 into 2d999669333fc7bd344e5bec0d74f0fbcd91

[GitHub] [druid] bolkedebruin opened a new pull request #9621: Add context to AuthenticationResult [WIP]

bolkedebruin opened a new pull request #9621: Add context to AuthenticationResult [WIP] URL: https://github.com/apache/druid/pull/9621 ### Description Looking for feedback. Proper authorization needs to be context aware. Context like IP Address, Proxy addresses etc. This PR a