Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to pass arguments to delegate PostgreSQL JDBC Driver #2339

Closed
brunopalos opened this issue Feb 10, 2020 · 3 comments
Closed

Unable to pass arguments to delegate PostgreSQL JDBC Driver #2339

brunopalos opened this issue Feb 10, 2020 · 3 comments
Labels

Comments

@brunopalos
Copy link

brunopalos commented Feb 10, 2020

Hi,

Due to ?loggerLevel=OFF at the end of the constructed URL on PostgresSQL module, is not possible to pass arguments to the underlying driver, causing the connection to hang.

return "jdbc:postgresql://" + getContainerIpAddress() + ":" + getMappedPort(POSTGRESQL_PORT) + "/" + databaseName + "?loggerLevel=OFF";

When the URL is being constructed on the bellow line, the queryString already has a ? with the query arguments.

final String url = constructUrlForConnection(queryString);

Example: jdbc:postgresql://localhost:5432/my_database??loggerLevel=OFF?currentSchema=my_schema.

Removing ?loggerLevel=OFF doesn't seem to have any effect on the logs. If you agree, I'm happy to create a PR and remove the loggerLevel from the URL.

Thanks in advance,
Bruno

@stale
Copy link

stale bot commented May 10, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you believe this is a mistake, please reply to this comment to keep it open. If there isn't one already, a PR to fix or at least reproduce the problem in a test case will always help us get back on track to tackle this.

@stale stale bot added the stale label May 10, 2020
@stale
Copy link

stale bot commented May 24, 2020

This issue has been automatically closed due to inactivity. We apologise if this is still an active problem for you, and would ask you to re-open the issue if this is the case.

@stale stale bot closed this as completed May 24, 2020
@marcelstoer
Copy link

Looks like this was fixed with #2049.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants