Quickstart getting-started/jdbc is missing Client Id and Client Secret

[pjanuario]

Describe the bug

As JDBC is being advertised as the prefered production deployment configuration and eclipse link is getting deprecated I was unable to follow the quickstart to launch a running instance following https://github.com/apache/polaris/tree/main/getting-started/jdbc#getting-started-with-apache-polaris-relational-jdbc-postgres-and-spark-sql

The client id and secret are missing.

To Reproduce

No response

Actual Behavior

polaris-1            | 2025-05-19 12:10:35,230 INFO  [org.apa.pol.ser.aut.TokenRequestValidator] [,POLARIS] [,,,] (executor-thread-1) Missing Client ID or Client Secret in Request Body
polaris-1            | 2025-05-19 12:10:35,247 INFO  [org.apa.pol.ser.qua.con.QuarkusJacksonConfig] [,POLARIS] [,,,] (executor-thread-1) Limiting request body size to 10485760 bytes
polaris-1            | 2025-05-19 12:10:35,274 INFO  [io.qua.htt.access-log] [,POLARIS] [,,,] (executor-thread-1) 172.18.0.4 - - [19/May/2025:12:10:35 +0000] "POST /api/catalog/v1/oauth/tokens HTTP/1.1" 400 87

Expected Behavior

The setup should proceed without errors

Additional context

No response

System information

OS: MacOS

[dimas-b]

Cf. #1610

[adnanhemani]

Orthogonal thought: Ideally, we should do the work to make JDBC the Quickstart experience instead of EclipseLink, which has all these things thought up of with much more details to make it a better user experience. I'm running behind on that task but if someone else would like to take it up, I'm happy to review.

[pjanuario]

Orthogonal thought: Ideally, we should do the work to make JDBC the Quickstart experience instead of EclipseLink, which has all these things thought up of with much more details to make it a better user experience. I'm running behind on that task but if someone else would like to take it up, I'm happy to review.

I am starting to play with the tool and I am having some trouble launching a instance that uses jdbc connected to postgres, somehow it always fallback to m2.

[adnanhemani]

@pjanuario is this after #1646?

[singhpk234]

No, i think its fixed !, closing this as well