david-leifker

Jackson, opentelemetry, and other versions updated in main branch. Closing this one, feel free to open with neo4j updates if in a good state. Thanks!

Overall this looks very nice, thanks!

1. removal of htrace, removed shaded log4j impl, which caused an slf4 forwarder to point to log4j forwarder (infinite loop) 2. applied project wide transitive exclusions to directly pick versions...

Awesome PR here, this looks really good!! There are a couple missing pieces needed I think to round out the feature. 1. Remove access request - For example, there doesn't...

@pedro93 - pointed out an earlier set of PRs which actually somewhat duplicates this PR. It seems like there is an `access` aspect which was only added to `dataset` entity...

Closing this PR due to preference for the implementation in this #10061. Thank you for the PR and happy to work to address any limitations of the mentioned PR.

Definitely not finding this in the image. I would have expected it to be included with [this](https://github.com/datahub-project/datahub/blob/master/docker/datahub-ingestion-base/Dockerfile#L79C5-L79C142) step in the docker build.

I would go through the files being changed as the root user and then at the end change the permissions back to the normal datahub user. @gopikaops

@LucaDorinAnton - Can you confirm that the logout url is configured in your oidc provider? It should point to the login page `/login`? If the logout url is set to...

From okta, the configuration is `post_logout_redirect_uri`. The underlying logout when oidc is enabled is using the pac4j library which is following configuration from the idp. I believe `post_logout_redirect_uri` is also...