Artem Zuikov

Results 20 comments of Artem Zuikov

I've fixed typo in comment. It's funny that it blocks tests. @bharatnc could you restart tests, please?

It looks like the crash in [PR / Upgrade check (amd_msan) (pull_request)](https://github.com/ClickHouse/ClickHouse/actions/runs/19127133880/job/54675401795?pr=88857) is not related to my changes. There's a "No changes in docs for new feature and no autogenerated...

> Am I only one person who feels nauseous from this JSON inside XML syntax mix? It's OK to place any hashes there for `password_*_hex` auth. What's the difference with...

> Also added support for dynamic JWKS workflow there Dynamic JWKS workflow is complicated. I understand why CH team does not want to merge it. I think we'd better try...

I've moved static_jwks into [named-collections](https://clickhouse.com/docs/operations/named-collections) config section. It looks like it's the best place for static JWK/JWKS.

Some changes from this PR are duplicated here #90581 (https://github.com/ClickHouse/ClickHouse/pull/90581/commits/ca2fee04c0953baa462332b696b42fbe5fe48999)

I'll make separate `AuthenticationType::JWKS` to resolve names conflict. It is required for ``` CREATE USER ... IDENTIFIED WITH jwks BY ``` `AuthenticationType::JWT` is reserved in code for ``` CREATE USER...

@bharatnc is there something wring with clang-tidy build? PR check fails do not look related to my changes

> This is clear per the spec: https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#decimal I didn't find it there. Is this md a point of truth for specs? Any case it's not in sync with https://parquet.apache.org/docs/file-format/types/logicaltypes/...