wangyum opened a new pull request, #7317: URL: https://github.com/apache/kyuubi/pull/7317
### Why are the changes needed? Fixed the `parsePropertyFromUrl` method in `Utils.java` to properly remove query string parts (everything after ?) from parsed JDBC URL property values, making it consistent with the `extractURLComponents` method. When parsing JDBC URLs with query strings like: `jdbc:hive2://host:10012/db;auth=JWT?kyuubi.session.cluster=clusterA`. The `parsePropertyFromUrl` method was incorrectly returning `JWT?kyuubi.session.cluster=clusterA` for the `auth` property, while `extractURLComponents` correctly returned only `JWT`. This inconsistency could cause authentication failures in JWT auth scenarios with additional query parameters. ### How was this patch tested? 1. URLs with query strings parse properties correctly without the query string 2. URLs without query strings continue to work as expected 3. Both parsing methods return consistent results ### Was this patch authored or co-authored using generative AI tooling? No. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
